diff options
| author | Christoph M. Becker <cmbecker69@gmx.de> | 2020-12-02 14:49:43 +0100 |
|---|---|---|
| committer | Christoph M. Becker <cmbecker69@gmx.de> | 2020-12-04 13:02:29 +0100 |
| commit | 8588ae72156eeead928a8fe93bb8a5ab293f1e89 (patch) | |
| tree | 54144ab47f21410d3ac86b19d52ce638661ffaab /ext/phar/tests | |
| parent | 8f8e6f95af219bc2e10d52c34016bc993c8419ef (diff) | |
| download | php-git-8588ae72156eeead928a8fe93bb8a5ab293f1e89.tar.gz | |
Fix #75102: `PharData` says invalid checksum for valid tar
Apparently, there are broken tarballs out there which are actually in
ustar format, but did not write the `ustar` marker. Since popular tar
tools like GNU tar and 7zip have no issues dealing with such tarballs,
Phar should also be more resilient.
Thus, when the first checksum check of a tarball in (presumed) in old-
style format fails, we check whether the checksum would be suitable for
ustar format; if so, we treat the tarball as being in ustar format.
Closes GH-6479.
Diffstat (limited to 'ext/phar/tests')
| -rw-r--r-- | ext/phar/tests/bug75102.phpt | 13 | ||||
| -rw-r--r-- | ext/phar/tests/bug75102.tar | bin | 0 -> 2048 bytes |
2 files changed, 13 insertions, 0 deletions
diff --git a/ext/phar/tests/bug75102.phpt b/ext/phar/tests/bug75102.phpt new file mode 100644 index 0000000000..9ed133c09d --- /dev/null +++ b/ext/phar/tests/bug75102.phpt @@ -0,0 +1,13 @@ +--TEST-- +Bug #75102 (`PharData` says invalid checksum for valid tar) +--SKIPIF-- +<?php +if (!extension_loaded('phar')) die('skip phar extension not available'); +?> +--FILE-- +<?php +$phar = new PharData(__DIR__ . '/bug75102.tar'); +var_dump(file_get_contents($phar['test.txt']->getPathName())); +?> +--EXPECT-- +string(9) "yada yada" diff --git a/ext/phar/tests/bug75102.tar b/ext/phar/tests/bug75102.tar Binary files differnew file mode 100644 index 0000000000..532cc3299b --- /dev/null +++ b/ext/phar/tests/bug75102.tar |