diff options
author | Senthil Kumaran <senthil@uthcode.com> | 2011-10-19 17:06:59 (GMT) |
---|---|---|
committer | Senthil Kumaran <senthil@uthcode.com> | 2011-10-19 17:06:59 (GMT) |
commit | 294c231aa5a9b52fca429209c6091d44c8060008 (patch) | |
tree | adbcb97c81f12932f380783fc607fc4c8c010e18 /Lib | |
parent | 95483b6a19b2dbed2353e023f70e87a5010b9a39 (diff) | |
parent | 1ef0c0349e8fdb5415e21231cb42edbf232b742a (diff) | |
download | cpython-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-x | Lib/cgi.py | 2 | ||||
-rw-r--r-- | Lib/test/test_cgi.py | 4 |
2 files changed, 5 insertions, 1 deletions
@@ -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" |