[issue39039] zlib.error with tarfile.open

2021-09-29 Thread Łukasz Langa
Łukasz Langa added the comment: Thanks for the fix, Jack! ✨ 🍰 ✨ Since the change translated `zlib.error` to `tarfile.ReadError` which already has to be handled by user code, it's strictly decreasing the surface of necessary exception handling. So, treating this as a bug fix, I backported th

[issue39039] zlib.error with tarfile.open

2021-09-29 Thread Łukasz Langa
Łukasz Langa added the comment: New changeset 7bff4d396f20451f20977be3ce23a879c6bc3e46 by Łukasz Langa in branch '3.9': [3.9] bpo-39039: tarfile raises descriptive exception from zlib.error (GH-27766) (GH-28614) https://github.com/python/cpython/commit/7bff4d396f20451f20977be3ce23a879c6bc3e46

[issue39039] zlib.error with tarfile.open

2021-09-29 Thread Łukasz Langa
Łukasz Langa added the comment: New changeset d6b69f21d8ec4af47a9c79f3f50d20be3d0875fc by Łukasz Langa in branch '3.10': [3.10] bpo-39039: tarfile raises descriptive exception from zlib.error (GH-27766) (GH-28613) https://github.com/python/cpython/commit/d6b69f21d8ec4af47a9c79f3f50d20be3d0875

[issue39039] zlib.error with tarfile.open

2021-09-29 Thread Łukasz Langa
Change by Łukasz Langa : -- pull_requests: +26984 pull_request: https://github.com/python/cpython/pull/28614 ___ Python tracker ___ _

[issue39039] zlib.error with tarfile.open

2021-09-29 Thread Łukasz Langa
Change by Łukasz Langa : -- pull_requests: +26983 pull_request: https://github.com/python/cpython/pull/28613 ___ Python tracker ___ _

[issue39039] zlib.error with tarfile.open

2021-09-29 Thread Łukasz Langa
Łukasz Langa added the comment: New changeset b6fe8572509b77d2002eaddf99d718e9b4835684 by Jack DeVries in branch 'main': bpo-39039: tarfile raises descriptive exception from zlib.error (GH-27766) https://github.com/python/cpython/commit/b6fe8572509b77d2002eaddf99d718e9b4835684 -- nos

[issue39039] zlib.error with tarfile.open

2021-08-20 Thread jvoisin
jvoisin added the comment: The file was created with a fuzzer, like the one described in https://dustri.org/b/fuzzing-python-in-python-and-doing-it-fast.html -- ___ Python tracker __

[issue39039] zlib.error with tarfile.open

2021-08-17 Thread Jack DeVries
Jack DeVries added the comment: @jvoisin I am able to reproduce the problem when I download your script, but I am having a hard time reproducing it by passing corrupt archives to `tarfile.open`. How exactly was this file corrupted? I am trying to figure out if there are any similar implement

[issue39039] zlib.error with tarfile.open

2021-08-13 Thread Jack DeVries
Change by Jack DeVries : -- keywords: +patch nosy: +jack__d nosy_count: 3.0 -> 4.0 pull_requests: +26242 stage: -> patch review pull_request: https://github.com/python/cpython/pull/27766 ___ Python tracker _

[issue39039] zlib.error with tarfile.open

2019-12-13 Thread jvoisin
jvoisin added the comment: Sure, but as a user, I would expect a better exception, like ValueError or ReadError, along with an error message, instead of an unexpected zlib exception. -- ___ Python tracker _

[issue39039] zlib.error with tarfile.open

2019-12-13 Thread Christian Heimes
Christian Heimes added the comment: This file is also an invalid tar file: $ tar xf crash-c10c9839d987fa0df6912cb4084f43f3ce08ca82 gzip: stdin: invalid compressed data--format violated tar: Child returned status 1 tar: Error is not recoverable: exiting now -- nosy: +christian.heimes

[issue39039] zlib.error with tarfile.open

2019-12-13 Thread Karthikeyan Singaravelan
Change by Karthikeyan Singaravelan : -- nosy: +ethan.furman ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue39039] zlib.error with tarfile.open

2019-12-13 Thread jvoisin
New submission from jvoisin : The attached file produces the following stacktrace when opened via `tarfile.open`, on Python 3.7.5rc1: ``` $ cat test.py import sys import tarfile tarfile.open(sys.argv[1]) $ python3 test.py ./crash-c10c9839d987fa0df6912cb4084f43f3ce08ca82 Traceback (most recen