diff --git a/Lib/tarfile.py b/Lib/tarfile.py index a87fe225d17..29e12f02167 100644 --- a/Lib/tarfile.py +++ b/Lib/tarfile.py @@ -1557,27 +1557,33 @@ class TarFile(object): self.inodes = {} # dictionary caching the inodes of # archive members already added - if self.mode == "r": - self.firstmember = None - self.firstmember = self.next() + try: + if self.mode == "r": + self.firstmember = None + self.firstmember = self.next() - if self.mode == "a": - # Move to the end of the archive, - # before the first empty block. - self.firstmember = None - while True: - if self.next() is None: - if self.offset > 0: - self.fileobj.seek(- BLOCKSIZE, 1) - break + if self.mode == "a": + # Move to the end of the archive, + # before the first empty block. + self.firstmember = None + while True: + if self.next() is None: + if self.offset > 0: + self.fileobj.seek(- BLOCKSIZE, 1) + break - if self.mode in "aw": - self._loaded = True + if self.mode in "aw": + self._loaded = True - if self.pax_headers: - buf = self.tarinfo.create_pax_global_header(self.pax_headers.copy()) - self.fileobj.write(buf) - self.offset += len(buf) + if self.pax_headers: + buf = self.tarinfo.create_pax_global_header(self.pax_headers.copy()) + self.fileobj.write(buf) + self.offset += len(buf) + except: + if not self._extfileobj: + self.fileobj.close() + self.closed = True + raise def _getposix(self): return self.format == USTAR_FORMAT diff --git a/Misc/NEWS b/Misc/NEWS index a52e28aad1a..d0ed47ce4fb 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -440,6 +440,9 @@ Core and Builtins Library ------- +- Issue #7341: Close the internal file object in the TarFile constructor in + case of an error. + - Issue #7293: distutils.test_msvc9compiler is fixed to work on any fresh Windows box. Help provided by David Bolen.