[Dovecot] test80: assert/core debug info

[Jeff A. Earickson]

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 <tss at iki.fi>
> To: Jeff A. Earickson <jaearick at colby.edu>
> Cc: dovecot at 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
>