Do not duplicate the attributes table in zend_get_call_trampoline_func()

Author: TimWolla

Zend/zend_object_handlers.c

@@ -1619,7 +1619,8 @@ ZEND_API zend_function *zend_get_call_trampoline_func(const zend_class_entry *ce
 		| ZEND_ACC_VARIADIC
 		| (fbc->common.fn_flags & (ZEND_ACC_RETURN_REFERENCE|ZEND_ACC_DEPRECATED));
 	if (fbc->common.attributes) {
-		func->attributes = zend_array_dup(fbc->common.attributes);
+		func->attributes = fbc->common.attributes;
+		GC_TRY_ADDREF(func->attributes);
 	} else {
 		func->attributes = NULL;
 	}

Zend/zend_object_handlers.h

@@ -339,11 +339,12 @@ ZEND_API bool ZEND_FASTCALL zend_asymmetric_property_has_set_access(const zend_p
 } while (0)
 
 #define zend_free_trampoline(func) do { \
-		if ((func)->common.attributes) { \
-			zend_array_destroy((func)->common.attributes); \
-			(func)->common.attributes = NULL; \
+		HashTable *attributes = (func)->common.attributes; \
+		if (attributes && !(GC_FLAGS(attributes) & GC_IMMUTABLE) && !GC_DELREF(attributes)) { \
+			zend_array_destroy(attributes); \
 		} \
 		if ((func) == &EG(trampoline)) { \
+			EG(trampoline).common.attributes = NULL; \
 			EG(trampoline).common.function_name = NULL; \
 		} else { \
 			efree(func); \