From ecf1ca8748b3fd62ee5cc085aa84acfd21d51c50 Mon Sep 17 00:00:00 2001 From: Matthias Andree Date: Wed, 20 Jul 2005 11:10:19 +0000 Subject: Fix Debian bug #301964, socket leak. svn path=/trunk/; revision=4127 --- NEWS | 4 ++-- driver.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/NEWS b/NEWS index 2982c161..0008b7a7 100644 --- a/NEWS +++ b/NEWS @@ -109,8 +109,6 @@ fetchmail 6.3.0 (not yet released officially): * Run fetchmail.man through automatic spell checker. Matthias Andree. * Major fetchmail(1) manual page overhaul by R. Hannes Beinert, to clarify singledrop vs. multidrop operation. (MA) - -since -pre4: * Make tracepolls a server option, as documented. Fixes Debian bug #156094. Matthias Andree. * Updated some translations. (MA) @@ -119,6 +117,8 @@ since -pre4: * Rename ESR's design notes to esrs-design-notes.html and add a new design-notes.html document. The NOTES file will contain both of them. Matthias Andree. +* Fix Debian bug #301964, fetchmail leaks sockets when SSL negotiation + fails. Fix suggested Goswin Brederlow. (MA) fetchmail-6.2.5 (Wed Oct 15 18:39:22 EDT 2003), 23079 lines: diff --git a/driver.c b/driver.c index 7cfb2cbf..905890d8 100644 --- a/driver.c +++ b/driver.c @@ -1161,7 +1161,7 @@ static int do_session( { report(stderr, GT_("SSL connection failed.\n")); err = PS_AUTHFAIL; - goto closeUp; + goto cleanUp; } /* Fetchmail didn't hang on SSLOpen, -- cgit v1.2.3