Can someone have a look at this - it appears to work now. - eg. It runs overloaded classes ok without segfaulting.
Regards Alan
-- Edit this bug report at http://bugs.php.net/?id=25831&edit=1
[EMAIL PROTECTED] wrote:
ID: 25831 Updated by: [EMAIL PROTECTED] Reported By: viking at dslnorthwest dot net Status: Verified Bug Type: Scripting Engine problem Operating System: all PHP Version: 4.3.4RC2-dev New Comment:
-- Can you help out? Need Consulting Services or Know of a Job? http://www.akbkhome.com
-- Can you help out? Need Consulting Services or Know of a Job? http://www.akbkhome.com
? diff.txt Index: overload.c =================================================================== RCS file: /repository/php-src/ext/overload/Attic/overload.c,v retrieving revision 1.20.2.3 diff -u -r1.20.2.3 overload.c --- overload.c 15 Apr 2003 04:14:21 -0000 1.20.2.3 +++ overload.c 25 Oct 2003 04:08:34 -0000 @@ -507,7 +507,7 @@ static void overload_call_method(INTERNAL_FUNCTION_PARAMETERS, zend_property_reference *property_reference) { zval ***args; - zval *retval = NULL; + int call_result; zend_bool use_call_handler = 1; zval *object = property_reference->object; @@ -526,13 +526,15 @@ args = (zval ***)emalloc(ZEND_NUM_ARGS() * sizeof(zval **)); - if (zend_get_parameters_array_ex(ZEND_NUM_ARGS(), args) == FAILURE) { - efree(args); - php_error(E_WARNING, "unable to obtain arguments"); - return; - } if (use_call_handler) { + zval *retval = NULL; + + if (zend_get_parameters_array_ex(ZEND_NUM_ARGS(), args) == FAILURE) { + efree(args); + php_error(E_WARNING, "unable to obtain arguments"); + return; + } zval **handler_args[3]; zval *arg_array; zval result, *result_ptr = &result; @@ -588,23 +590,29 @@ } zval_ptr_dtor(&retval); } else { + zval retval; + if (zend_get_parameters_array(ht, ZEND_NUM_ARGS(), (zval **)args) == FAILURE) { + efree(args); + php_error(E_WARNING, "unable to obtain arguments"); + return; + } + ZVAL_STRINGL(&call_handler, Z_STRVAL(method->element), Z_STRLEN(method->element), 0); - call_result = call_user_function_ex(NULL, + call_result = call_user_function(NULL, &object, &call_handler, - &retval, - ZEND_NUM_ARGS(), args, - 0, NULL TSRMLS_CC); + return_value, + ZEND_NUM_ARGS(),(zval **)args TSRMLS_CC); - if (call_result == FAILURE || !retval) { + if (call_result == FAILURE || !&return_value) { efree(args); php_error(E_WARNING, "unable to call %s::%s() method", Z_OBJCE_P(object)->name, Z_STRVAL(method->element)); return; } - *return_value = *retval; + zval_copy_ctor(return_value); - zval_ptr_dtor(&retval); + } efree(args);
-- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php