dovecot - Mar 2008 - antispam plugin / mailtrain backend: t_pop() leak & killed with signal 6

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hello,

antispam plugin version downloaded HEAD snapshot today.

Leaked a t_pop() call in I/O handler 0x805b350

Raw backtrace: imap [0x80ad741] -> imap [0x80ad65c] -> 
imap(io_loop_handler _run+0x1ce) [0x80b35ce] -> imap(io_loop_run+0x28) 
[0x80b29a8] -> imap(main+0x5a0) [0x8063590] -> 
/lib/tls/i686/cmov/libc.so.6(__lib c_start_main+0xc8) [0xb7e37ea8] -> imap 
[0x8055e31]

child 18478 (imap) killed with signal 6

Cause is, if sendmail fails with an exit code != 0, the function returns 
with -1 immediately skipping the t_pop() before the end of the function.

See attached diff.

Bye,

- --
Steffen Kaiser
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFHzpkeVJMDrex4hCIRAhhzAJ9uwU9pYEOZoSZiYJiERcKJJOzI/ACfXG10
BmSVMKMh2FUFve9Qu8atY9k=wE4j
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mailtrain.diff.gz
Type: application/octet-stream
Size: 401 bytes
Desc: 
URL:
<http://dovecot.org/pipermail/dovecot/attachments/20080305/2ade2b19/attachment-0002.obj>

> Leaked a t_pop() call in I/O handler 0x805b350
> 
> Raw backtrace: imap [0x80ad741] -> imap [0x80ad65c] -> 
> imap(io_loop_handler _run+0x1ce) [0x80b35ce] -> imap(io_loop_run+0x28) 
> [0x80b29a8] -> imap(main+0x5a0) [0x8063590] -> 
> /lib/tls/i686/cmov/libc.so.6(__lib c_start_main+0xc8) [0xb7e37ea8] ->
imap
> [0x8055e31]
> 
> child 18478 (imap) killed with signal 6
> 
> Cause is, if sendmail fails with an exit code != 0, the function returns 
> with -1 immediately skipping the t_pop() before the end of the function.
> 
> See attached diff.
Merged, thanks.

johannes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 828 bytes
Desc: This is a digitally signed message part
URL:
<http://dovecot.org/pipermail/dovecot/attachments/20080305/0970db2d/attachment-0002.bin>