com php-src: FIx bug #70433 - Uninitialized pointer in phar_make_dirstream when zip entry filename is "/": ext/phar/dirstream.c ext/phar/tests/bug70
433.phpt ext/phar/tests/bug70433.zip

Commit: e78ac461dbefb7c4a3e9fde78d50fbc56b7b0183 Author: Stanislav Malyshev <stas@php.net> Mon, 28 Sep 2015 17:12:35 -0700 Parents: d698f0ae51f67c9cce870b09c59df3d6ba959244 Branches: PHP-5.5 PHP-5.6 PHP-7.0 master Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=e78ac461dbefb7c4a3e9fde78d50fbc56b7b0183 Log: FIx bug #70433 - Uninitialized pointer in phar_make_dirstream when zip entry filename is "/" Bugs: https://bugs.php.net/70433 Changed paths: M ext/phar/dirstream.c A ext/phar/tests/bug70433.phpt A ext/phar/tests/bug70433.zip Diff: diff --git a/ext/phar/dirstream.c b/ext/phar/dirstream.c index 75cf049..4728e29 100644 --- a/ext/phar/dirstream.c +++ b/ext/phar/dirstream.c @@ -207,7 +207,7 @@ static php_stream *phar_make_dirstream(char *dir, HashTable *manifest TSRMLS_DC) zend_hash_internal_pointer_reset(manifest); while (FAILURE != zend_hash_has_more_elements(manifest)) { - if (HASH_KEY_NON_EXISTENT == zend_hash_get_current_key_ex(manifest, &key, &keylen, &unused, 0, NULL)) { + if (HASH_KEY_IS_STRING != zend_hash_get_current_key_ex(manifest, &key, &keylen, &unused, 0, NULL)) { break; } diff --git a/ext/phar/tests/bug70433.phpt b/ext/phar/tests/bug70433.phpt new file mode 100644 index 0000000..cbebb34 --- /dev/null +++ b/ext/phar/tests/bug70433.phpt @@ -0,0 +1,23 @@ +--TEST-- +Phar - bug #70433 - Uninitialized pointer in phar_make_dirstream when zip entry filename is "/" +--SKIPIF-- +<?php if (!extension_loaded("phar")) die("skip"); ?> +--FILE-- +<?php +$phar = new PharData(__DIR__."/bug70433.zip"); +var_dump($phar); +$meta = $phar->getMetadata(); +var_dump($meta); +?> +DONE +--EXPECTF-- +object(PharData)#1 (3) { + ["pathName":"SplFileInfo":private]=> + string(0) "" + ["glob":"DirectoryIterator":private]=> + bool(false) + ["subPathName":"RecursiveDirectoryIterator":private]=> + string(0) "" +} +NULL +DONE diff --git a/ext/phar/tests/bug70433.zip b/ext/phar/tests/bug70433.zip new file mode 100755 index 0000000..3994a30 Binary files /dev/null and b/ext/phar/tests/bug70433.zip differ