Fixed bug #48781 (Cyclical garbage collector memory leak)

diff --git a/NEWS b/NEWS
index 7ac9fb380e4a9e37364895c0b10300d876740d4c..0eb325bb2e6814607c113d59e491882a53675180 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -75,6 +75,7 @@ PHP                                                                        NEWS
   (yoarvi@gmail.com, Derick)
 - Fixed bug #48983 (DomDocument : saveHTMLFile wrong charset). (Rob)
 - Fixed bug #48902 (Timezone database fallback map is outdated). (Derick)
+- Fixed bug #48781 (Cyclical garbage collector memory leak). (Dmitry)
 - Fixed bug #46111 (Some timezone identifiers can not be parsed). (Derick)
 - Fixed bug #35673 (formatOutput does not work with saveHTML). (Rob)
 - Implement feature request #35638 (Adding udate to imap_fetch_overview results).

diff --git a/Zend/zend_execute.c b/Zend/zend_execute.c
index ae54c5fa1bfce719b1988a2d752d0c1e1a554ce3..5ddb03a8c35e142375dfceb80884bb9b0365a584 100644 (file)
--- a/Zend/zend_execute.c
+++ b/Zend/zend_execute.c
@@ -708,6 +708,7 @@ static inline zval* zend_assign_to_variable(zval **variable_ptr_ptr, zval *value
                                return variable_ptr;
                        }
                } else { /* we need to split */
+                       GC_ZVAL_CHECK_POSSIBLE_ROOT(*variable_ptr_ptr);
                        if (!is_tmp_var) {
                                if (PZVAL_IS_REF(value) && Z_REFCOUNT_P(value) > 0) {
                                        ALLOC_ZVAL(variable_ptr);