Re: memory leak in the new power operator tests

On Thu, Feb 6, 2014 at 4:40 PM, Ferenc Kovacs <tyra3l@gmail.com> wrote: > Hi, > > After Bob merged the pow related changes, executing the new tests with > --enable-debug I've got two memory leaks: > > [tyrael@Ferencs-MacBook-Pro-135 php-src.git (PHP-5.6.0 ✗)]$ cat > ext/standard/tests/math/pow_variation1_64bit.diff > ext/standard/tests/math/pow_variation2.diff > 083+ [Thu Feb 6 16:20:51 2014] Script: > > '/Users/tyrael/checkouts/php-src.git/ext/standard/tests/math/pow_variation1_64bit.php' > 084+ /Users/tyrael/checkouts/php-src.git/Zend/zend_API.c(314) : Freeing > 0x10D280C50 (72 bytes), > script=/Users/tyrael/checkouts/php-src.git/ext/standard/tests/math/pow_variation1_64bit.php > 085+ /Users/tyrael/checkouts/php-src.git/Zend/zend_variables.c(141) : > Actual location (location was relayed) > 086+ === Total 1 memory leaks detected ===083+ [Thu Feb 6 16:20:51 2014] > Script: > '/Users/tyrael/checkouts/php-src.git/ext/standard/tests/math/pow_variation2.php' > 084+ /Users/tyrael/checkouts/php-src.git/Zend/zend_API.c(314) : Freeing > 0x10CB8CBE0 (72 bytes), > script=/Users/tyrael/checkouts/php-src.git/ext/standard/tests/math/pow_variation2.php > 085+ /Users/tyrael/checkouts/php-src.git/Zend/zend_variables.c(141) : > Actual location (location was relayed) > 086+ === Total 1 memory leaks detected ===% > > not sure if these were originally in the PR or just something went wrong > with the merge/cherry pick, but would be nice if somebody could look into > it. > thanks! > > -- > Ferenc Kovács > @Tyr43l - http://tyrael.hu > it seems the leak occures when pow() is called with an array: [tyrael@Ferencs-MacBook-Pro-135 php-src.git (PHP-5.6.0 ✗)]$ ./sapi/cli/php -r 'pow(array(),2);' [Thu Feb 6 17:32:43 2014] Script: '-' Zend/zend_vm_execute.h(28951) : Freeing 0x1100B9ED8 (72 bytes), script=- /Users/tyrael/checkouts/php-src.git/Zend/zend_API.c(1011) : Actual location (location was relayed) === Total 1 memory leaks detected === -- Ferenc Kovács @Tyr43l - http://tyrael.hu