- Fix string compare in unicode mode

johannes · johannes · commit a9d356663b1d · 2007-03-17T23:38:41.000Z
diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
@@ -2027,15 +2027,15 @@ ZEND_METHOD(reflection_parameter, getClass)
 		 * TODO: Think about moving these checks to the compiler or some sort of
 		 * lint-mode.
 		 */
-		if (0 == strncmp(param->arg_info->class_name.v, "self", sizeof("self")- 1)) {
+		if (ZEND_U_EQUAL(ZEND_STR_TYPE, param->arg_info->class_name, param->arg_info->class_name_len, "self", sizeof("self")- 1)) {
 			zend_class_entry *ce= param->fptr->common.scope;
 			if (!ce) {
 			   zend_throw_exception_ex(reflection_exception_ptr, 0 TSRMLS_CC, 
 				   "Parameter uses 'self' as type hint but function is not a class member!");
 			   return;
 			}
 			pce= &ce;
-		} else if (0 == strncmp(param->arg_info->class_name.v, "parent", sizeof("parent")- 1)) {
+		} else if (ZEND_U_EQUAL(ZEND_STR_TYPE, param->arg_info->class_name,  param->arg_info->class_name_len, "parent", sizeof("parent")- 1)) {
 			zend_class_entry *ce= param->fptr->common.scope;
 			if (!ce) {
 			   zend_throw_exception_ex(reflection_exception_ptr, 0 TSRMLS_CC, 
diff --git a/ext/reflection/tests/bug39884.phpt b/ext/reflection/tests/bug39884.phpt
@@ -20,3 +20,9 @@ object(ReflectionClass)#4 (1) {
   ["name"]=>
   string(13) "stubParamTest"
 }
+--UEXPECT--
+object(ReflectionClass)#4 (1) {
+  [u"name"]=>
+  unicode(13) "stubParamTest"
+}
+
