Clarify ZEND_CATCH code

nikic · nikic · commit 81c824f41f5e · 2020-05-26T11:34:07.000+02:00
UNDEF the opcode result instead of addref'ing the exception.
diff --git a/Zend/zend_vm_def.h b/Zend/zend_vm_def.h
@@ -4493,7 +4493,7 @@ ZEND_VM_HANDLER(107, ZEND_CATCH, CONST, JMP_ADDR, LAST_CATCH|CACHE_SLOT)
 	zval_ptr_dtor(ex);
 	ZVAL_OBJ(ex, EG(exception));
 	if (UNEXPECTED(EG(exception) != exception)) {
-		GC_ADDREF(EG(exception));
+		ZVAL_UNDEF(ex);
 		HANDLE_EXCEPTION();
 	} else {
 		EG(exception) = NULL;
diff --git a/Zend/zend_vm_execute.h b/Zend/zend_vm_execute.h
@@ -3709,7 +3709,7 @@ static ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_CATCH_SPEC_CONST_HANDLER(ZEND_
 	zval_ptr_dtor(ex);
 	ZVAL_OBJ(ex, EG(exception));
 	if (UNEXPECTED(EG(exception) != exception)) {
-		GC_ADDREF(EG(exception));
+		ZVAL_UNDEF(ex);
 		HANDLE_EXCEPTION();
 	} else {
 		EG(exception) = NULL;
