op_array->function_name, op_array->doc_comments and op_array->filename may be kept...

diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
index 9aea15c882bb4e272e63ba697cdfc2c3aaef3783..d080b8146766fafb16e7797292b2466fbf062851 100644 (file)
--- a/ext/reflection/php_reflection.c
+++ b/ext/reflection/php_reflection.c
@@ -1800,7 +1800,8 @@ ZEND_METHOD(reflection_function, getFileName)
        }
        GET_REFLECTION_OBJECT_PTR(fptr);
        if (fptr->type == ZEND_USER_FUNCTION) {
-               RETURN_STR(STR_COPY(fptr->op_array.filename));
+// TODO: we have to duplicate it, becaise it may be in opcache SHM ???
+               RETURN_STR(STR_DUP(fptr->op_array.filename, 0));
        }
        RETURN_FALSE;
 }
@@ -1854,7 +1855,8 @@ ZEND_METHOD(reflection_function, getDocComment)
        }
        GET_REFLECTION_OBJECT_PTR(fptr);
        if (fptr->type == ZEND_USER_FUNCTION && fptr->op_array.doc_comment) {
-               RETURN_STR(STR_COPY(fptr->op_array.doc_comment));
+// TODO: we have to duplicate it, becaise it may be stored in opcache SHM ???
+               RETURN_STR(STR_DUP(fptr->op_array.doc_comment, 0));
        }
        RETURN_FALSE;
 }
@@ -3566,7 +3568,8 @@ ZEND_METHOD(reflection_class, getFileName)
        }
        GET_REFLECTION_OBJECT_PTR(ce);
        if (ce->type == ZEND_USER_CLASS) {
-               RETURN_STR(STR_COPY(ce->info.user.filename));
+// TODO: we have to duplicate it, becaise it may be stored in opcache SHM ???
+               RETURN_STR(STR_DUP(ce->info.user.filename, 0));
        }
        RETURN_FALSE;
 }