zebra: ipset and ipset entry deletion remove entry from hash list too

This commit is a fix that removes the structure from the hash list,
instead of just removing that structure.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

diff --git a/zebra/zebra_pbr.c b/zebra/zebra_pbr.c
index 93c523bf5026300b22d83039492bc9e90ce4bdde..ac0192f2dedeb0443de90d2a63bf32fa12af3dab 100644 (file)
--- a/zebra/zebra_pbr.c
+++ b/zebra/zebra_pbr.c
@@ -360,9 +360,10 @@ void zebra_pbr_destroy_ipset(struct zebra_ns *zns,
         * - Netlink destroy from kernel
         * - ?? destroy ipset entries before
         */
-       if (lookup)
+       if (lookup) {
+               hash_release(zns->ipset_hash, lookup);
                XFREE(MTYPE_TMP, lookup);
-       else
+       } else
                zlog_warn("%s: IPSet Entry being deleted we know nothing about",
                          __PRETTY_FUNCTION__);
 }
@@ -437,9 +438,10 @@ void zebra_pbr_del_ipset_entry(struct zebra_ns *zns,
         * - detach from ipset list
         * - ?? if no more entres, delete ipset
         */
-       if (lookup)
+       if (lookup) {
+               hash_release(zns->ipset_entry_hash, lookup);
                XFREE(MTYPE_TMP, lookup);
-       else
+       } else
                zlog_warn("%s: IPSet being deleted we know nothing about",
                          __PRETTY_FUNCTION__);
 }