Re: [Dovecot] over quota + antispam crash

2009-02-20 Thread Johannes Berg
Thanks for the patch, can you check whether it works with just the if
(dest_mail)?

johannes


signature.asc
Description: This is a digitally signed message part


Re: [Dovecot] over quota + antispam crash

2009-02-20 Thread Stéphane Cottin
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Johannes Berg a écrit :
 Thanks for the patch, can you check whether it works with just the if
 (dest_mail)?
 

It doesn't work

the return -1 is inspired from the end of quota_save_init function (in
src/plugins/quota/quota-storage.c)

the if(dest_mail) is not in quota_save_init.
it may be unnecessary.

Stéphane

 johannes

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkme5AcACgkQNgQUYqHIqqs7vACgjNp27Pf+Hc+9v2GK0NKRXyKC
BjUAn0MApH6xBjw+FHCOCT693rhGYduT
=kAgf
-END PGP SIGNATURE-
begin:vcard
fn;quoted-printable:St=C3=A9phane Cottin
n;quoted-printable:Cottin;St=C3=A9phane
org:VIXNS
adr:;;172 avenue de toulon;Marseille;;13010;France
email;internet:stephane.cot...@vixns.com
x-mozilla-html:FALSE
url:http://www.vixns.com
version:2.1
end:vcard



Re: [Dovecot] over quota + antispam crash

2009-02-20 Thread Allan Cassaro
On Thu, Feb 19, 2009 at 8:57 AM, Stéphane Cottin
stephane.cot...@vixns.com wrote:
 -BEGIN PGP SIGNED MESSAGE-
 Hash: SHA1

 Stéphane Cottin a écrit :
 Hi,

 I am using dovecot 1.1.11 , quota, imap_quota  antispam plugins.

 Dovecot crashes when saving a message to the Sent maildir and the
 account is over quota.

 gdb trace attached.


 Stéphane Cottin

All day long without segfaults!!! :D
Many thanks!!


Re: [Dovecot] over quota + antispam crash

2009-02-19 Thread Stéphane Cottin
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Stéphane Cottin a écrit :
 Hi,

 I am using dovecot 1.1.11 , quota, imap_quota  antispam plugins.

 Dovecot crashes when saving a message to the Sent maildir and the
 account is over quota.

 gdb trace attached.


 Stéphane Cottin


Hi,

The attached patch seems to fix my bug.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkmdSSUACgkQNgQUYqHIqqt+WgCeOfxt3CiMVs4cQLYVHFl932pM
H1gAni6WIB3V+qUr/hwScxMSWVjAvq1s
=2zbd
-END PGP SIGNATURE-
diff --git a/antispam-storage-1.1.c b/antispam-storage-1.1.c
index 348ae9d..bc370ad 100644
--- a/antispam-storage-1.1.c
+++ b/antispam-storage-1.1.c
@@ -167,7 +167,11 @@ static int antispam_save_init(struct mailbox_transaction_context *t,
 		timezone_offset, from_envelope,
 		input, dest_mail, ctx_r);
 
-	(*ctx_r)-dest_mail = dest_mail;
+	if(ret  0)
+		return -1;
+
+	if (dest_mail != NULL)
+		(*ctx_r)-dest_mail = dest_mail;
 
 	return ret;
 }

begin:vcard
fn;quoted-printable:St=C3=A9phane Cottin
n;quoted-printable:Cottin;St=C3=A9phane
org:VIXNS
adr:;;172 avenue de toulon;Marseille;;13010;France
email;internet:stephane.cot...@vixns.com
x-mozilla-html:FALSE
url:http://www.vixns.com
version:2.1
end:vcard