diff options
author | Martin v. Löwis <martin@v.loewis.de> | 2001-10-07 08:53:32 (GMT) |
---|---|---|
committer | Martin v. Löwis <martin@v.loewis.de> | 2001-10-07 08:53:32 (GMT) |
commit | 322c0d187d834013d70ffdb0f7f1b6f72eef6443 (patch) | |
tree | ca8bbe6526e9f3c88b6311691e7936bfdc0480b0 /Lib/ftplib.py | |
parent | fb163784ab8ec237fdc3bffd018efe653ae07a1d (diff) | |
download | cpython-322c0d187d834013d70ffdb0f7f1b6f72eef6443.zip cpython-322c0d187d834013d70ffdb0f7f1b6f72eef6443.tar.gz cpython-322c0d187d834013d70ffdb0f7f1b6f72eef6443.tar.bz2 |
Only close sockets if they have been created. Reported by Blake Winton.
Diffstat (limited to 'Lib/ftplib.py')
-rw-r--r-- | Lib/ftplib.py | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/Lib/ftplib.py b/Lib/ftplib.py index 4aa6244..4d5ac1a 100644 --- a/Lib/ftplib.py +++ b/Lib/ftplib.py @@ -122,7 +122,8 @@ class FTP: self.sock = socket.socket(af, socktype, proto) self.sock.connect(sa) except socket.error, msg: - self.sock.close() + if self.sock: + self.sock.close() self.sock = None continue break @@ -272,13 +273,15 @@ class FTP: def makeport(self): '''Create a new socket and send a PORT command for it.''' msg = "getaddrinfo returns an empty list" + 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, msg: - sock.close() + if sock: + sock.close() sock = None continue break |