From: Senthil Kumaran Date: Wed, 19 Oct 2011 16:52:24 +0000 (+0800) Subject: Fix closes Issue12529 - cgi.parse_header failure on double quotes and X-Git-Tag: v2.7.3rc1~395 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=59a06d412d3a130c770faa1fc8d8fe13b0a67d3c;p=thirdparty%2FPython%2Fcpython.git Fix closes Issue12529 - cgi.parse_header failure on double quotes and semicolons. Patch by Ben Darnell and Petri Lehtinen. --- diff --git a/Lib/cgi.py b/Lib/cgi.py index b4c620a9e405..e7cd7783164a 100755 --- a/Lib/cgi.py +++ b/Lib/cgi.py @@ -293,7 +293,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 e282a51503a4..63547b24c204 100644 --- a/Lib/test/test_cgi.py +++ b/Lib/test/test_cgi.py @@ -377,6 +377,9 @@ 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'})) def test_main():