Re: com php-src: Fixed bug #74878: NEWS Zend/zend_API.c

From: Date: Mon, 10 Jul 2017 09:16:08 +0000
Subject: Re: com php-src: Fixed bug #74878: NEWS Zend/zend_API.c
References: 1 2  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-98487@lists.php.net to get a copy of this message
On Mon, Jul 10, 2017 at 10:08 AM, Dmitry Stogov <dmitry@zend.com> wrote: > This commit introduced a lot of memory leaks. > > Probably, you meant string interning only for internal classes. > > Even for internal classes only, this may require additional changes in > opcache, that copies interned strings into shared memory. > > > It's better to revert this, before complete fix. > If this is causing issues, please revert it. I think it should always intern even for non-internal classes (these strings are likely duplicated compile-time strings, something we usually intern), but I clearly did not do it correctly. Nikita

« previous php.cvs (#98487) next »