diff --git a/NEWS b/NEWS index 75e565dc5ba..6ef5276cf8e 100644 --- a/NEWS +++ b/NEWS @@ -14,6 +14,8 @@ PHP NEWS . Fixed bug GH-14003 (Broken cleanup of unfinished calls with callable convert parameters). (ilutov) . Fixed bug GH-14013 (Erroneous dnl appended in configure). (Peter Kokot) + . Fixed bug GH-10232 (If autoloading occurs during constant resolution + filename and lineno are identified incorrectly). (ranvis) - Fibers: . Fixed bug GH-13903 (ASAN false positive underflow when executing copy()). diff --git a/Zend/tests/gh10232.phpt b/Zend/tests/gh10232.phpt new file mode 100644 index 00000000000..8e8d7ff86b5 --- /dev/null +++ b/Zend/tests/gh10232.phpt @@ -0,0 +1,33 @@ +--TEST-- +GH-10232 (Weird behaviour when a file is autoloaded in assignment of a constant) +--FILE-- +getLine(), "\n"; + require_once __DIR__ . '/gh10232/constant_def.inc'; +}, true); + + +class ConstantRef +{ + public const VALUE = ConstantDef::VALUE; +} + +ConstantRef::VALUE; + +?> +--EXPECTF-- +Notice: 7 in %sgh10232.php on line 7 +Exception on line 8 + +Notice: constant_def.inc in %sconstant_def.inc on line 3 +Exception in constant_def.inc on line 4 + +Notice: required.inc in %srequired.inc on line 3 +Exception in required.inc on line 4 diff --git a/Zend/tests/gh10232/constant_def.inc b/Zend/tests/gh10232/constant_def.inc new file mode 100644 index 00000000000..22e469af638 --- /dev/null +++ b/Zend/tests/gh10232/constant_def.inc @@ -0,0 +1,12 @@ +getFile()), ' on line ', $ex->getLine(), "\n"; + +require_once 'required.inc'; // The script of the same directory. + +class ConstantDef +{ + const VALUE = true; +} diff --git a/Zend/tests/gh10232/required.inc b/Zend/tests/gh10232/required.inc new file mode 100644 index 00000000000..d5ff90da021 --- /dev/null +++ b/Zend/tests/gh10232/required.inc @@ -0,0 +1,5 @@ +getFile()), ' on line ', $ex->getLine(), "\n"; diff --git a/Zend/tests/gh7771_3.phpt b/Zend/tests/gh7771_3.phpt index f6780814b89..e44a01a8aec 100644 --- a/Zend/tests/gh7771_3.phpt +++ b/Zend/tests/gh7771_3.phpt @@ -12,5 +12,5 @@ spl_autoload_register(function($class) use ($classlist) { var_dump(D::HW); ?> --EXPECTF-- -Fatal error: Constant expression contains invalid operations in %sgh7771_3.php(7) : eval()'d code(1) : eval()'d code on line 1 +Fatal error: Constant expression contains invalid operations in %sgh7771_3.php(7) : eval()'d code on line 1 diff --git a/Zend/zend_execute_API.c b/Zend/zend_execute_API.c index 28862525e65..63967a97a74 100644 --- a/Zend/zend_execute_API.c +++ b/Zend/zend_execute_API.c @@ -1213,9 +1213,15 @@ ZEND_API zend_class_entry *zend_lookup_class_ex(zend_string *name, zend_string * autoload_name = zend_string_copy(name); } + zend_string *previous_filename = EG(filename_override); + zend_long previous_lineno = EG(lineno_override); + EG(filename_override) = NULL; + EG(lineno_override) = -1; zend_exception_save(); ce = zend_autoload(autoload_name, lc_name); zend_exception_restore(); + EG(filename_override) = previous_filename; + EG(lineno_override) = previous_lineno; zend_string_release_ex(autoload_name, 0); zend_hash_del(EG(in_autoload), lc_name);