From e266c44345f117773b85008664601107d679eee4 Mon Sep 17 00:00:00 2001 From: Philippe Guibert Date: Wed, 6 Dec 2023 18:34:02 +0100 Subject: [PATCH] lib: fix memory leak when disabling nexthop_hold contexts A memory leak is detected when stopping the sharpd daemon with a nexthop group configuration that includes nexthops. The nexthop_hold structure and its attributes are not freed. Fix it by adding the missing free function. Fixes: 98cbbaea91f6 ("lib: Handle if up/down and vrf enable/disable events") Signed-off-by: Philippe Guibert --- lib/nexthop_group.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/nexthop_group.c b/lib/nexthop_group.c index d1fd526d6e..3f408e0a71 100644 --- a/lib/nexthop_group.c +++ b/lib/nexthop_group.c @@ -1260,6 +1260,8 @@ void nexthop_group_disable_vrf(struct vrf *vrf) nexthop_free(nh); list_delete_node(nhgc->nhg_list, node); + + nhgl_delete(nhh); } } } -- 2.39.5