From 94ad49fabc15c2eaafb5b590701ceb642d56bec0 Mon Sep 17 00:00:00 2001 From: Serhiy Storchaka Date: Fri, 8 Apr 2016 15:00:02 +0300 Subject: [PATCH] Issue #26709: Fixed Y2038 problem in loading binary PLists. --- Lib/plistlib.py | 2 +- Lib/test/test_plistlib.py | 9 +++++++++ Misc/NEWS | 2 ++ 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/Lib/plistlib.py b/Lib/plistlib.py index b9946fd313a..a39151fb0be 100644 --- a/Lib/plistlib.py +++ b/Lib/plistlib.py @@ -685,7 +685,7 @@ class _BinaryPlistParser: f = struct.unpack('>d', self._fp.read(8))[0] # timestamp 0 of binary plists corresponds to 1/1/2001 # (year of Mac OS X 10.0), instead of 1/1/1970. - return datetime.datetime.utcfromtimestamp(f + (31 * 365 + 8) * 86400) + return datetime.datetime(2001, 1, 1) + datetime.timedelta(seconds=f) elif tokenH == 0x40: # data s = self._get_size(tokenL) diff --git a/Lib/test/test_plistlib.py b/Lib/test/test_plistlib.py index fef9f399723..f0e9e5a9ea3 100644 --- a/Lib/test/test_plistlib.py +++ b/Lib/test/test_plistlib.py @@ -428,6 +428,15 @@ class TestPlistlib(unittest.TestCase): b'\x00\x00\x00\x00\x00\x00\x00\x13') self.assertEqual(plistlib.loads(data), {'a': 'b'}) + def test_large_timestamp(self): + # Issue #26709: 32-bit timestamp out of range + for ts in -2**31-1, 2**31: + with self.subTest(ts=ts): + d = (datetime.datetime.utcfromtimestamp(0) + + datetime.timedelta(seconds=ts)) + data = plistlib.dumps(d, fmt=plistlib.FMT_BINARY) + self.assertEqual(plistlib.loads(data), d) + class TestPlistlibDeprecated(unittest.TestCase): def test_io_deprecated(self): diff --git a/Misc/NEWS b/Misc/NEWS index 7027b30c717..bd482851801 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -99,6 +99,8 @@ Core and Builtins Library ------- +- Issue #26709: Fixed Y2038 problem in loading binary PLists. + - Issue #23735: Handle terminal resizing with Readline 6.3+ by installing our own SIGWINCH handler. Patch by Eric Price.