diff --git a/Lib/tarfile.py b/Lib/tarfile.py index 125d95dd8a8..bc608e9d790 100644 --- a/Lib/tarfile.py +++ b/Lib/tarfile.py @@ -1857,7 +1857,7 @@ def gettarinfo(self, name=None, arcname=None, fileobj=None): tarinfo.mode = stmd tarinfo.uid = statres.st_uid tarinfo.gid = statres.st_gid - if stat.S_ISREG(stmd): + if type == REGTYPE: tarinfo.size = statres.st_size else: tarinfo.size = 0 diff --git a/Lib/test/test_tarfile.py b/Lib/test/test_tarfile.py index 1a556eb0f59..124f0e97f84 100644 --- a/Lib/test/test_tarfile.py +++ b/Lib/test/test_tarfile.py @@ -616,10 +616,14 @@ def test_link_size(self): if hasattr(os, "link"): link = os.path.join(TEMPDIR, "link") target = os.path.join(TEMPDIR, "link_target") - open(target, "wb").close() + fobj = open(target, "wb") + fobj.write(b"aaa") + fobj.close() os.link(target, link) try: tar = tarfile.open(tmpname, self.mode) + # Record the link target in the inodes list. + tar.gettarinfo(target) tarinfo = tar.gettarinfo(link) self.assertEqual(tarinfo.size, 0) finally: diff --git a/Misc/NEWS b/Misc/NEWS index ea7e5034a03..91d59b18280 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -54,6 +54,9 @@ C-API Library ------- +- Issue #8833: tarfile created hard link entries with a size field != 0 by + mistake. + - Charset.body_encode now correctly handles base64 encoding by encoding with the output_charset before calling base64mime.encode. Passes the tests from 2.x issue 1368247.