Timo, Thank you, the patch solved the problem of core dumps and asserts. There have been none in the last eight hours since I applied the patch. The syslog output has much fewer complaints. There are occasional mail.error syslog warnings like: IMAP(user): UIDs broken with partial sync in mbox file /var/mail/user and one occurance of: IMAP(user): mbox sync: UID inserted in the middle of mailbox /var/mail/user (195 > 34, seq=2, idx_msgs=31) Anything to worry about? Jeff Earickson Colby College On Wed, 17 Aug 2005, Timo Sirainen wrote:
Date: Wed, 17 Aug 2005 02:32:20 +0300 From: Timo Sirainen
To: Jeff A. Earickson Cc: dovecot@dovecot.org Subject: Re: [Dovecot] test80: assert/core debug info On 16.8.2005, at 18:14, Jeff A. Earickson wrote:
Attached is gdb information from core dumps related to the following assert in test-80:
IMAP(username): file mbox-sync-update.c: line 442 (mbox_sync_update_header_from): assertion failed: (ctx->mail.uid == 0 || ctx->mail.uid_broken || ctx->mail.uid == mail->uid) ..
This patch will fix it I think?
--- src/lib-storage/index/mbox/mbox-sync-parse.c 7 Aug 2005 11:28:13 -0000 1.44 +++ src/lib-storage/index/mbox/mbox-sync-parse.c 16 Aug 2005 23:30:00 -0000 @@ -342,7 +342,7 @@ return TRUE; }
- if (ctx->sync_ctx->seq == 1 && !ctx->seen_imapbase) { + if (ctx->seq == 1 && !ctx->seen_imapbase) { /* Don't bother allowing X-UID before X-IMAPbase header. c-client doesn't allow it either, and this way the UID doesn't have to be reset if X-IMAPbase