Your message dated Mon, 17 Dec 2012 12:17:36 +0000
with message-id <[email protected]>
and subject line Bug#671294: fixed in fetchmail 6.3.22-2
has caused the Debian Bug report #671294,
regarding combination of --plugin and -f - fails
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)
--
671294: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=671294
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: fetchmail
Version: 6.3.18-2
Severity: normal
Tags: patch
this problem applies to both the version in squeeze and sid's 6.3.21-3.
scenario: you want to remote-control fetchmail, but you don't want to write
passwords into files,
so you feed fetchmail a minimal rcfile via stdin with -f -. this by itself
works fine. if you also
want or need to use a --plugin (eg. socat for socks), then things fail badly:
the plugin is run without
a stdin fd, hence can't take input from fetchmail, lots of fun ensues.
plugins without -f - work fine, it's just the combination that fails.
explanation: the root cause is rcfile_y.y, line 493, which closes whatever fd
carried the rcfile. with -f -
this closes fetchmail's stdin - and so far that's unproblematic.
however, in socket.c lines 166ff things go wrong: fetchmail sets up the plugin
with a socketpair, which
will likely include the first unused fd - and fd zero is now indeed unused.
in line 180ff a dup2 replumbing from "that fd" (=zero) to zero is performed -
and then "that fd" is closed.
and hey presto, we've got no fd zero = stdin for the plugin.
solution: the simplest solution (patch attached) is to make the fclose of the
rcfile conditional,
ie. don't close if it's stdin. in the long run the dup2+close code might be
made more robust by
not doing a dup2+close if fd[0] is already 0 or 1.
regards
az
--- rcfile_y.y.orig 2012-05-03 14:12:01.000000000 +1000
+++ rcfile_y.y 2012-05-03 14:12:15.000000000 +1000
@@ -490,7 +490,8 @@
yyparse(); /* parse entire file */
- fclose(yyin); /* not checking this should be safe, file mode was r */
+ if (yyin != stdin)
+ fclose(yyin); /* not checking this should be safe, file mode was r */
if (prc_errflag)
return(PS_SYNTAX);
--- End Message ---
--- Begin Message ---
Source: fetchmail
Source-Version: 6.3.22-2
We believe that the bug you reported is fixed in the latest version of
fetchmail, which is due to be installed in the Debian FTP archive.
A summary of the changes between this version and the previous one is
attached.
Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Nico Golde <[email protected]> (supplier of updated fetchmail package)
(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Format: 1.8
Date: Mon, 17 Dec 2012 13:04:01 +0100
Source: fetchmail
Binary: fetchmail fetchmailconf
Architecture: source all amd64
Version: 6.3.22-2
Distribution: unstable
Urgency: low
Maintainer: Fetchmail Maintainers <[email protected]>
Changed-By: Nico Golde <[email protected]>
Description:
fetchmail - SSL enabled POP3, APOP, IMAP mail gatherer/forwarder
fetchmailconf - fetchmail configurator
Closes: 671294 688015
Changes:
fetchmail (6.3.22-2) unstable; urgency=low
.
* Fix memory leak in OpenSSL's certificate varification callback.
Thanks Erik Thiele and Dominik! (Closes: #688015).
* Fix combination of --plugin and -f - (Closes: #671294).
Checksums-Sha1:
e22677c900231681b5ad7fbfcde65f5a580609c7 1411 fetchmail_6.3.22-2.dsc
8ddd4a9f68fd78461054579c3c14b78f2209f16b 51541 fetchmail_6.3.22-2.debian.tar.gz
dcae4d7284d01472236c3c6dc3ccf28f1c78649c 66738 fetchmailconf_6.3.22-2_all.deb
6f241c668ef954392fc542833cd92522892a1ca1 967798 fetchmail_6.3.22-2_amd64.deb
Checksums-Sha256:
fc6450ff729a5e5e6e335b2fde3cc6b1eeb4a6887f1dd9a44e3ba8df284b1455 1411
fetchmail_6.3.22-2.dsc
a28ef6a4236cc7fcdef4dc8d4cc7b5f8cc27a3621bbc35a5ebae5bd4d4a3e3cf 51541
fetchmail_6.3.22-2.debian.tar.gz
c2b1074e654e3745ef19f2a4dab2b316624c0c9eefe1e1f8c280e48f3aab66f1 66738
fetchmailconf_6.3.22-2_all.deb
dce33e8ad8c6e4a1324675989f6edf4ca168de44e302c6e0b744b9c3045280b0 967798
fetchmail_6.3.22-2_amd64.deb
Files:
fcea72fdc31efc229736ee5267304eab 1411 mail optional fetchmail_6.3.22-2.dsc
4b699a4c8070e6d7de2380a8aa82d1fb 51541 mail optional
fetchmail_6.3.22-2.debian.tar.gz
d86ffd5261f9b79ecc3daa0f331dd3f0 66738 mail optional
fetchmailconf_6.3.22-2_all.deb
39454911241e482793ade564972ce91b 967798 mail optional
fetchmail_6.3.22-2_amd64.deb
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)
iEYEARECAAYFAlDPDIoACgkQHYflSXNkfP/E2gCfZAeDPlouQVgo3ymf87QSoQS0
tjoAn3ehqNid/sOmmxj95sCRmPjJdSVe
=O+rN
-----END PGP SIGNATURE-----
--- End Message ---
