dmitry          Fri Sep 19 12:48:45 2008 UTC

  Modified files:              
    /php-src/ext/spl    spl_array.c 
    /php-src/ext/spl/tests      bug46115.phpt 
  Log:
  Fixed bug #46115 (Memory leak when calling a method using Reflection)
  
  
http://cvs.php.net/viewvc.cgi/php-src/ext/spl/spl_array.c?r1=1.148&r2=1.149&diff_format=u
Index: php-src/ext/spl/spl_array.c
diff -u php-src/ext/spl/spl_array.c:1.148 php-src/ext/spl/spl_array.c:1.149
--- php-src/ext/spl/spl_array.c:1.148   Mon Aug 25 19:07:12 2008
+++ php-src/ext/spl/spl_array.c Fri Sep 19 12:48:45 2008
@@ -16,7 +16,7 @@
    +----------------------------------------------------------------------+
  */
 
-/* $Id: spl_array.c,v 1.148 2008/08/25 19:07:12 jani Exp $ */
+/* $Id: spl_array.c,v 1.149 2008/09/19 12:48:45 dmitry Exp $ */
 
 #ifdef HAVE_CONFIG_H
 # include "config.h"
@@ -1230,6 +1230,7 @@
        spl_array_object *intern = 
(spl_array_object*)zend_object_store_get_object(getThis() TSRMLS_CC);
        HashTable *aht = spl_array_get_hash_table(intern, 0 TSRMLS_CC);
        zval *tmp, *arg;
+       zval *retval_ptr = NULL;
        
        MAKE_STD_ZVAL(tmp);
        Z_TYPE_P(tmp) = IS_ARRAY;
@@ -1240,14 +1241,15 @@
                        zend_throw_exception(spl_ce_BadMethodCallException, 
"Function expects exactly one argument", 0 TSRMLS_CC);
                        return;
                }
-               zval_ptr_dtor(return_value_ptr);
-               zend_call_method(NULL, NULL, NULL, fname, fname_len, 
return_value_ptr, 2, tmp, arg TSRMLS_CC);
+               zend_call_method(NULL, NULL, NULL, fname, fname_len, 
&retval_ptr, 2, tmp, arg TSRMLS_CC);
        } else {
-               zval_ptr_dtor(return_value_ptr);
-               zend_call_method(NULL, NULL, NULL, fname, fname_len, 
return_value_ptr, 1, tmp, NULL TSRMLS_CC);
+               zend_call_method(NULL, NULL, NULL, fname, fname_len, 
&retval_ptr, 1, tmp, NULL TSRMLS_CC);
        }
        Z_TYPE_P(tmp) = IS_NULL; /* we want to destroy the zval, not the 
hashtable */
        zval_ptr_dtor(&tmp);
+       if (retval_ptr) {
+               COPY_PZVAL_TO_ZVAL(*return_value, retval_ptr);
+       }
 } /* }}} */
 
 /* {{{ SPL_ARRAY_METHOD */
http://cvs.php.net/viewvc.cgi/php-src/ext/spl/tests/bug46115.phpt?r1=1.1&r2=1.2&diff_format=u
Index: php-src/ext/spl/tests/bug46115.phpt
diff -u /dev/null php-src/ext/spl/tests/bug46115.phpt:1.2
--- /dev/null   Fri Sep 19 12:48:45 2008
+++ php-src/ext/spl/tests/bug46115.phpt Fri Sep 19 12:48:45 2008
@@ -0,0 +1,11 @@
+--TEST--
+Bug #46115 (Memory leak when calling a method using Reflection)
+--FILE--
+<?php
+$h = new RecursiveArrayIterator(array());
+$x = new reflectionmethod('RecursiveArrayIterator', 'asort');
+$z = $x->invoke($h);
+?>
+DONE
+--EXPECT--
+DONE



-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to