dmitry Mon Sep 12 05:58:37 2005 EDT
Modified files:
/php-src/ext/standard array.c
Log:
Additinal fix for bug #34277 (array_filter() crashes with references and
objects)
http://cvs.php.net/diff.php/php-src/ext/standard/array.c?r1=1.324&r2=1.325&ty=u
Index: php-src/ext/standard/array.c
diff -u php-src/ext/standard/array.c:1.324 php-src/ext/standard/array.c:1.325
--- php-src/ext/standard/array.c:1.324 Mon Sep 12 05:53:22 2005
+++ php-src/ext/standard/array.c Mon Sep 12 05:58:35 2005
@@ -21,7 +21,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: array.c,v 1.324 2005/09/12 09:53:22 derick Exp $ */
+/* $Id: array.c,v 1.325 2005/09/12 09:58:35 dmitry Exp $ */
#include "php.h"
#include "php_ini.h"
@@ -4364,7 +4364,7 @@
PHP_FUNCTION(array_filter)
{
zval **input, **callback = NULL;
- zval *array;
+ zval *array, *func = NULL;
zval **operand;
zval **args[1];
zval *retval = NULL;
@@ -4384,10 +4384,13 @@
php_error_docref(NULL TSRMLS_CC, E_WARNING, "The first argument
should be an array");
return;
}
+ if (callback) {
+ func = *callback;
+ }
array = *input;
if (ZEND_NUM_ARGS() > 1) {
- if (!zend_is_callable(*callback, 0, &callback_name)) {
+ if (!zend_is_callable(func, 0, &callback_name)) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "The second
argument, '%R', should be a valid callback", Z_TYPE(callback_name),
Z_UNIVAL(callback_name));
zval_dtor(&callback_name);
return;
@@ -4405,14 +4408,14 @@
zend_hash_move_forward_ex(Z_ARRVAL_P(array), &pos)) {
zend_uchar utype;
- if (callback) {
+ if (func) {
zend_fcall_info fci;
args[0] = operand;
fci.size = sizeof(fci);
fci.function_table = EG(function_table);
- fci.function_name = *callback;
+ fci.function_name = func;
fci.symbol_table = NULL;
fci.object_pp = NULL;
fci.retval_ptr_ptr = &retval;
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
