Issue #7341: Close the internal file object in the TarFile
constructor in case of an error.
diff --git a/Lib/tarfile.py b/Lib/tarfile.py
index a87fe22..29e12f0 100644
--- a/Lib/tarfile.py
+++ b/Lib/tarfile.py
@@ -1557,27 +1557,33 @@
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