From: Georg Brandl <georg@python.org>
Date: Thu, 16 Nov 2006 17:08:45 +0000 (+0000)
Subject: Bug #1588217: don't parse "= " as a soft line break in binascii's
X-Git-Tag: v2.6a1~2428
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=25aabf4cbb7ff8d1a38f8ef4602a409c00474a30;p=thirdparty%2FPython%2Fcpython.git

Bug #1588217: don't parse "= " as a soft line break in binascii's
a2b_qp() function, instead leave it in the string as quopri.decode()
does.
---

diff --git a/Lib/test/test_binascii.py b/Lib/test/test_binascii.py
index 8df450456ee3..8272ad91aa81 100755
--- a/Lib/test/test_binascii.py
+++ b/Lib/test/test_binascii.py
@@ -134,7 +134,7 @@ class BinASCIITest(unittest.TestCase):
             pass
         else:
             self.fail("binascii.a2b_qp(**{1:1}) didn't raise TypeError")
-        self.assertEqual(binascii.a2b_qp("= "), "")
+        self.assertEqual(binascii.a2b_qp("= "), "= ")
         self.assertEqual(binascii.a2b_qp("=="), "=")
         self.assertEqual(binascii.a2b_qp("=AX"), "=AX")
         self.assertRaises(TypeError, binascii.b2a_qp, foo="bar")
diff --git a/Misc/NEWS b/Misc/NEWS
index 32cdb1f688d6..8c9351992d95 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -219,6 +219,10 @@ Library
 Extension Modules
 -----------------
 
+- Bug #1588217: don't parse "= " as a soft line break in binascii's
+  a2b_qp() function, instead leave it in the string as quopri.decode()
+  does.
+
 - Bug #1567666: Emulate GetFileAttributesExA for Win95.
 
 - Patch #1576166: Support os.utime for directories on Windows NT+.
diff --git a/Modules/binascii.c b/Modules/binascii.c
index 3b2c8b254cd3..4dee45198ef1 100644
--- a/Modules/binascii.c
+++ b/Modules/binascii.c
@@ -1057,8 +1057,7 @@ binascii_a2b_qp(PyObject *self, PyObject *args, PyObject *kwargs)
 			in++;
 			if (in >= datalen) break;
 			/* Soft line breaks */
-			if ((data[in] == '\n') || (data[in] == '\r') ||
-			    (data[in] == ' ') || (data[in] == '\t')) {
+			if ((data[in] == '\n') || (data[in] == '\r')) {
 				if (data[in] != '\n') {
 					while (in < datalen && data[in] != '\n') in++;
 				}