SOURCES: dovecot-hg-5253-5288.patch (NEW) - patch from hg
arekm
arekm at pld-linux.org
Sun May 27 16:24:13 CEST 2007
Author: arekm Date: Sun May 27 14:24:13 2007 GMT
Module: SOURCES Tag: HEAD
---- Log message:
- patch from hg
---- Files affected:
SOURCES:
dovecot-hg-5253-5288.patch (NONE -> 1.1) (NEW)
---- Diffs:
================================================================
Index: SOURCES/dovecot-hg-5253-5288.patch
diff -u /dev/null SOURCES/dovecot-hg-5253-5288.patch:1.1
--- /dev/null Sun May 27 16:24:13 2007
+++ SOURCES/dovecot-hg-5253-5288.patch Sun May 27 16:24:08 2007
@@ -0,0 +1,908 @@
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/auth/auth-request-handler.c
+--- a/src/auth/auth-request-handler.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/auth/auth-request-handler.c Sun May 27 16:45:26 2007 +0300
+@@ -101,7 +101,7 @@ void auth_request_handler_check_timeouts
+ while (hash_iterate(iter, &key, &value)) {
+ struct auth_request *request = value;
+
+- if (request->created + AUTH_REQUEST_TIMEOUT < ioloop_time)
++ if (request->last_access + AUTH_REQUEST_TIMEOUT < ioloop_time)
+ auth_request_handler_remove(handler, request);
+ }
+ hash_iterate_deinit(iter);
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/auth/auth-request.c
+--- a/src/auth/auth-request.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/auth/auth-request.c Sun May 27 16:45:26 2007 +0300
+@@ -33,7 +33,7 @@ auth_request_new(struct auth *auth, stru
+ request->userdb = auth->userdbs;
+
+ request->refcount = 1;
+- request->created = ioloop_time;
++ request->last_access = ioloop_time;
+ request->credentials = -1;
+
+ request->auth = auth;
+@@ -53,7 +53,7 @@ struct auth_request *auth_request_new_du
+ auth_request->pool = pool;
+
+ auth_request->refcount = 1;
+- auth_request->created = ioloop_time;
++ auth_request->last_access = ioloop_time;
+ auth_request->auth = auth;
+ auth_request->passdb = auth->passdbs;
+ auth_request->userdb = auth->userdbs;
+@@ -75,6 +75,7 @@ void auth_request_success(struct auth_re
+
+ request->state = AUTH_REQUEST_STATE_FINISHED;
+ request->successful = TRUE;
++ request->last_access = ioloop_time;
+ request->callback(request, AUTH_CLIENT_RESULT_SUCCESS,
+ data, data_size);
+ }
+@@ -84,6 +85,7 @@ void auth_request_fail(struct auth_reque
+ i_assert(request->state == AUTH_REQUEST_STATE_MECH_CONTINUE);
+
+ request->state = AUTH_REQUEST_STATE_FINISHED;
++ request->last_access = ioloop_time;
+ request->callback(request, AUTH_CLIENT_RESULT_FAILURE, NULL, 0);
+ }
+
+@@ -133,6 +135,8 @@ void auth_request_export(struct auth_req
+ str_append(str, "\trip=");
+ str_append(str, net_ip2addr(&request->remote_ip));
+ }
++ if (request->secured)
++ str_append(str, "\tsecured=1");
+ }
+
+ bool auth_request_import(struct auth_request *request,
+@@ -177,6 +181,7 @@ void auth_request_continue(struct auth_r
+ {
+ i_assert(request->state == AUTH_REQUEST_STATE_MECH_CONTINUE);
+
++ request->last_access = ioloop_time;
+ request->mech->auth_continue(request, data, data_size);
+ }
+
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/auth/auth-request.h
+--- a/src/auth/auth-request.h Fri Apr 13 14:41:56 2007 +0300
++++ b/src/auth/auth-request.h Sun May 27 16:45:26 2007 +0300
+@@ -54,7 +54,7 @@ struct auth_request {
+ unsigned int connect_uid;
+ unsigned int client_pid;
+ unsigned int id;
+- time_t created;
++ time_t last_access;
+
+ const char *service;
+ struct ip_addr local_ip, remote_ip;
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/deliver/deliver.c
+--- a/src/deliver/deliver.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/deliver/deliver.c Sun May 27 16:45:26 2007 +0300
+@@ -158,7 +158,8 @@ const char *deliver_get_return_address(s
+ message_address_parse(pool_datastack_create(),
+ (const unsigned char *)str,
+ strlen(str), 1, FALSE);
+- return addr == NULL || addr->mailbox == NULL || addr->domain == NULL ?
++ return addr == NULL || addr->mailbox == NULL || addr->domain == NULL ||
++ *addr->mailbox == '\0' || *addr->domain == '\0' ?
+ NULL : t_strconcat(addr->mailbox, "@", addr->domain, NULL);
+ }
+
+@@ -276,6 +277,15 @@ static void config_file_init(const char
+ }
+ if (setting_is_bool(key) && strcasecmp(value, "yes") != 0)
+ continue;
++
++ if (strcasecmp(key, "mail_debug") == 0) {
++ /* We'll internally use only DEBUG environment.
++ Postfix's sendmail binary uses MAIL_DEBUG
++ environment for different purposes and we really
++ don't want to enable it. */
++ env_put("DEBUG=1");
++ continue;
++ }
+
+ env_put(t_strconcat(t_str_ucase(key), "=", value, NULL));
+ }
+@@ -564,9 +574,6 @@ int main(int argc, char *argv[])
+ config_file_init(config_path);
+ open_logfile(user);
+
+- if (getenv("MAIL_DEBUG") != NULL)
+- env_put("DEBUG=1");
+-
+ if (getenv("MAIL_PLUGINS") == NULL)
+ modules = NULL;
+ else {
+@@ -631,17 +638,20 @@ int main(int argc, char *argv[])
+ mail_storage_init();
+ mail_storage_register_all();
+
+- /* MAIL comes from userdb, MAIL_LOCATION from dovecot.conf */
++ /* MAIL comes from userdb, MAIL_LOCATION from dovecot.conf.
++ We don't want to expand settings coming from userdb. */
+ mail_env = getenv("MAIL");
+- if (mail_env == NULL)
++ if (mail_env == NULL) {
+ mail_env = getenv("MAIL_LOCATION");
+- if (mail_env == NULL) {
+- /* Keep this for backwards compatibility */
+- mail_env = getenv("DEFAULT_MAIL_ENV");
+- }
+- if (mail_env != NULL) {
+- table = get_var_expand_table(destination, getenv("HOME"));
+- mail_env = expand_mail_env(mail_env, table);
++ if (mail_env == NULL) {
++ /* Keep this for backwards compatibility */
++ mail_env = getenv("DEFAULT_MAIL_ENV");
++ }
++ if (mail_env != NULL) {
++ table = get_var_expand_table(destination,
++ getenv("HOME"));
++ mail_env = expand_mail_env(mail_env, table);
++ }
+ }
+
+ module_dir_init(modules);
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/deliver/smtp-client.c
+--- a/src/deliver/smtp-client.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/deliver/smtp-client.c Sun May 27 16:45:26 2007 +0300
+@@ -27,28 +27,6 @@ static struct smtp_client *smtp_client_d
+ return client;
+ }
+
+-static void smtp_env_clean(void)
+-{
+- extern char **environ;
+- char **p;
+- array_t ARRAY_DEFINE(new_env, char *);
+- unsigned int i, count;
+-
+- /* copy the environment, but drop out the unwanted fields. */
+- ARRAY_CREATE(&new_env, pool_datastack_create(), char *, 128);
+- for (p = environ; *p != NULL; p++) {
+- if (strncmp(*p, "DEBUG=", 6) != 0)
+- array_append(&new_env, p, 1);
+- }
+-
+- /* then recreate the environment */
+- env_clean();
+-
+- p = array_get_modifyable(&new_env, &count);
+- for (i = 0; i < count; i++)
+- env_put(p[i]);
+-}
+-
+ static void smtp_client_run_sendmail(const char *destination,
+ const char *return_path, int fd)
+ {
+@@ -65,14 +43,6 @@ static void smtp_client_run_sendmail(con
+
+ if (dup2(fd, STDIN_FILENO) < 0)
+ i_fatal("dup2() failed: %m");
+-
+- if (getenv("DEBUG") != NULL) {
+- /* Postfix's sendmail binary uses DEBUG environment for its own
+- purposes, which pretty much break things. Remove it before
+- continuing. FIXME: Perhaps the whole environment could be
+- cleaned? */
+- smtp_env_clean();
+- }
+
+ (void)execv(deliver_set->sendmail_path, (char **)argv);
+ i_fatal("execv(%s) failed: %m", deliver_set->sendmail_path);
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/imap/cmd-copy.c
+--- a/src/imap/cmd-copy.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/imap/cmd-copy.c Sun May 27 16:45:26 2007 +0300
+@@ -12,13 +12,14 @@
+
+ static void client_send_sendalive_if_needed(struct client *client)
+ {
+- time_t now;
++ time_t now, last_io;
+
+ if (o_stream_get_buffer_used_size(client->output) != 0)
+ return;
+
+ now = time(NULL);
+- if (now - client->last_output > MAIL_STORAGE_STAYALIVE_SECS) {
++ last_io = I_MAX(client->last_input, client->last_output);
++ if (now - last_io > MAIL_STORAGE_STAYALIVE_SECS) {
+ o_stream_send_str(client->output, "* OK Hang in there..\r\n");
+ o_stream_flush(client->output);
+ client->last_output = now;
+@@ -49,7 +50,7 @@ static int fetch_and_copy(struct client
+ break;
+ }
+
+- if ((++copy_count % COPY_CHECK_INTERVAL) != 0)
++ if ((++copy_count % COPY_CHECK_INTERVAL) == 0)
+ client_send_sendalive_if_needed(client);
+
+ keywords_list = mail_get_keywords(mail);
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-index/mail-cache-compress.c
+--- a/src/lib-index/mail-cache-compress.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-index/mail-cache-compress.c Sun May 27 16:45:26 2007 +0300
+@@ -123,7 +123,7 @@ mail_cache_copy(struct mail_cache *cache
+ struct mail_cache_record cache_rec;
+ struct ostream *output;
+ buffer_t *buffer;
+- uint32_t i, message_count, seq, first_new_seq, old_offset;
++ uint32_t i, message_count, seq, first_new_seq, ext_offset, old_offset;
+ uoff_t offset;
+
+ /* get sequence of first message which doesn't need its temp fields
+@@ -179,7 +179,8 @@ mail_cache_copy(struct mail_cache *cache
+ if (cache_rec.size == sizeof(cache_rec))
+ continue;
+
+- mail_index_update_ext(t, seq, cache->ext_id, &output->offset,
++ ext_offset = output->offset;
++ mail_index_update_ext(t, seq, cache->ext_id, &ext_offset,
+ &old_offset);
+
+ buffer_write(ctx.buffer, 0, &cache_rec, sizeof(cache_rec));
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-index/mail-index-lock.c
+--- a/src/lib-index/mail-index-lock.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-index/mail-index-lock.c Sun May 27 16:45:26 2007 +0300
+@@ -455,8 +455,11 @@ static void mail_index_write_map(struct
+ } else {
+ /* write the modified parts. header is small enough to be
+ always written, write_seq_* specifies the record range. */
+- if (mail_index_write_map_over(index) < 0)
++ if (mail_index_write_map_over(index) < 0) {
++ mail_index_set_error(index,
++ "pwrite_full(%s) failed: %m", index->filepath);
+ mail_index_set_inconsistent(index);
++ }
+ }
+
+ map->write_to_disk = FALSE;
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-mail/istream-header-filter.c
+--- a/src/lib-mail/istream-header-filter.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-mail/istream-header-filter.c Sun May 27 16:45:26 2007 +0300
+@@ -70,6 +70,16 @@ static ssize_t read_header(struct header
+ mstream->istream.istream.v_offset +
+ (mstream->istream.pos - mstream->istream.skip) ==
+ mstream->header_size.virtual_size) {
++ /* if there's no body at all, return EOF */
++ (void)i_stream_get_data(mstream->input, &pos);
++ if (pos == 0) {
++ ret = i_stream_read(mstream->input);
++ if (ret == -1) {
++ /* EOF */
++ mstream->istream.istream.eof = TRUE;
++ return -1;
++ }
++ }
+ /* we don't support mixing headers and body.
+ it shouldn't be needed. */
+ return -2;
+@@ -180,11 +190,13 @@ static ssize_t read_header(struct header
+ }
+
+ if (ret == 0) {
++ /* we're at the end of headers. */
+ i_assert(hdr == NULL);
+ i_assert(mstream->istream.istream.v_offset +
+ mstream->istream.pos ==
+ mstream->header_size.virtual_size);
+- return -2;
++
++ return read_header(mstream);
+ }
+
+ return ret;
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-storage/index/index-mail-headers.c
+--- a/src/lib-storage/index/index-mail-headers.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-storage/index/index-mail-headers.c Sun May 27 16:45:26 2007 +0300
+@@ -358,8 +358,7 @@ int index_mail_parse_headers(struct inde
+
+ index_mail_parse_header_init(mail, headers);
+
+- if (data->parser_ctx == NULL && (data->parts == NULL ||
+- data->save_bodystructure_header)) {
++ if (data->parts == NULL || data->save_bodystructure_header) {
+ /* initialize bodystructure parsing in case we read the whole
+ message. */
+ if (data->parser_ctx != NULL)
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-storage/index/maildir/maildir-storage.c
+--- a/src/lib-storage/index/maildir/maildir-storage.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-storage/index/maildir/maildir-storage.c Sun May 27 16:45:26 2007 +0300
+@@ -552,6 +552,10 @@ maildir_open(struct maildir_storage *sto
+ }
+
+ index_storage_mailbox_init(&mbox->ibox, name, flags, FALSE);
++
++ if (access(t_strconcat(path, "/cur", NULL), W_OK) < 0 &&
++ errno == EACCES)
++ mbox->ibox.readonly = TRUE;
+ return &mbox->ibox.box;
+ }
+
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-storage/index/maildir/maildir-sync.c
+--- a/src/lib-storage/index/maildir/maildir-sync.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-storage/index/maildir/maildir-sync.c Sun May 27 16:45:26 2007 +0300
+@@ -182,6 +182,7 @@
+
+ #include <stdio.h>
+ #include <stddef.h>
++#include <stdlib.h>
+ #include <unistd.h>
+ #include <dirent.h>
+ #include <sys/stat.h>
+@@ -298,13 +299,20 @@ int maildir_filename_get_flags(struct ma
+ return 1;
+ }
+
++static int char_cmp(const void *p1, const void *p2)
++{
++ const unsigned char *c1 = p1, *c2 = p2;
++
++ return *c1 - *c2;
++}
++
+ static void
+ maildir_filename_append_keywords(struct maildir_keywords_sync_ctx *ctx,
+ array_t *keywords, string_t *str)
+ {
+ ARRAY_SET_TYPE(keywords, unsigned int);
+ const unsigned int *indexes;
+- unsigned int i, count;
++ unsigned int i, count, start = str_len(str);
+ char chr;
+
+ indexes = array_get(keywords, &count);
+@@ -313,6 +321,8 @@ maildir_filename_append_keywords(struct
+ if (chr != '\0')
+ str_append_c(str, chr);
+ }
++
++ qsort(str_c_modifyable(str) + start, str_len(str) - start, 1, char_cmp);
+ }
+
+ const char *maildir_filename_set_flags(struct maildir_keywords_sync_ctx *ctx,
+@@ -756,7 +766,8 @@ static int maildir_fix_duplicate(struct
+ i_warning("Fixed a duplicate: %s -> %s", old_fname, new_fname);
+ else if (errno != ENOENT) {
+ mail_storage_set_critical(STORAGE(mbox->storage),
+- "rename(%s, %s) failed: %m", old_path, new_path);
++ "Couldn't fix a duplicate: rename(%s, %s) failed: %m",
++ old_path, new_path);
+ ret = -1;
+ }
+ t_pop();
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-storage/index/maildir/maildir-uidlist.c
+--- a/src/lib-storage/index/maildir/maildir-uidlist.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-storage/index/maildir/maildir-uidlist.c Sun May 27 16:45:26 2007 +0300
+@@ -347,6 +347,11 @@ maildir_uidlist_update_read(struct maild
+ "%s: next_uid was lowered (%u -> %u)",
+ uidlist->fname, uidlist->next_uid, next_uid);
+ ret = 0;
++ } else if (uid_validity == 0 || next_uid == 0) {
++ mail_storage_set_critical(storage,
++ "%s: Broken header (uidvalidity = %u, next_uid=%u)",
++ uidlist->fname, uid_validity, next_uid);
++ ret = 0;
+ } else {
+ uidlist->uid_validity = uid_validity;
+ uidlist->next_uid = next_uid;
+@@ -543,7 +548,8 @@ void maildir_uidlist_set_uid_validity(st
+ uint32_t uid_validity, uint32_t next_uid)
+ {
+ uidlist->uid_validity = uid_validity;
+- if (next_uid != 0)
++ /* set next_uid only if we know newer UIDs haven't been added yet */
++ if (uidlist->next_uid < next_uid)
+ uidlist->next_uid = next_uid;
+ }
+
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-storage/index/maildir/maildir-util.c
+--- a/src/lib-storage/index/maildir/maildir-util.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-storage/index/maildir/maildir-util.c Sun May 27 16:45:26 2007 +0300
+@@ -42,6 +42,23 @@ static int maildir_file_do_try(struct ma
+ return ret;
+ }
+
++static int do_racecheck(struct maildir_mailbox *mbox, const char *path,
++ void *context __attr_unused__)
++{
++ struct stat st;
++
++ if (lstat(path, &st) == 0 && (st.st_mode & S_IFLNK) != 0) {
++ /* most likely a symlink pointing to a non-existing file */
++ mail_storage_set_critical(STORAGE(mbox->storage),
++ "Maildir: Symlink destination doesn't exist: %s", path);
++ return -2;
++ } else {
++ mail_storage_set_critical(STORAGE(mbox->storage),
++ "maildir_file_do(%s): Filename keeps changing", path);
++ return -1;
++ }
++}
++
+ int maildir_file_do(struct maildir_mailbox *mbox, uint32_t uid,
+ maildir_file_do_func *func, void *context)
+ {
+@@ -58,11 +75,8 @@ int maildir_file_do(struct maildir_mailb
+ ret = maildir_file_do_try(mbox, uid, func, context);
+ }
+
+- if (i == 10) {
+- ret = -1;
+- mail_storage_set_critical(STORAGE(mbox->storage),
+- "maildir_file_do(%s) racing", mbox->path);
+- }
++ if (i == 10)
++ ret = maildir_file_do_try(mbox, uid, do_racecheck, context);
+
+ return ret == -2 ? 0 : ret;
+ }
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib-storage/index/mbox/mbox-save.c
+--- a/src/lib-storage/index/mbox/mbox-save.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib-storage/index/mbox/mbox-save.c Sun May 27 16:45:26 2007 +0300
+@@ -48,6 +48,7 @@ struct mbox_save_context {
+ char last_char;
+
+ struct mbox_md5_context *mbox_md5_ctx;
++ unsigned int x_delivery_id_pos;
+
+ unsigned int synced:1;
+ unsigned int failed:1;
+@@ -304,12 +305,6 @@ mbox_save_init_file(struct mbox_save_con
+ syncing. */
+ mbox_save_init_sync(t);
+ }
+-
+- if (mbox_seek_to_end(ctx, &ctx->append_offset) < 0)
+- return -1;
+-
+- ctx->output = o_stream_create_file(mbox->mbox_fd, default_pool,
+- 0, FALSE);
+ }
+
+ if (!ctx->synced && want_mail) {
+@@ -319,6 +314,14 @@ mbox_save_init_file(struct mbox_save_con
+ mbox_save_init_sync(t);
+ }
+
++ /* the syncing above could have changed the append offset */
++ if (ctx->append_offset == (uoff_t)-1) {
++ if (mbox_seek_to_end(ctx, &ctx->append_offset) < 0)
++ return -1;
++
++ ctx->output = o_stream_create_file(mbox->mbox_fd, default_pool,
++ 0, FALSE);
++ }
+ return 0;
+ }
+
+@@ -368,6 +371,7 @@ static void mbox_save_x_delivery_id(stru
+ md5_get_digest(buf->data, buf->used, md5_result);
+
+ str_append(ctx->headers, "X-Delivery-ID: ");
++ ctx->x_delivery_id_pos = str_len(ctx->headers);
+ base64_encode(md5_result, sizeof(md5_result), ctx->headers);
+ str_append_c(ctx->headers, '\n');
+ t_pop();
+@@ -413,6 +417,8 @@ int mbox_save_init(struct mailbox_transa
+ save_flags = (flags & ~MAIL_RECENT) | MAIL_RECENT;
+ str_truncate(ctx->headers, 0);
+ if (ctx->synced) {
++ if (ctx->mbox->mbox_save_md5)
++ ctx->mbox_md5_ctx = mbox_md5_init();
+ if (ctx->output->offset == 0) {
+ /* writing the first mail. Insert X-IMAPbase as well. */
+ str_printfa(ctx->headers, "X-IMAPbase: %u %010u\n",
+@@ -477,8 +483,6 @@ int mbox_save_init(struct mailbox_transa
+ MAIL_STORAGE_FLAG_SAVE_CRLF) != 0 ?
+ o_stream_create_crlf(default_pool, ctx->output) :
+ o_stream_create_lf(default_pool, ctx->output);
+- if (ctx->mbox->mbox_save_md5 && ctx->synced)
+- ctx->mbox_md5_ctx = mbox_md5_init();
+ }
+
+ *ctx_r = &ctx->ctx;
+@@ -550,6 +554,22 @@ int mbox_save_continue(struct mail_save_
+ if (ctx->mbox_md5_ctx) {
+ unsigned char hdr_md5_sum[16];
+
++ if (ctx->x_delivery_id_pos != 0) {
++ struct message_header_line hdr;
++ const unsigned char *p;
++
++ memset(&hdr, 0, sizeof(hdr));
++ hdr.name = "X-Delivery-ID";
++ hdr.name_len = strlen(hdr.name);
++ hdr.middle = (const unsigned char *)": ";
++ hdr.middle_len = 2;
++ hdr.value = hdr.full_value = str_data(ctx->headers) +
++ ctx->x_delivery_id_pos;
++
++ for (p = hdr.value; *p != '\n'; p++) ;
++ hdr.value_len = hdr.full_value_len = p - hdr.value;
++ mbox_md5_continue(ctx->mbox_md5_ctx, &hdr);
++ }
+ mbox_md5_finish(ctx->mbox_md5_ctx, hdr_md5_sum);
+ mail_index_update_ext(ctx->trans, ctx->seq,
+ ctx->mbox->ibox.md5hdr_ext_idx,
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib/file-dotlock.c
+--- a/src/lib/file-dotlock.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib/file-dotlock.c Sun May 27 16:45:26 2007 +0300
+@@ -521,7 +521,12 @@ int file_dotlock_create(const struct dot
+ /* some NFS implementations may have used cached mtime in previous
+ fstat() call. Check again to avoid "dotlock was modified" errors. */
+ if (stat(lock_path, &st) < 0) {
+- i_error("stat(%s) failed: %m", lock_path);
++ if (errno != ENOENT)
++ i_error("stat(%s) failed: %m", lock_path);
++ else {
++ i_error("dotlock %s was immediately deleted under us",
++ lock_path);
++ }
+ file_dotlock_free(dotlock);
+ t_pop();
+ return -1;
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib/ioloop.c
+--- a/src/lib/ioloop.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib/ioloop.c Sun May 27 16:45:26 2007 +0300
+@@ -237,10 +237,13 @@ void io_loop_handle_timeouts(struct iolo
+ if (diff > IOLOOP_MAX_TIME_BACKWARDS_SLEEP) {
+ i_fatal("Time just moved backwards by %ld seconds. "
+ "This might cause a lot of problems, "
+- "so I'll just kill myself now.", (long)diff);
++ "so I'll just kill myself now. "
++ "http://wiki.dovecot.org/TimeMovedBackwards",
++ (long)diff);
+ } else {
+ i_error("Time just moved backwards by %ld seconds. "
+- "I'll sleep now until we're back in present.",
++ "I'll sleep now until we're back in present. "
++ "http://wiki.dovecot.org/TimeMovedBackwards",
+ (long)diff);
+ /* Sleep extra second to make sure usecs also grows. */
+ diff++;
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/lib/mountpoint.c
+--- a/src/lib/mountpoint.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/lib/mountpoint.c Sun May 27 16:45:26 2007 +0300
+@@ -58,7 +58,11 @@ int mountpoint_get(const char *path, poo
+
+ point_r->device_path = p_strdup(pool, buf.f_mntfromname);
+ point_r->mount_path = p_strdup(pool, buf.f_mntonname);
++#ifdef __osf__ /* Tru64 */
++ point_r->type = p_strdup(pool, getvfsbynumber(buf.f_type));
++#else
+ point_r->type = p_strdup(pool, buf.f_fstypename);
++#endif
+ point_r->block_size = buf.f_bsize;
+ return 1;
+ #else
+diff -r 96fb9dad0a0d -r 2b462faf9867 src/master/main.c
+--- a/src/master/main.c Fri Apr 13 14:41:56 2007 +0300
++++ b/src/master/main.c Sun May 27 16:45:26 2007 +0300
+@@ -218,6 +218,9 @@ static void sigchld_handler(int signo __
+ i_error("unknown child %s exited "
+ "successfully", dec2str(pid));
+ }
++ } else if (status == 1 &&
++ process_type == PROCESS_TYPE_SSL_PARAM) {
++ /* kludgy. hide this failure. */
+ } else {
<<Diff was trimmed, longer than 597 lines>>
More information about the pld-cvs-commit
mailing list