diff options
| author | Stanislav Malyshev <stas@php.net> | 2018-12-29 18:25:37 -0800 |
|---|---|---|
| committer | Stanislav Malyshev <stas@php.net> | 2019-01-06 11:33:32 -0800 |
| commit | 428d8164ffcf6f75a6cc9d4056e54bfd450dac03 (patch) | |
| tree | ea1bb45b93653801688a8793fb58c4596b23fff9 | |
| parent | 4fc0bceb7c39be206c73f69993e3936ef329f656 (diff) | |
| download | php-git-428d8164ffcf6f75a6cc9d4056e54bfd450dac03.tar.gz | |
Fix bug #77247 (heap buffer overflow in phar_detect_phar_fname_ext)
| -rw-r--r-- | ext/phar/phar.c | 2 | ||||
| -rw-r--r-- | ext/phar/tests/bug77247.phpt | 14 |
2 files changed, 15 insertions, 1 deletions
diff --git a/ext/phar/phar.c b/ext/phar/phar.c index 47ff8cd790..6e5cec2462 100644 --- a/ext/phar/phar.c +++ b/ext/phar/phar.c @@ -2017,7 +2017,7 @@ next_extension: } while (pos != filename && (*(pos - 1) == '/' || *(pos - 1) == '\0')) { - pos = memchr(pos + 1, '.', filename_len - (pos - filename) + 1); + pos = memchr(pos + 1, '.', filename_len - (pos - filename) - 1); if (!pos) { return FAILURE; } diff --git a/ext/phar/tests/bug77247.phpt b/ext/phar/tests/bug77247.phpt new file mode 100644 index 0000000000..588975f9f2 --- /dev/null +++ b/ext/phar/tests/bug77247.phpt @@ -0,0 +1,14 @@ +--TEST-- +PHP bug #77247 (heap buffer overflow in phar_detect_phar_fname_ext) +--SKIPIF-- +<?php if (!extension_loaded("phar")) die("skip"); ?> +--FILE-- +<?php +try { +var_dump(new Phar('a/.b', 0,'test.phar')); +} catch(UnexpectedValueException $e) { + echo "OK"; +} +?> +--EXPECT-- +OK
\ No newline at end of file |