From: Giampaolo Rodolà <g.rodola@gmail.com>
Date: Wed, 30 Jun 2010 17:47:39 +0000 (+0000)
Subject: Merged revisions 82404 via svnmerge from
X-Git-Tag: v3.2a1~372
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=610aa4f07f8f6d13cd5510b007a5edf574c0f7da;p=python

Merged revisions 82404 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r82404 | giampaolo.rodola | 2010-06-30 19:38:28 +0200 (mer, 30 giu 2010) | 1 line

  fix issue #6589: cleanup asyncore.socket_map if smtpd.SMTPServer constructor raises an exception
........
---

diff --git a/Lib/smtpd.py b/Lib/smtpd.py
index 17b74270b2..d7c5c93814 100755
--- a/Lib/smtpd.py
+++ b/Lib/smtpd.py
@@ -274,14 +274,19 @@ class SMTPServer(asyncore.dispatcher):
         self._localaddr = localaddr
         self._remoteaddr = remoteaddr
         asyncore.dispatcher.__init__(self)
-        self.create_socket(socket.AF_INET, socket.SOCK_STREAM)
-        # try to re-use a server port if possible
-        self.set_reuse_addr()
-        self.bind(localaddr)
-        self.listen(5)
-        print('%s started at %s\n\tLocal addr: %s\n\tRemote addr:%s' % (
-            self.__class__.__name__, time.ctime(time.time()),
-            localaddr, remoteaddr), file=DEBUGSTREAM)
+        try:
+            self.create_socket(socket.AF_INET, socket.SOCK_STREAM)
+            # try to re-use a server port if possible
+            self.set_reuse_addr()
+            self.bind(localaddr)
+            self.listen(5)
+        except:
+            self.close()
+            raise
+        else:
+            print('%s started at %s\n\tLocal addr: %s\n\tRemote addr:%s' % (
+                self.__class__.__name__, time.ctime(time.time()),
+                localaddr, remoteaddr), file=DEBUGSTREAM)
 
     def handle_accept(self):
         conn, addr = self.accept()
diff --git a/Misc/NEWS b/Misc/NEWS
index 5d3c14e1e5..78fe3fc1a3 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -460,6 +460,9 @@ C-API
 Library
 -------
 
+- Issue #6589: cleanup asyncore.socket_map in case smtpd.SMTPServer constructor 
+  raises an exception.
+
 - Issue #9110: Addition of ContextDecorator to contextlib, for creating APIs
   that act as both context managers and decorators. contextmanager changes
   to use ContextDecorator.