cvs: ZendEngine2 / zend_builtin_functions.c /tests get_class_vars_003.phpt get_class_vars_006.phpt

lbarnaud		Fri May 22 11:18:57 2009 UTC

  Modified files:              
    /ZendEngine2	zend_builtin_functions.c 
    /ZendEngine2/tests	get_class_vars_003.phpt get_class_vars_006.phpt 
  Log:
  Use zend_get_property_info() in get_class_vars() (fixes #46812)
  
  
http://cvs.php.net/viewvc.cgi/ZendEngine2/zend_builtin_functions.c?r1=1.403&r2=1.404&diff_format=u
Index: ZendEngine2/zend_builtin_functions.c
diff -u ZendEngine2/zend_builtin_functions.c:1.403 ZendEngine2/zend_builtin_functions.c:1.404
--- ZendEngine2/zend_builtin_functions.c:1.403	Fri May 15 17:04:30 2009
+++ ZendEngine2/zend_builtin_functions.c	Fri May 22 11:18:57 2009
@@ -17,7 +17,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: zend_builtin_functions.c,v 1.403 2009/05/15 17:04:30 felipe Exp $ */
+/* $Id: zend_builtin_functions.c,v 1.404 2009/05/22 11:18:57 lbarnaud Exp $ */
 
 #include "zend.h"
 #include "zend_API.h"
@@ -885,28 +885,23 @@
 		while (zend_hash_get_current_data_ex(properties, (void **) &prop, &pos) == SUCCESS) {
 			zstr key, class_name, prop_name;
 			uint key_len;
-			ulong num_index, h;
+			ulong num_index;
 			int prop_name_len = 0;
 			zval *prop_copy;
 			zend_uchar key_type;
 			zend_property_info *property_info;
+			zval zprop_name;
 
 			key_type = zend_hash_get_current_key_ex(properties, &key, &key_len, &num_index, 0, &pos);
 			zend_hash_move_forward_ex(properties, &pos);
 
 			zend_u_unmangle_property_name(key_type, key, key_len-1, &class_name, &prop_name);
 			prop_name_len = ZSTR_LEN(key_type, prop_name);
-			
-			h = zend_u_get_hash_value(key_type, prop_name, prop_name_len+1);
-			if (zend_u_hash_quick_find(&ce->properties_info, key_type, prop_name, prop_name_len+1, h, (void **) &property_info) == FAILURE) {
-				continue;
-			}
-			
-			if (property_info->flags & ZEND_ACC_SHADOW) {
-				continue;
-			} else if ((property_info->flags & ZEND_ACC_PRIVATE) && EG(scope) != ce) {
-				continue;
-			} else if ((property_info->flags & ZEND_ACC_PROTECTED) && zend_check_protected(ce, EG(scope)) == 0) {
+
+			ZVAL_ZSTRL(&zprop_name, key_type, prop_name, prop_name_len, 0);
+			property_info = zend_get_property_info(ce, &zprop_name, 1 TSRMLS_CC);
+
+			if (!property_info || property_info == &EG(std_property_info)) {
 				continue;
 			}
 
http://cvs.php.net/viewvc.cgi/ZendEngine2/tests/get_class_vars_003.phpt?r1=1.1&r2=1.2&diff_format=u
Index: ZendEngine2/tests/get_class_vars_003.phpt
diff -u ZendEngine2/tests/get_class_vars_003.phpt:1.1 ZendEngine2/tests/get_class_vars_003.phpt:1.2
--- ZendEngine2/tests/get_class_vars_003.phpt:1.1	Fri Aug 22 00:54:00 2008
+++ ZendEngine2/tests/get_class_vars_003.phpt	Fri May 22 11:18:57 2009
@@ -33,7 +33,7 @@
 
 ?>
 --EXPECT--
-array(5) {
+array(6) {
   [u"aaa"]=>
   int(7)
   [u"ccc"]=>
@@ -42,6 +42,8 @@
   int(1)
   [u"aa"]=>
   int(4)
+  [u"bb"]=>
+  int(5)
   [u"cc"]=>
   int(6)
 }
http://cvs.php.net/viewvc.cgi/ZendEngine2/tests/get_class_vars_006.phpt?r1=1.1&r2=1.2&diff_format=u
Index: ZendEngine2/tests/get_class_vars_006.phpt
diff -u ZendEngine2/tests/get_class_vars_006.phpt:1.1 ZendEngine2/tests/get_class_vars_006.phpt:1.2
--- ZendEngine2/tests/get_class_vars_006.phpt:1.1	Fri Aug 22 00:54:00 2008
+++ ZendEngine2/tests/get_class_vars_006.phpt	Fri May 22 11:18:57 2009
@@ -44,5 +44,7 @@
   [u"a"]=>
   int(1)
 }
-array(0) {
+array(1) {
+  [u"a"]=>
+  int(1)
 }