From 396c1e990a0e513cf2e8510e9b6f82ab425c9e3a Mon Sep 17 00:00:00 2001 From: Dmitry Stogov Date: Tue, 20 Nov 2012 12:51:55 +0400 Subject: [PATCH] Fixed bug #63468 (wrong called method as callback with inheritance) --- NEWS | 2 ++ Zend/tests/bug63468.phpt | 30 ++++++++++++++++++++++++++++++ Zend/zend_API.c | 2 +- 3 files changed, 33 insertions(+), 1 deletion(-) create mode 100644 Zend/tests/bug63468.phpt diff --git a/NEWS b/NEWS index 4f89d074344..717a382b20c 100644 --- a/NEWS +++ b/NEWS @@ -5,6 +5,8 @@ PHP NEWS - Zend Engine: . Fixed bug #63512 (parse_ini_file() with INI_SCANNER_RAW removes quotes from value). (Pierrick) + . Fixed bug #63468 (wrong called method as callback with inheritance). + (Laruence) - Core: . Fixed bug #63451 (config.guess file does not have AIX 7 defined, shared objects are not created). (kemcline at au1 dot ibm dot com) diff --git a/Zend/tests/bug63468.phpt b/Zend/tests/bug63468.phpt new file mode 100644 index 00000000000..00b5a41c901 --- /dev/null +++ b/Zend/tests/bug63468.phpt @@ -0,0 +1,30 @@ +--TEST-- +Bug #63468 (wrong called method as callback with inheritance) +--FILE-- +run()); +--EXPECT-- +string(3) "Bar" + diff --git a/Zend/zend_API.c b/Zend/zend_API.c index d529775764b..c1b501b0b7e 100644 --- a/Zend/zend_API.c +++ b/Zend/zend_API.c @@ -2520,7 +2520,7 @@ static int zend_is_callable_check_func(int check_flags, zval *callable, zend_fca } else if (zend_hash_find(ftable, lmname, mlen+1, (void**)&fcc->function_handler) == SUCCESS) { retval = 1; if ((fcc->function_handler->op_array.fn_flags & ZEND_ACC_CHANGED) && - EG(scope) && + !strict_class && EG(scope) && instanceof_function(fcc->function_handler->common.scope, EG(scope) TSRMLS_CC)) { zend_function *priv_fbc;