funnel: Fix memleaks in funnel_remove_menu

The loop to remove all matching callbacks was skipping every second
entry which would give some leaks when reloading Lua plugins.

Add funnel_cleanup() to be called in epan_cleanup() at shutdown
to remove all allocated menu entries.

Change-Id: I3a50ba2070c8675fee1385f25e9e109db57c2dc5
Reviewed-on: https://code.wireshark.org/review/31769
Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org>
Tested-by: Petri Dish Buildbot
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>

1 files changed, 6 insertions, 1 deletions

diff --git a/epan/funnel.c b/epan/funnel.c
index dd96db79cd..a021106612 100644
--- a/epan/funnel.c
+++ b/epan/funnel.c
@@ -62,7 +62,7 @@ static void funnel_remove_menu (funnel_menu_t ** menu_list, funnel_menu_t *menu)
             if (p) {
                 m = p->next;
             } else {
-                m = *menu_list ? (*menu_list)->next : NULL;
+                m = *menu_list;
             }
         } else {
             p = m;
@@ -139,6 +139,11 @@ void funnel_reload_menus(funnel_deregistration_cb_t d_cb,
     funnel_clear_menu(&added_menus);
 }
 
+void funnel_cleanup(void)
+{
+    funnel_clear_menu(&registered_menus);
+}
+
 /*
  * Editor modelines  -  http://www.wireshark.org/tools/modelines.html
  *