summaryrefslogtreecommitdiff
path: root/Zend/tests/use_no_eval_conflict.phpt
diff options
context:
space:
mode:
authorNikita Popov <nikic@php.net>2016-10-06 23:09:41 +0200
committerNikita Popov <nikic@php.net>2016-10-07 00:12:55 +0200
commit8754b191f77efdbb3d5dd6440e0546589fb65473 (patch)
treea99fcfdd1c7f8ab20a6f4e7f87850e771affab4b /Zend/tests/use_no_eval_conflict.phpt
parent940731df04f95d1ced5c336da39248f7229a8ddc (diff)
downloadphp-git-8754b191f77efdbb3d5dd6440e0546589fb65473.tar.gz
Fix "already in use" check inconsistencies/bugs
This fixes the following issues: * "use function" and "use const" inside namespaced code were checking for conflicts against class imports. Now they always check against the correct symbol type. * Symbol conflicts are now always checked within a single file only. Previously class uses inside namespaced code were checked globally. This behavior is illegal because symbols from other files are not visible if opcache is used, resulting in behavioral discrepancies. Additionally this made the presence/absence of symbol errors dependent on autoloading order, which is volatile. * The "single file" restriction is now enforced by collecting defined symbols inside a separate hash table. Previously it was enforced (for the non-namespaced case) by comparing the filename of the symbol declaration. However this is inaccurate if the same filename is used multiple times, such as may happen if eval() is used. * Additionally the previous approach relies on symbols being registered at compile-time, which is not the case for late-bound classes, which makes the behavior dependent on class declaration order, as well as opcache (which may cause delayed early-binding). * Lastly, conflicts are now consistently checked for conditionally defined symbols. Previously only declaration-after-use conflicts were checked in this case. Now use-after-declaration conflicts are detected as well.
Diffstat (limited to 'Zend/tests/use_no_eval_conflict.phpt')
-rw-r--r--Zend/tests/use_no_eval_conflict.phpt13
1 files changed, 13 insertions, 0 deletions
diff --git a/Zend/tests/use_no_eval_conflict.phpt b/Zend/tests/use_no_eval_conflict.phpt
new file mode 100644
index 0000000000..cf9014b77d
--- /dev/null
+++ b/Zend/tests/use_no_eval_conflict.phpt
@@ -0,0 +1,13 @@
+--TEST--
+Use conflicts should not occur across eval()s
+--FILE--
+<?php
+
+/* It is important that these two eval()s occur on the same line,
+ * as this forces them to have the same filename. */
+eval("class A {}"); eval("use Foo\A;");
+
+?>
+===DONE===
+--EXPECT--
+===DONE===