I opened this issue
https://bugzilla.andrew.cmu.edu/show_bug.cgi?id=2986
The problem is in the openpkg patch cyrus-imapd-2.3.7-morelogging.patch
in the patched imapd.c
-------------------------------------------------------------
index_closemailbox(imapd_mailbox);
mailbox_close(imapd_mailbox);
imapd_mailbox = 0;
if (r) {
prot_printf(imapd_out, "%s NO %s\r\n", tag, error_message(r));
}
else {
prot_printf(imapd_out, "%s OK %s\r\n", tag,
error_message(IMAP_OK_COMPLETED));
>> syslog(LOG_DEBUG, "cmd_expunge: user %s, mailbox %s, sequence %s",
imapd_userid, imapd_mailbox->name, "''");
}
-------------------------------------------------------------
You can see the use of "imapd_mailbox->name" just after "imapd_mailbox = 0;"
was set to NULL !
I propose move the 3 lines of code after the logging.
I looked at other part of the patch, they look ok.
Here is my patch that should replace it
Regards
-- snip -----------------------------------------
diff -Naur cyrus-imapd-2.3.9.orig/imap/append.c cyrus-imapd-2.3.9/imap/append.c
--- cyrus-imapd-2.3.9.orig/imap/append.c 2007-02-05
19:41:45.000000000 +0100
+++ cyrus-imapd-2.3.9/imap/append.c 2007-08-31 17:14:47.000000000 +0200
@@ -654,6 +654,9 @@
/* ok, we've successfully added a message */
as->quota_used += message_index.size;
+ syslog(LOG_DEBUG, "append_fromstage: message %d added to %s",
+ message_index.uid, mailbox->name );
+
return 0;
}
diff -Naur cyrus-imapd-2.3.9.orig/imap/imapd.c cyrus-imapd-2.3.9/imap/imapd.c
--- cyrus-imapd-2.3.9.orig/imap/imapd.c 2007-08-02 16:18:51.000000000 +0200
+++ cyrus-imapd-2.3.9/imap/imapd.c 2007-08-31 17:16:45.000000000 +0200
@@ -3504,17 +3504,19 @@
if (!r) sync_log_mailbox(imapd_mailbox->name);
}
- index_closemailbox(imapd_mailbox);
- mailbox_close(imapd_mailbox);
- imapd_mailbox = 0;
-
if (r) {
prot_printf(imapd_out, "%s NO %s\r\n", tag, error_message(r));
}
else {
prot_printf(imapd_out, "%s OK %s\r\n", tag,
error_message(IMAP_OK_COMPLETED));
+ syslog(LOG_DEBUG, "cmd_expunge: user %s, mailbox %s, sequence %s",
+ imapd_userid, imapd_mailbox->name, "''");
}
+
+ index_closemailbox(imapd_mailbox);
+ mailbox_close(imapd_mailbox);
+ imapd_mailbox = 0;
}
/*
diff -Naur cyrus-imapd-2.3.9.orig/imap/mailbox.c
cyrus-imapd-2.3.9/imap/mailbox.c
--- cyrus-imapd-2.3.9.orig/imap/mailbox.c 2007-07-20
16:21:57.000000000 +0200
+++ cyrus-imapd-2.3.9/imap/mailbox.c 2007-08-31 17:14:47.000000000 +0200
@@ -2457,6 +2457,8 @@
*(fname->tail)++ = '/';
fname->len++;
for (msgno = 0; msgno < numdeleted; msgno++) {
+ syslog(LOG_DEBUG, "mailbox_expunge: removing mail %s:%d",
+ mailbox->name, deleted[msgno]);
mailbox_message_get_fname(mailbox, deleted[msgno],
fname->tail,
sizeof(fname->buf) - fname->len);
-- snip -----------------------------------------
--
Alain Spineux
aspineux gmail com
May the sources be with you
______________________________________________________________________
OpenPKG http://openpkg.org
User Communication List openpkg-users@openpkg.org
