Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:77096
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.220.174 as permitted sender)
MIME-Version: 1.0
Date: Mon, 8 Sep 2014 11:26:10 +0400
Message-ID: <CA+9eiLvfUyZMgXi+p=RWtPrMfxw5fu6YS8=JRvOJ=4Uoaj2doQ@mail.gmail.com>
To: Nikita Popov <nikita.ppv@gmail.com>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a113397e4a8361c050288bb69
Subject: Memory corruptions related to AST patch
From: dmitry@zend.com (Dmitry Stogov)

--001a113397e4a8361c050288bb69
Content-Type: text/plain; charset=UTF-8

Hi Nikita,

On weekend I ran PHP test suite with valgrind and opcache.
I noticed few new tests failures, that most probably introduced by AST
patch.

Bug #21820 ("$arr['foo']" generates bogus E_NOTICE, should be E_PARSE)
[tests/lang/bug21820.phpt]
Bug #66286: Incorrect object comparison with inheritance
[Zend/tests/bug66286.phpt]
unbraced complex variable replacement test (heredoc)
[Zend/tests/heredoc_005.phpt]
isset() can be used on dereferences of temporary expressions
[Zend/tests/varSyntax/issetOnTemp.phpt]

I suppose they indicate two different problems.
One of them indicates that some zvals became a subject for GC, however they
won't before. This may be fixed in opcache, but it's better to understand
what was changed.

Please take a look.

Thanks. Dmitry.

--001a113397e4a8361c050288bb69--