summaryrefslogtreecommitdiffstats
path: root/Lib
diff options
context:
space:
mode:
authorSenthil Kumaran <senthil@uthcode.com>2011-10-19 17:06:59 (GMT)
committerSenthil Kumaran <senthil@uthcode.com>2011-10-19 17:06:59 (GMT)
commit294c231aa5a9b52fca429209c6091d44c8060008 (patch)
treeadbcb97c81f12932f380783fc607fc4c8c010e18 /Lib
parent95483b6a19b2dbed2353e023f70e87a5010b9a39 (diff)
parent1ef0c0349e8fdb5415e21231cb42edbf232b742a (diff)
downloadcpython-294c231aa5a9b52fca429209c6091d44c8060008.zip
cpython-294c231aa5a9b52fca429209c6091d44c8060008.tar.gz
cpython-294c231aa5a9b52fca429209c6091d44c8060008.tar.bz2
default - Fix closes Issue12529 - cgi.parse_header failure on double quotes and
semicolons. Patch by Ben Darnell and Petri Lehtinen.
Diffstat (limited to 'Lib')
-rwxr-xr-xLib/cgi.py2
-rw-r--r--Lib/test/test_cgi.py4
2 files changed, 5 insertions, 1 deletions
diff --git a/Lib/cgi.py b/Lib/cgi.py
index 63771fc..b3e32f1 100755
--- a/Lib/cgi.py
+++ b/Lib/cgi.py
@@ -300,7 +300,7 @@ def _parseparam(s):
while s[:1] == ';':
s = s[1:]
end = s.find(';')
- while end > 0 and s.count('"', 0, end) % 2:
+ while end > 0 and (s.count('"', 0, end) - s.count('\\"', 0, end)) % 2:
end = s.find(';', end + 1)
if end < 0:
end = len(s)
diff --git a/Lib/test/test_cgi.py b/Lib/test/test_cgi.py
index dba7727..cb59008 100644
--- a/Lib/test/test_cgi.py
+++ b/Lib/test/test_cgi.py
@@ -342,6 +342,10 @@ this is the content of the fake file
self.assertEqual(
cgi.parse_header('attachment; filename="strange;name";size=123;'),
("attachment", {"filename": "strange;name", "size": "123"}))
+ self.assertEqual(
+ cgi.parse_header('form-data; name="files"; filename="fo\\"o;bar"'),
+ ("form-data", {"name": "files", "filename": 'fo"o;bar'}))
+
BOUNDARY = "---------------------------721837373350705526688164684"