diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2016-10-30 18:52:29 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2016-10-30 18:52:29 (GMT) |
commit | a89d22aff3fb1e67f8f08eac22b08a58dfa048a8 (patch) | |
tree | 2324a9b41897d39e8b10e28f39a60ee5186c27fe /Lib/test/test_tarfile.py | |
parent | d1af5effc214f474bcb1df62eb2089c48f657ee5 (diff) | |
download | cpython-a89d22aff3fb1e67f8f08eac22b08a58dfa048a8.zip cpython-a89d22aff3fb1e67f8f08eac22b08a58dfa048a8.tar.gz cpython-a89d22aff3fb1e67f8f08eac22b08a58dfa048a8.tar.bz2 |
Issue #28449: tarfile.open() with mode "r" or "r:" now tries to open a tar
file with compression before trying to open it without compression. Otherwise
it had 50% chance failed with ignore_zeros=True.
Diffstat (limited to 'Lib/test/test_tarfile.py')
-rw-r--r-- | Lib/test/test_tarfile.py | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/Lib/test/test_tarfile.py b/Lib/test/test_tarfile.py index abfb34d..1efb841 100644 --- a/Lib/test/test_tarfile.py +++ b/Lib/test/test_tarfile.py @@ -3,6 +3,7 @@ import os import io from hashlib import md5 from contextlib import contextmanager +from random import Random import unittest import unittest.mock @@ -349,12 +350,17 @@ class CommonReadTest(ReadTest): def test_ignore_zeros(self): # Test TarFile's ignore_zeros option. + # generate 512 pseudorandom bytes + data = Random(0).getrandbits(512*8).to_bytes(512, 'big') for char in (b'\0', b'a'): # Test if EOFHeaderError ('\0') and InvalidHeaderError ('a') # are ignored correctly. with self.open(tmpname, "w") as fobj: fobj.write(char * 1024) - fobj.write(tarfile.TarInfo("foo").tobuf()) + tarinfo = tarfile.TarInfo("foo") + tarinfo.size = len(data) + fobj.write(tarinfo.tobuf()) + fobj.write(data) tar = tarfile.open(tmpname, mode="r", ignore_zeros=True) try: |