Bug #61000 [Asn->Csd]: Exceeding max nesting level doesn't delete numerical vars
Edit report at https://bugs.php.net/bug.php?id=61000&edit=1
ID: 61000
Updated by: laruence@php.net
Reported by: cataphract@php.net
Summary: Exceeding max nesting level doesn't delete numerical
vars
-Status: Assigned
+Status: Closed
Type: Bug
Package: Scripting Engine problem
Operating System: Irrelevant
PHP Version: trunk-SVN-2012-02-07 (snap)
Assigned To: laruence
Block user comment: N
Private report: N
New Comment:
This bug has been fixed in SVN.
Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
For Windows:
http://windows.php.net/snapshots/
Thank you for the report, and for helping us make PHP better.
Previous Comments:
------------------------------------------------------------------------
[2012-03-02 03:17:57] laruence@php.net
Automatic comment from SVN on behalf of laruence
Revision: http://svn.php.net/viewvc/?view=revision&revision=323769
Log: MFH: Fixed bug #61000 (Exceeding max nesting level doesn't delete numerical vars)
------------------------------------------------------------------------
[2012-02-08 14:10:43] laruence@php.net
fixed in 5.3, trunk. will close this when I commit to 5.4 after 5.4 release.
thanks
------------------------------------------------------------------------
[2012-02-08 14:07:28] laruence@php.net
Automatic comment from SVN on behalf of laruence
Revision: http://svn.php.net/viewvc/?view=revision&revision=323123
Log: Fixed bug #61000 (Exceeding max nesting level doesn't delete numerical vars).
------------------------------------------------------------------------
[2012-02-08 06:10:48] laruence@php.net
I am wondering that was there a disscussion about how to fix this? seems replace
zend_hash_del with zend_symbol_del will solve this issue.
since it is so suspicious(seems too easy), so I attached my fix.
<laruence> I was wondering is there already a discussion about how to fix it ?
if not, I think zend_symbol_del will works
<Rasmus> I think Catahract already had a fix, didn't he?
<Rasmus> it was probably just that though
thanks
------------------------------------------------------------------------
[2012-02-08 06:08:12] laruence@php.net
The following patch has been added/updated:
Patch Name: bug61000.patch
Revision: 1328681292
URL: https://bugs.php.net/patch-display.php?bug=61000&patch=bug61000.patch&revision=1328681292
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=61000
--
Edit this bug report at https://bugs.php.net/bug.php?id=61000&edit=1
Thread (5 messages)