Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:97199
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmx.de designates 212.227.15.15 as permitted sender)
To: Nikita Popov <nikita.ppv@gmail.com>
References: <56e89e82-00aa-5521-f014-14cf4d588157@cubiclesoft.com>
 <CAJW__o2vn0PGJEYQ8-qojYCBTd7Czjf193NEzzaCqK+NHEHZ6A@mail.gmail.com>
 <5d865bc1-dc06-54b8-800e-99f1c2914632@cubiclesoft.com>
 <CAJW__o3qa=Fio_ZenLvVhL=DXiD=e-r8haxUcLEq2pjVib8xwQ@mail.gmail.com>
 <CAEKnhAFbXaydKSNP2pmDLnDnhx7M+vTVXiUhwcHCNX=ooHiKpA@mail.gmail.com>
 <024beaa8-b93a-10e5-bbe1-f1af1b950b81@cubiclesoft.com>
 <c2fbde5e-00cf-0cb7-f9d4-e25b8805c8e6@gmx.de>
 <CAF+90c-p+=zm3qq-kU3OxAt+GG4yJ88Z4tyBSBz02bZfR=OCgw@mail.gmail.com>
 <ceeed1be-0167-a7bc-d312-e05bcebea4ae@gmx.de>
Cc: Thomas Hruska <thruska@cubiclesoft.com>, Jakub Zelenka <bukka@php.net>,
 Kalle Sommer Nielsen <kalle@php.net>,
 PHP internals list <internals@lists.php.net>
Message-ID: <976f94e2-bda7-5411-1d8a-5840f02f7557@gmx.de>
Date: Sat, 26 Nov 2016 15:14:26 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.5.0
MIME-Version: 1.0
In-Reply-To: <ceeed1be-0167-a7bc-d312-e05bcebea4ae@gmx.de>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Translating optional zval
 **withzend_parse_parameters()to PHP 7
From: cmbecker69@gmx.de ("Christoph M. Becker")

On 26.11.2016 at 13:55, Christoph M. Becker wrote:

> On 26.11.2016 at 12:45, Nikita Popov wrote:
> 
>> The fact that php_pcre.c uses zval_dtor() is simply a bug, because code like
>>
>>     $obj = new stdClass;
>>     $obj->obj = $obj;
>>     preg_match('/./', 'x', $obj);
>>
>> leaks.
> 
> Indeed.  At least preg_replace()'s $count parameter has the same issue.
> Is there already a bug report about this?

FTR: reported as <https://bugs.php.net/bug.php?id=73612>.

-- 
Christoph M. Becker