Bug #60082 [Asn->Csd]: 100% CPU / when using references with ArrayObject(&$ref).

Edit report at https://bugs.php.net/bug.php?id=60082&edit=1 ID: 60082 Updated by: tony2001@php.net Reported by: tklingenberg at lastflood dot net Summary: 100% CPU / when using references with ArrayObject(&$ref). -Status: Assigned +Status: Closed Type: Bug Package: SPL related Operating System: GNU/Linux PHP Version: 5.3.8 Assigned To: helly 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: ------------------------------------------------------------------------ [2011-11-18 05:22:24] tony2001@php.net Automatic comment from SVN on behalf of tony2001 Revision: http://svn.php.net/viewvc/?view=revision&revision=319432 Log: fix bug #60082 (Crash in ArrayObject() when using recursive references) --Tis line, and those below, will be ignored-- M trunk/ext/spl/spl_array.c M branches/PHP_5_4/ext/spl/spl_array.c M branches/PHP_5_3/ext/spl/spl_array.c M branches/PHP_5_3/NEWS ------------------------------------------------------------------------ [2011-11-01 10:10:18] tyrael@php.net Automatic comment from SVN on behalf of tyrael Revision: http://svn.php.net/viewvc/?view=revision&revision=318635 Log: marking ext/spl/tests/bug60082.phpt as slow test ------------------------------------------------------------------------ [2011-10-20 12:22:20] laruence@php.net hmm, your patch seems better, agree. ------------------------------------------------------------------------ [2011-10-20 05:46:43] tony2001@php.net I'd suggest to try to detect recursion in this case. Unfortunately, I don't see any way to do it except by adding another field to spl_array internal struct. But on the other hand, this struct is not used anywhere except spl_array.c itself, so it should be fine. The patch is attached. ------------------------------------------------------------------------ [2011-10-20 05:44:42] tony2001@php.net The following patch has been added/updated: Patch Name: recursion-detection Revision: 1319089482 URL: https://bugs.php.net/patch-display.php?bug=60082&patch=recursion-detection&revision=1319089482 ------------------------------------------------------------------------ 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=60082 -- Edit this bug report at https://bugs.php.net/bug.php?id=60082&edit=1