Fix unserialize key leak

iluuu1994 · iluuu1994 · commit 67f6e6d558a6 · 2024-07-11T14:11:10.000+02:00
diff --git a/ext/standard/var_unserializer.re b/ext/standard/var_unserializer.re
@@ -563,6 +563,7 @@ static int is_property_visibility_changed(zend_class_entry *ce, zval *key)
 				php_error_docref(NULL, E_WARNING,
 					"Cannot unserialize value for hooked property %s::$%s",
 					ZSTR_VAL(existing_propinfo->ce->name), Z_STRVAL_P(key));
+				zval_ptr_dtor_str(key);
 				return -1;
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -563,6 +563,7 @@ static int is_property_visibility_changed(zend_class_entry ce, zval key)`
`563`	`563`	`php_error_docref(NULL, E_WARNING,`
`564`	`564`	`"Cannot unserialize value for hooked property %s::$%s",`
`565`	`565`	`ZSTR_VAL(existing_propinfo->ce->name), Z_STRVAL_P(key));`
	`566`	`+ zval_ptr_dtor_str(key);`
`566`	`567`	`return -1;`
`567`	`568`	`}`
`568`	`569`	`}`