diff options
| -rw-r--r-- | NEWS | 1 | ||||
| -rw-r--r-- | ext/phar/tar.c | 9 | ||||
| -rw-r--r-- | ext/phar/tests/bug75102.phpt | 13 | ||||
| -rw-r--r-- | ext/phar/tests/bug75102.tar | bin | 0 -> 2048 bytes |
4 files changed, 23 insertions, 0 deletions
@@ -36,6 +36,7 @@ PHP NEWS - Phar: . Fixed bug #73809 (Phar Zip parse crash - mmap fail). (cmb) + . Fixed #75102 (`PharData` says invalid checksum for valid tar). (cmb) - Phpdbg: . Fixed bug #76813 (Access violation near NULL on source operand). (cmb) diff --git a/ext/phar/tar.c b/ext/phar/tar.c index fb9cacfc73..bae1a4bf7b 100644 --- a/ext/phar/tar.c +++ b/ext/phar/tar.c @@ -268,6 +268,15 @@ int phar_parse_tarfile(php_stream* fp, char *fname, size_t fname_len, char *alia memset(hdr->checksum, ' ', sizeof(hdr->checksum)); sum2 = phar_tar_checksum(buf, old?sizeof(old_tar_header):sizeof(tar_header)); + if (old && sum2 != sum1) { + uint32_t sum3 = phar_tar_checksum(buf, sizeof(tar_header)); + if (sum3 == sum1) { + /* apparently a broken tar which is in ustar format w/o setting the ustar marker */ + sum2 = sum3; + old = 0; + } + } + size = entry.uncompressed_filesize = entry.compressed_filesize = phar_tar_number(hdr->size, sizeof(hdr->size)); 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 |