[Python-checkins] r80063 - in python/branches/release26-maint: Lib/email/message.py Lib/email/test/test_email.py Misc/ACKS Misc/NEWS

[r.david.murray]

Author: r.david.murray
Date: Tue Apr 13 23:05:24 2010
New Revision: 80063

Log:
Merged revisions 80062 via svnmerge from 
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r80062 | r.david.murray | 2010-04-13 16:57:40 -0400 (Tue, 13 Apr 2010) | 2 lines
  
  Issue #5277: Fix quote counting when parsing RFC 2231 encoded parameters.
........


Modified:
   python/branches/release26-maint/   (props changed)
   python/branches/release26-maint/Lib/email/message.py
   python/branches/release26-maint/Lib/email/test/test_email.py
   python/branches/release26-maint/Misc/ACKS
   python/branches/release26-maint/Misc/NEWS

Modified: python/branches/release26-maint/Lib/email/message.py
==============================================================================
--- python/branches/release26-maint/Lib/email/message.py	(original)
+++ python/branches/release26-maint/Lib/email/message.py	Tue Apr 13 23:05:24 2010
@@ -62,7 +62,7 @@
     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)

Modified: python/branches/release26-maint/Lib/email/test/test_email.py
==============================================================================
--- python/branches/release26-maint/Lib/email/test/test_email.py	(original)
+++ python/branches/release26-maint/Lib/email/test/test_email.py	Tue Apr 13 23:05:24 2010
@@ -338,6 +338,14 @@
         self.assertEqual(msg.get_param('name', unquote=False),
                          '"Jim&amp;&amp;Jill"')
 
+    def test_get_param_with_quotes(self):
+        msg = email.message_from_string(
+            'Content-Type: foo; bar*0="baz\\"foobar"; bar*1="\\"baz"')
+        self.assertEqual(msg.get_param('bar'), 'baz"foobar"baz')
+        msg = email.message_from_string(
+            "Content-Type: foo; bar*0=\"baz\\\"foobar\"; bar*1=\"\\\"baz\"")
+        self.assertEqual(msg.get_param('bar'), 'baz"foobar"baz')
+
     def test_has_key(self):
         msg = email.message_from_string('Header: exists')
         self.failUnless(msg.has_key('header'))

Modified: python/branches/release26-maint/Misc/ACKS
==============================================================================
--- python/branches/release26-maint/Misc/ACKS	(original)
+++ python/branches/release26-maint/Misc/ACKS	Tue Apr 13 23:05:24 2010
@@ -149,6 +149,7 @@
 Steve Cousins
 Alex Coventry
 Matthew Dixon Cowles
+Ryan Coyner
 Christopher A. Craig
 Laura Creighton
 Simon Cross
@@ -627,6 +628,7 @@
 Rich Salz
 Kevin Samborn
 Ilya Sandler
+Mark Sapiro
 Ty Sarna
 Ben Sayer
 Michael Scharf

Modified: python/branches/release26-maint/Misc/NEWS
==============================================================================
--- python/branches/release26-maint/Misc/NEWS	(original)
+++ python/branches/release26-maint/Misc/NEWS	Tue Apr 13 23:05:24 2010
@@ -28,6 +28,8 @@
 Library
 -------
 
+- Issue #5277: Fix quote counting when parsing RFC 2231 encoded parameters.
+
 - Issue #8179: Fix macpath.realpath() on a non-existing path.
 
 - Issue #8310: Allow dis to examine new style classes.