aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteven Luong <sluong@cisco.com>2022-05-16 15:53:43 -0700
committerSteven Luong <sluong@cisco.com>2022-05-16 18:54:44 -0700
commit59a08e65094db28884fc40e9562e303fde3b21d8 (patch)
tree2742198f5e5364a55a16acdc4b792afddbc65a96
parent86c7856edae44483e7303fb4cf4efdec626716c8 (diff)
acl: memory leak in acl_fa_session_cleaner_process
The statement pw0->pending_clear_sw_if_index_bitmap = clib_bitmap_dup (pw0->serviced_sw_if_index_bitmap); will cause pw0->pending_clear_sw_if_index_bitmap's previous vector to be gone. Need to free it prior to calling clib_bitmap_dup() Type: fix Signed-off-by: Steven Luong <sluong@cisco.com> Change-Id: I20de780e73daea7be17efa0bf660af2592cd4680
-rw-r--r--src/plugins/acl/sess_mgmt_node.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/plugins/acl/sess_mgmt_node.c b/src/plugins/acl/sess_mgmt_node.c
index d0ec5936896..eb73fd0bb45 100644
--- a/src/plugins/acl/sess_mgmt_node.c
+++ b/src/plugins/acl/sess_mgmt_node.c
@@ -723,6 +723,7 @@ acl_fa_session_cleaner_process (vlib_main_t * vm, vlib_node_runtime_t * rt,
}
else
{
+ clib_bitmap_free (pw0->pending_clear_sw_if_index_bitmap);
if (clear_all)
{
/* if we need to clear all, then just clear the interfaces that we are servicing */