RE: [Dovecot] APPEND bug in dovecot-20060628
Timo,
I'm running maildir quotas with rc1 (dovecot-20060628) and get the following when trying to APPEND: "NO Unknown error" System logs show: Quota: Couldn't get new message's size
.... and the file isn't there (unknown if Dovecot removed it, or if the lack of the file is the cause of the error).
I am seeing this problem as well. Running this morning's CVS.
It is hitting this condition on line 229 of src/lib-storage/index/maildir/maildir-mail.c, in function maildir_mail_get_physical_size():
   /* size can be included in filename */
    if (!maildir_filename_get_size(fname, MAILDIR_EXTRA_FILE_SIZE, &size)) {
            if (_mail->uid != 0) {
                    if (maildir_file_do(mbox, _mail->uid,
                                        do_stat, &st) <= 0) {
                            return (uoff_t)-1;
                    }
            } else {
                    /* saved mail which hasn't been committed yet */
                    if (do_stat(mbox, fname, &st) <= 0) {
line 229 >>>>>> return (uoff_t)-1; } } size = st.st_size; }
On Thu, 2006-06-29 at 18:11 -0400, Bill Boebel wrote:
Timo,
I'm running maildir quotas with rc1 (dovecot-20060628) and get the following when trying to APPEND: "NO Unknown error" System logs show: Quota: Couldn't get new message's size
.... and the file isn't there (unknown if Dovecot removed it, or if the lack of the file is the cause of the error).
I am seeing this problem as well. Running this morning's CVS.
It is hitting this condition on line 229 of src/lib-storage/index/maildir/maildir-mail.c, in function maildir_mail_get_physical_size():
/* size can be included in filename */ if (!maildir_filename_get_size(fname, MAILDIR_EXTRA_FILE_SIZE, &size)) { if (_mail->uid != 0) { if (maildir_file_do(mbox, _mail->uid, do_stat, &st) <= 0) { return (uoff_t)-1; } } else { /* saved mail which hasn't been committed yet */ if (do_stat(mbox, fname, &st) <= 0) {line 229 >>>>>> return (uoff_t)-1;
Right. Fix here:
participants (2)
- 
                
                Bill Boebel
 - 
                
                Timo Sirainen