Avoid freeing newthis

iluuu1994 · iluuu1994 · commit 165bbb96b049 · 2026-01-06T18:24:42.000+01:00
It will be freed by zend_vm_stack_free_args() automatically.
diff --git a/Zend/zend_closures.c b/Zend/zend_closures.c
@@ -86,9 +86,6 @@ static bool zend_valid_closure_binding(
 				zend_error(E_WARNING, "Cannot bind an instance to a static closure, this will be an error in PHP 9");
 				return false;
 			} else {
-				zend_object *obj = Z_OBJ_P(newthis);
-				ZVAL_UNDEF(newthis);
-				GC_DTOR(obj);
 				*newthis_ptr = NULL;
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -86,9 +86,6 @@ static bool zend_valid_closure_binding(`
`86`	`86`	`zend_error(E_WARNING, "Cannot bind an instance to a static closure, this will be an error in PHP 9");`
`87`	`87`	`return false;`
`88`	`88`	`} else {`
`89`		`- zend_object *obj = Z_OBJ_P(newthis);`
`90`		`- ZVAL_UNDEF(newthis);`
`91`		`- GC_DTOR(obj);`
`92`	`89`	`*newthis_ptr = NULL;`
`93`	`90`	`}`
`94`	`91`	`}`