diff options
| -rw-r--r-- | Doc/library/zipfile.rst | 6 | ||||
| -rw-r--r-- | Lib/test/test_zipfile.py | 10 | ||||
| -rw-r--r-- | Lib/zipfile.py | 4 | ||||
| -rw-r--r-- | Misc/NEWS | 3 |
4 files changed, 22 insertions, 1 deletions
diff --git a/Doc/library/zipfile.rst b/Doc/library/zipfile.rst index 30a0361..14e40c8 100644 --- a/Doc/library/zipfile.rst +++ b/Doc/library/zipfile.rst @@ -388,7 +388,7 @@ Instances have the following attributes: +-------+--------------------------+ | Index | Value | +=======+==========================+ - | ``0`` | Year | + | ``0`` | Year (>= 1980) | +-------+--------------------------+ | ``1`` | Month (one-based) | +-------+--------------------------+ @@ -401,6 +401,10 @@ Instances have the following attributes: | ``5`` | Seconds (zero-based) | +-------+--------------------------+ + .. note:: + + The ZIP file format does not support timestamps before 1980. + .. attribute:: ZipInfo.compress_type diff --git a/Lib/test/test_zipfile.py b/Lib/test/test_zipfile.py index 6cba26c..7ebb663 100644 --- a/Lib/test/test_zipfile.py +++ b/Lib/test/test_zipfile.py @@ -477,6 +477,12 @@ class TestsWithSourceFile(unittest.TestCase): except zipfile.BadZipfile: self.assertTrue(zipfp2.fp is None, 'zipfp is not closed') + def test_add_file_before_1980(self): + # Set atime and mtime to 1970-01-01 + os.utime(TESTFN, (0, 0)) + with zipfile.ZipFile(TESTFN2, "w") as zipfp: + self.assertRaises(ValueError, zipfp.write, TESTFN) + def tearDown(self): unlink(TESTFN) unlink(TESTFN2) @@ -990,6 +996,10 @@ class OtherTests(unittest.TestCase): pass self.assertRaises(zipfile.BadZipfile, zipfile.ZipFile, TESTFN, 'r') + def test_create_zipinfo_before_1980(self): + self.assertRaises(ValueError, + zipfile.ZipInfo, 'seventies', (1979, 1, 1, 0, 0, 0)) + def tearDown(self): unlink(TESTFN) unlink(TESTFN2) diff --git a/Lib/zipfile.py b/Lib/zipfile.py index f876f42..d9181f2 100644 --- a/Lib/zipfile.py +++ b/Lib/zipfile.py @@ -290,6 +290,10 @@ class ZipInfo (object): self.filename = filename # Normalized file name self.date_time = date_time # year, month, day, hour, min, sec + + if date_time[0] < 1980: + raise ValueError('ZIP does not support timestamps before 1980') + # Standard values: self.compress_type = ZIP_STORED # Type of compression for the file self.comment = "" # Comment for each file @@ -63,6 +63,9 @@ Core and Builtins Library ------- +- Issue #6090: zipfile raises a ValueError when a document with a timestamp + earlier than 1980 is provided. Patch contributed by Petri Lehtinen. + - Issue #13194: zlib.compressobj().copy() and zlib.decompressobj().copy() are now available on Windows. |