diff options
author | Georg Brandl <georg@python.org> | 2011-08-13 09:54:33 (GMT) |
---|---|---|
committer | Georg Brandl <georg@python.org> | 2011-08-13 09:54:33 (GMT) |
commit | d88e5af061c4c3206699fe7b86e736c42fdeba1c (patch) | |
tree | 8f1ac2f3c1df785a6a327f19433067d57f507277 | |
parent | b3f0ce4d1ea6a48774af22e20b8f748c19f87462 (diff) | |
parent | 3abb372c810c13af4207124c175cfd0a842b5f56 (diff) | |
download | cpython-d88e5af061c4c3206699fe7b86e736c42fdeba1c.zip cpython-d88e5af061c4c3206699fe7b86e736c42fdeba1c.tar.gz cpython-d88e5af061c4c3206699fe7b86e736c42fdeba1c.tar.bz2 |
Merge with cpython.
-rw-r--r-- | Lib/tarfile.py | 6 | ||||
-rw-r--r-- | Lib/test/test_tarfile.py | 8 | ||||
-rw-r--r-- | Misc/NEWS | 3 |
3 files changed, 15 insertions, 2 deletions
diff --git a/Lib/tarfile.py b/Lib/tarfile.py index 63c7554..3d52a81 100644 --- a/Lib/tarfile.py +++ b/Lib/tarfile.py @@ -1804,11 +1804,13 @@ class TarFile(object): fileobj = gzip.GzipFile(name, mode + "b", compresslevel, fileobj) t = cls.taropen(name, mode, fileobj, **kwargs) except IOError: - if not extfileobj: + if not extfileobj and fileobj is not None: fileobj.close() + if fileobj is None: + raise raise ReadError("not a gzip file") except: - if not extfileobj: + if not extfileobj and fileobj is not None: fileobj.close() raise t._extfileobj = extfileobj diff --git a/Lib/test/test_tarfile.py b/Lib/test/test_tarfile.py index 8036c5c..b45f168 100644 --- a/Lib/test/test_tarfile.py +++ b/Lib/test/test_tarfile.py @@ -1682,6 +1682,14 @@ class LinkEmulationTest(ReadTest): class GzipMiscReadTest(MiscReadTest): tarname = gzipname mode = "r:gz" + + def test_non_existent_targz_file(self): + # Test for issue11513: prevent non-existent gzipped tarfiles raising + # multiple exceptions. + with self.assertRaisesRegex(IOError, "xxx") as ex: + tarfile.open("xxx", self.mode) + self.assertEqual(ex.exception.errno, errno.ENOENT) + class GzipUstarReadTest(UstarReadTest): tarname = gzipname mode = "r:gz" @@ -44,6 +44,9 @@ Core and Builtins Library ------- +- Issue #11513: Fix exception handling ``tarfile.TarFile.gzopen()`` when + the file cannot be opened. + - Issue #12687: Fix a possible buffering bug when unpickling text mode (protocol 0, mostly) pickles. |