From fc3278802c51bc8d56bc8a26cc9f95e4f6cfb5bd Mon Sep 17 00:00:00 2001 From: Giampaolo Rodola' Date: Mon, 17 Dec 2012 14:20:27 +0100 Subject: Fix issue #16646: ftplib.FTP.makeport() might lose socket error details. (patch by Serhiy Storchaka) --- Lib/ftplib.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'Lib/ftplib.py') diff --git a/Lib/ftplib.py b/Lib/ftplib.py index 8e53023270..94237caf21 100644 --- a/Lib/ftplib.py +++ b/Lib/ftplib.py @@ -284,20 +284,24 @@ class FTP: def makeport(self): '''Create a new socket and send a PORT command for it.''' - msg = "getaddrinfo returns an empty list" + err = None sock = None for res in socket.getaddrinfo(None, 0, self.af, socket.SOCK_STREAM, 0, socket.AI_PASSIVE): af, socktype, proto, canonname, sa = res try: sock = socket.socket(af, socktype, proto) sock.bind(sa) - except socket.error as msg: + except socket.error as err: if sock: sock.close() sock = None continue break - if not sock: + if sock is None: + if err is not None: + raise err + else: + raise socket.error("getaddrinfo returns an empty list") raise socket.error(msg) sock.listen(1) port = sock.getsockname()[1] # Get proper port -- cgit v1.2.1