ID: 31348 Updated by: [EMAIL PROTECTED] Reported By: tony2001 at phpclub dot net -Status: Open +Status: Assigned Bug Type: SPL related Operating System: Linux 2.6.9 PHP Version: 5CVS-2004-12-30 (dev) -Assigned To: +Assigned To: helly
Previous Comments: ------------------------------------------------------------------------ [2004-12-30 10:46:51] tony2001 at phpclub dot net Description: ------------ CachingIterator::rewind() leaks some memory. I can propose this small patch, but I'm almost sure it's not the real solution. Index: spl_iterators.c =================================================================== RCS file: /repository/php-src/ext/spl/spl_iterators.c,v retrieving revision 1.61 diff -u -r1.61 spl_iterators.c --- spl_iterators.c 29 Dec 2004 16:11:42 -0000 1.61 +++ spl_iterators.c 30 Dec 2004 09:43:14 -0000 @@ -752,7 +752,8 @@ } if (intern->dit_type == DIT_CachingIterator || intern->dit_type == DIT_CachingRecursiveIterator) { if (intern->u.caching.zstr) { - zval_ptr_dtor(&intern->u.caching.zstr); + zval_dtor(intern->u.caching.zstr); + FREE_ZVAL(intern->u.caching.zstr); intern->u.caching.zstr = NULL; } if (intern->u.caching.zchildren) { Reproduce code: --------------- <?php $a = Array("some","blah"); $i = new ArrayIterator($a); $ci = new CachingIterator($i); $ci->rewind(); echo "Done\n"; ?> Expected result: ---------------- Done Actual result: -------------- Done /home/dev/php-src/ext/spl/spl_iterators.c(1276) : Freeing 0x082E9064 (16 bytes), script=/www/index.php /home/dev/php-src/Zend/zend_variables.h(45) : Freeing 0x082E8E9C (5 bytes), script=/www/index.php /home/dev/php-src/Zend/zend_variables.c(120) : Actual location (location was relayed) === Total 2 memory leaks detected === ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=31348&edit=1