Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:72345
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.216.182 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAH-PCH5WD2x-qEzexCO2A9oKFZQKPH39ZyoSD08DoxjQwx_Axg@mail.gmail.com>
References: <CAH-PCH5WD2x-qEzexCO2A9oKFZQKPH39ZyoSD08DoxjQwx_Axg@mail.gmail.com>
Date: Thu, 6 Feb 2014 17:34:33 +0100
Message-ID: <CAH-PCH5HWa4NipQMgGR+dcRZ1qVPg4k4yPFONM7TZ6K9LcOqiw@mail.gmail.com>
To: Bob Weinand <bobwei9@hotmail.com>, Tjerk Meesters <tjerk.meesters@gmail.com>, 
	PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a113a359ccd4efc04f1bf7247
Subject: Re: memory leak in the new power operator tests
From: tyra3l@gmail.com (Ferenc Kovacs)

--001a113a359ccd4efc04f1bf7247
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

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 =E2=9C=97)]$ 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_variati=
on1_64bit.php'
> 084+ /Users/tyrael/checkouts/php-src.git/Zend/zend_API.c(314) :  Freeing
> 0x10D280C50 (72 bytes),
> script=3D/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+ =3D=3D=3D Total 1 memory leaks detected =3D=3D=3D083+ [Thu Feb  6 16=
:20:51 2014]
>  Script:
>  '/Users/tyrael/checkouts/php-src.git/ext/standard/tests/math/pow_variati=
on2.php'
> 084+ /Users/tyrael/checkouts/php-src.git/Zend/zend_API.c(314) :  Freeing
> 0x10CB8CBE0 (72 bytes),
> script=3D/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+ =3D=3D=3D Total 1 memory leaks detected =3D=3D=3D%
>
> 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=C3=A1cs
> @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 =E2=9C=97)]$ ./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=3D-
/Users/tyrael/checkouts/php-src.git/Zend/zend_API.c(1011) : Actual location
(location was relayed)
=3D=3D=3D Total 1 memory leaks detected =3D=3D=3D

--=20
Ferenc Kov=C3=A1cs
@Tyr43l - http://tyrael.hu

--001a113a359ccd4efc04f1bf7247--