- Fixed bug #54624 (class_alias and type hint)

diff --git a/Zend/tests/bug54624.phpt b/Zend/tests/bug54624.phpt
new file mode 100644 (file)
index 0000000..1d0c1ce
--- /dev/null
+++ b/Zend/tests/bug54624.phpt
@@ -0,0 +1,26 @@
+--TEST--
+Bug #54624 (class_alias and type hint)
+--FILE--
+<?php
+class A
+{
+       function foo(A $param) {
+       }
+
+}
+
+class_alias('A', 'AliasA');
+
+eval(' 
+       class B extends A
+       {
+               function foo(AliasA $param) {
+               }
+
+       }
+');
+
+echo "DONE\n";
+?>
+--EXPECTF--
+DONE

diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c
index 2f3fad9ba482d928df60a0ad864f3a929c87acaf..8cc6eb5ee75f4ee451c95aede3fdd72519307fe2 100644 (file)
--- a/Zend/zend_compile.c
+++ b/Zend/zend_compile.c
@@ -2918,11 +2918,23 @@ static zend_bool zend_do_perform_implementation_check(const zend_function *fe, c
                        && strcasecmp(fe->common.arg_info[i].class_name, proto->common.arg_info[i].class_name)!=0) {
                        char *colon;
 
-                       if (fe->common.type != ZEND_USER_FUNCTION ||
-                           strchr(proto->common.arg_info[i].class_name, '\\') != NULL ||
+                       if (fe->common.type != ZEND_USER_FUNCTION) {
+                               return 0;
+                       } else if (strchr(proto->common.arg_info[i].class_name, '\\') != NULL ||
                            (colon = zend_memrchr(fe->common.arg_info[i].class_name, '\\', fe->common.arg_info[i].class_name_len)) == NULL ||
                            strcasecmp(colon+1, proto->common.arg_info[i].class_name) != 0) {
-                               return 0;
+                               zend_class_entry **fe_ce, **proto_ce;
+                               TSRMLS_FETCH();
+                               int found = zend_lookup_class(fe->common.arg_info[i].class_name, fe->common.arg_info[i].class_name_len, &fe_ce TSRMLS_CC);
+                               int found2 = zend_lookup_class(proto->common.arg_info[i].class_name, proto->common.arg_info[i].class_name_len, &proto_ce TSRMLS_CC);
+                               
+                               /* Check for class alias */
+                               if (found != SUCCESS || found2 != SUCCESS ||
+                                       (*fe_ce)->type == ZEND_INTERNAL_CLASS ||
+                                       (*proto_ce)->type == ZEND_INTERNAL_CLASS ||
+                                       *fe_ce != *proto_ce) {
+                                       return 0;
+                               }
                        }
                }
                if (fe->common.arg_info[i].type_hint != proto->common.arg_info[i].type_hint) {