[Python-checkins] cpython (3.5): Issue #28321: Fixed writing non-BMP characters with binary format in plistlib.
serhiy.storchaka
python-checkins at python.org
Tue Oct 4 13:09:37 EDT 2016
https://hg.python.org/cpython/rev/381ef0f08f89
changeset: 104284:381ef0f08f89
branch: 3.5
parent: 104279:b674fd340a23
user: Serhiy Storchaka <storchaka at gmail.com>
date: Tue Oct 04 20:04:30 2016 +0300
summary:
Issue #28321: Fixed writing non-BMP characters with binary format in plistlib.
files:
Lib/plistlib.py | 2 +-
Lib/test/test_plistlib.py | 7 +++++++
Misc/NEWS | 2 ++
3 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/Lib/plistlib.py b/Lib/plistlib.py
--- a/Lib/plistlib.py
+++ b/Lib/plistlib.py
@@ -918,7 +918,7 @@
self._write_size(0x50, len(value))
except UnicodeEncodeError:
t = value.encode('utf-16be')
- self._write_size(0x60, len(value))
+ self._write_size(0x60, len(t) // 2)
self._fp.write(t)
diff --git a/Lib/test/test_plistlib.py b/Lib/test/test_plistlib.py
--- a/Lib/test/test_plistlib.py
+++ b/Lib/test/test_plistlib.py
@@ -361,6 +361,13 @@
plistlib.dumps,
testString)
+ def test_non_bmp_characters(self):
+ pl = {'python': '\U0001f40d'}
+ for fmt in ALL_FORMATS:
+ with self.subTest(fmt=fmt):
+ data = plistlib.dumps(pl, fmt=fmt)
+ self.assertEqual(plistlib.loads(data), pl)
+
def test_nondictroot(self):
for fmt in ALL_FORMATS:
with self.subTest(fmt=fmt):
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -89,6 +89,8 @@
Library
-------
+- Issue #28321: Fixed writing non-BMP characters with binary format in plistlib.
+
- Issue #28322: Fixed possible crashes when unpickle itertools objects from
incorrect pickle data. Based on patch by John Leitch.
--
Repository URL: https://hg.python.org/cpython
More information about the Python-checkins
mailing list