- Fixed bug #53366 (Reflection doesnt get dynamic property value from getProperty())

diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
index 6834955436042127a0d4eff84297222ce241e721..9f06deeabf91418d11e69fcf72ae0a161d0788f8 100644 (file)
--- a/ext/reflection/php_reflection.c
+++ b/ext/reflection/php_reflection.c
@@ -198,7 +198,8 @@ typedef enum {
        REF_TYPE_OTHER,      /* Must be 0 */
        REF_TYPE_FUNCTION,
        REF_TYPE_PARAMETER,
-       REF_TYPE_PROPERTY
+       REF_TYPE_PROPERTY,
+       REF_TYPE_DYNAMIC_PROPERTY
 } reflection_type_t;
 
 /* Struct for reflection objects */
@@ -284,6 +285,7 @@ static void reflection_free_objects_storage(void *object TSRMLS_DC) /* {{{ */
 {
        reflection_object *intern = (reflection_object *) object;
        parameter_reference *reference;
+       property_reference *prop_reference;
 
        if (intern->ptr) {
                switch (intern->ref_type) {
@@ -298,6 +300,11 @@ static void reflection_free_objects_storage(void *object TSRMLS_DC) /* {{{ */
                case REF_TYPE_PROPERTY:
                        efree(intern->ptr);
                        break;
+               case REF_TYPE_DYNAMIC_PROPERTY:
+                       prop_reference = (property_reference*)intern->ptr;
+                       efree(prop_reference->prop.name);
+                       efree(intern->ptr);
+                       break;
                case REF_TYPE_OTHER:
                        break;
                }
@@ -3825,13 +3832,15 @@ ZEND_METHOD(reflection_class, getProperty)
                if (zend_hash_exists(Z_OBJ_HT_P(intern->obj)->get_properties(intern->obj TSRMLS_CC), name, name_len+1)) {
                        zend_property_info property_info_tmp;
                        property_info_tmp.flags = ZEND_ACC_IMPLICIT_PUBLIC;
-                       property_info_tmp.name = name;
+                       property_info_tmp.name = estrndup(name, name_len);
                        property_info_tmp.name_length = name_len;
                        property_info_tmp.h = zend_get_hash_value(name, name_len+1);
                        property_info_tmp.doc_comment = NULL;
                        property_info_tmp.ce = ce;
 
                        reflection_property_factory(ce, &property_info_tmp, return_value TSRMLS_CC);
+                       intern = (reflection_object *) zend_object_store_get_object(return_value TSRMLS_CC);
+                       intern->ref_type = REF_TYPE_DYNAMIC_PROPERTY;
                        return;
                }
        }

diff --git a/ext/reflection/tests/bug53366.phpt b/ext/reflection/tests/bug53366.phpt
new file mode 100644 (file)
index 0000000..5fb119d
--- /dev/null
+++ b/ext/reflection/tests/bug53366.phpt
@@ -0,0 +1,25 @@
+--TEST--
+Bug #53366 (Reflection doesnt get dynamic property value from getProperty())
+--FILE--
+<?php
+
+class UserClass {
+}
+
+$myClass = new UserClass;
+$myClass->id = 1000;
+
+$reflect = new ReflectionObject($myClass);
+
+var_dump($reflect->getProperty('id'));
+var_dump($reflect->getProperty('id')->getValue($myClass));
+
+?>
+--EXPECTF--
+object(ReflectionProperty)#%d (2) {
+  ["name"]=>
+  string(2) "id"
+  ["class"]=>
+  string(9) "UserClass"
+}
+int(1000)