Fix memory leak in zend_abstract_interface/hook/hook.c

iamluc · iamluc · commit 98fc95f5143f · 2024-09-19T18:01:32.000+02:00
diff --git a/loader/valgrind.supp b/loader/valgrind.supp
@@ -1,14 +1,3 @@
-{
-   <leak from zai interceptor>
-   Memcheck:Leak
-   match-leak-kinds: possible
-   ...
-   fun:zai_hook_register_inheritor
-   fun:zai_hook_register_all_inheritors
-   fun:zai_hook_post_startup
-   fun:zai_interceptor_post_startup
-   ...
-}
 {
    <insert_a_suppression_name_here>
    Memcheck:Leak
diff --git a/zend_abstract_interface/hook/hook.c b/zend_abstract_interface/hook/hook.c
@@ -1226,7 +1226,10 @@ void zai_hook_rshutdown(void) {
 
 void zai_hook_gshutdown(void) { free(zai_hook_tls); }
 
-void zai_hook_mshutdown(void) { zend_hash_destroy(&zai_hook_static); } /* }}} */
+void zai_hook_mshutdown(void) {
+    zend_hash_destroy(&zai_hook_static);
+    zend_hash_destroy(&zai_hook_static_inheritors);
+} /* }}} */
 
 /* {{{ */
 zend_long zai_hook_install_resolved_generator(zend_function *function,
