From: Bob Weinand Date: Mon, 3 Aug 2015 18:49:41 +0000 (+0200) Subject: Fix last invalid read X-Git-Tag: php-7.0.0beta3~5^2~8 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b20953118ba41cd9465ccc84e79b6eef3f539573;p=php Fix last invalid read Also revert debugging run-tests.php changes... --- diff --git a/run-tests.php b/run-tests.php index ae39fe4c71..ba44ce5464 100755 --- a/run-tests.php +++ b/run-tests.php @@ -1878,7 +1878,7 @@ TEST $file if (version_compare($valgrind_version, '3.3.0', '>=')) { /* valgrind 3.3.0+ doesn't have --log-file-exactly option */ - $cmd = "valgrind -q --tool=memcheck --trace-children=yes --show-leak-kinds=definite,indirect --log-file=$memcheck_filename $cmd"; + $cmd = "valgrind -q --tool=memcheck --trace-children=yes --log-file=$memcheck_filename $cmd"; } else { $cmd = "valgrind -q --tool=memcheck --trace-children=yes --log-file-exactly=$memcheck_filename $cmd"; } @@ -1933,7 +1933,7 @@ COMMAND $cmd $passed = false; if ($leak_check) { // leak check - $leaked = filesize($memcheck_filename) > 367; + $leaked = filesize($memcheck_filename) > 0; if (!$leaked) { @unlink($memcheck_filename); diff --git a/sapi/phpdbg/phpdbg_list.c b/sapi/phpdbg/phpdbg_list.c index 89e352a971..8369018fc9 100644 --- a/sapi/phpdbg/phpdbg_list.c +++ b/sapi/phpdbg/phpdbg_list.c @@ -244,11 +244,11 @@ zend_op_array *phpdbg_compile_file(zend_file_handle *file, int type) { return NULL; } - data.buf = emalloc(data.len + 1); + data.buf = emalloc(data.len + ZEND_MMAP_AHEAD + 1); if (data.len > 0) { memcpy(data.buf, bufptr, data.len); } - data.buf[data.len] = 0; + memset(data.buf + data.len, 0, ZEND_MMAP_AHEAD + 1); data.filename = filename; data.line[0] = 0;