changeset:   86780:493a99acaf00
parent:      86773:28c04e954bb6
parent:      86779:544b654d000c
user:        Benjamin Peterson <benjamin@python.org>
date:        Wed Oct 30 12:51:16 2013 -0400
files:       Lib/http/server.py Lib/test/test_httpservers.py Misc/NEWS
description:
merge 3.3 (#19435)


diff -r 28c04e954bb6 -r 493a99acaf00 Lib/http/server.py
--- a/Lib/http/server.py	Tue Oct 29 22:25:55 2013 -0400
+++ b/Lib/http/server.py	Wed Oct 30 12:51:16 2013 -0400
@@ -995,18 +995,17 @@
 
     def run_cgi(self):
         """Execute a CGI script."""
-        path = self.path
         dir, rest = self.cgi_info
 
-        i = path.find('/', len(dir) + 1)
+        i = rest.find('/')
         while i >= 0:
-            nextdir = path[:i]
-            nextrest = path[i+1:]
+            nextdir = rest[:i]
+            nextrest = rest[i+1:]
 
             scriptdir = self.translate_path(nextdir)
             if os.path.isdir(scriptdir):
                 dir, rest = nextdir, nextrest
-                i = path.find('/', len(dir) + 1)
+                i = rest.find('/')
             else:
                 break
 
diff -r 28c04e954bb6 -r 493a99acaf00 Lib/test/test_httpservers.py
--- a/Lib/test/test_httpservers.py	Tue Oct 29 22:25:55 2013 -0400
+++ b/Lib/test/test_httpservers.py	Wed Oct 30 12:51:16 2013 -0400
@@ -346,6 +346,7 @@
         self.parent_dir = tempfile.mkdtemp()
         self.cgi_dir = os.path.join(self.parent_dir, 'cgi-bin')
         os.mkdir(self.cgi_dir)
+        self.nocgi_path = None
         self.file1_path = None
         self.file2_path = None
 
@@ -366,6 +367,11 @@
             self.tearDown()
             self.skipTest("Python executable path is not encodable to utf-8")
 
+        self.nocgi_path = os.path.join(self.parent_dir, 'nocgi.py')
+        with open(self.nocgi_path, 'w') as fp:
+            fp.write(cgi_file1 % self.pythonexe)
+        os.chmod(self.nocgi_path, 0o777)
+
         self.file1_path = os.path.join(self.cgi_dir, 'file1.py')
         with open(self.file1_path, 'w', encoding='utf-8') as file1:
             file1.write(cgi_file1 % self.pythonexe)
@@ -383,6 +389,8 @@
             os.chdir(self.cwd)
             if self.pythonexe != sys.executable:
                 os.remove(self.pythonexe)
+            if self.nocgi_path:
+                os.remove(self.nocgi_path)
             if self.file1_path:
                 os.remove(self.file1_path)
             if self.file2_path:
@@ -439,6 +447,10 @@
         self.assertEqual((b'Hello World' + self.linesep, 'text/html', 200),
             (res.read(), res.getheader('Content-type'), res.status))
 
+    def test_issue19435(self):
+        res = self.request('///////////nocgi.py/../cgi-bin/nothere.sh')
+        self.assertEqual(res.status, 404)
+
     def test_post(self):
         params = urllib.parse.urlencode(
             {'spam' : 1, 'eggs' : 'python', 'bacon' : 123456})
diff -r 28c04e954bb6 -r 493a99acaf00 Misc/NEWS
--- a/Misc/NEWS	Tue Oct 29 22:25:55 2013 -0400
+++ b/Misc/NEWS	Wed Oct 30 12:51:16 2013 -0400
@@ -34,6 +34,8 @@
 - Issue #19424: Fix the warnings module to accept filename containing surrogate
   characters.
 
+- Issue #19435: Fix directory traversal attack on CGIHttpRequestHandler.
+
 - Issue #19227: Remove pthread_atfork() handler. The handler was added to
   solve #18747 but has caused issues.