Commit 9fbec7ad authored by Serhiy Storchaka's avatar Serhiy Storchaka

Issue #20238: TarFile opened with external fileobj and "w:gz" mode didn't

write complete output on close.
parent 53ad0cd2
...@@ -1642,7 +1642,7 @@ class TarFile(object): ...@@ -1642,7 +1642,7 @@ class TarFile(object):
if not extfileobj and fileobj is not None: if not extfileobj and fileobj is not None:
fileobj.close() fileobj.close()
raise raise
t._extfileobj = extfileobj t._extfileobj = False
return t return t
@classmethod @classmethod
......
...@@ -855,6 +855,12 @@ class WriteTestBase(TarTest): ...@@ -855,6 +855,12 @@ class WriteTestBase(TarTest):
tar.addfile(tarfile.TarInfo("foo")) tar.addfile(tarfile.TarInfo("foo"))
tar.close() tar.close()
self.assertFalse(fobj.closed, "external fileobjs must never closed") self.assertFalse(fobj.closed, "external fileobjs must never closed")
# Issue #20238: Incomplete gzip output with mode="w:gz"
data = fobj.getvalue()
del tar
support.gc_collect()
self.assertFalse(fobj.closed)
self.assertEqual(data, fobj.getvalue())
class WriteTest(WriteTestBase, unittest.TestCase): class WriteTest(WriteTestBase, unittest.TestCase):
......
...@@ -43,6 +43,9 @@ Core and Builtins ...@@ -43,6 +43,9 @@ Core and Builtins
Library Library
------- -------
- Issue #20238: TarFile opened with external fileobj and "w:gz" mode didn't
write complete output on close.
- Issue #20245: The open functions in the tarfile module now correctly handle - Issue #20245: The open functions in the tarfile module now correctly handle
empty mode. empty mode.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment