#1664522: in urllib, don't read non-existing directories in ftp mode,

returning a 0-byte file -- raise an IOError instead.
Original patch from Phil Knirsch.

1 files changed, 13 insertions, 3 deletions

diff --git a/Lib/urllib.py b/Lib/urllib.py
index 2e720ac..6608abc 100644
--- a/Lib/urllib.py
+++ b/Lib/urllib.py
@@ -872,9 +872,19 @@ class ftpwrapper:
         if not conn:
             # Set transfer mode to ASCII!
             self.ftp.voidcmd('TYPE A')
-            # Try a directory listing
-            if file: cmd = 'LIST ' + file
-            else: cmd = 'LIST'
+            # Try a directory listing. Verify that directory exists.
+            if file:
+                pwd = self.ftp.pwd()
+                try:
+                    try:
+                        self.ftp.cwd(file)
+                    except ftplib.error_perm, reason:
+                        raise IOError, ('ftp error', reason), sys.exc_info()[2]
+                finally:
+                    self.ftp.cwd(pwd)
+                cmd = 'LIST ' + file
+            else:
+                cmd = 'LIST'
             conn = self.ftp.ntransfercmd(cmd)
         self.busy = 1
         # Pass back both a suitably decorated object and a retrieval length