diff options
| author |   Steven Luong <sluong@cisco.com> | 2022-05-16 15:53:43 -0700 |
|---|---|---|
| committer | Steven Luong <sluong@cisco.com> | 2022-05-16 18:54:44 -0700 |
| commit | 59a08e65094db28884fc40e9562e303fde3b21d8 (patch) | |
| tree | 2742198f5e5364a55a16acdc4b792afddbc65a96 /src | |
| parent | 86c7856edae44483e7303fb4cf4efdec626716c8 (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
Diffstat (limited to 'src')
| -rw-r--r-- | src/plugins/acl/sess_mgmt_node.c | 1 |
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 */ |