From fbd8981cddf9ea558d5480cc3b02d3334482210f Mon Sep 17 00:00:00 2001 From: Hrvoje Habjanic Date: Tue, 26 May 2020 19:24:55 +0200 Subject: [PATCH] sched: fix subport freeing In function rte_sched_subport_free, there is code to free all allocated stuff related to scheduler subport. First there are some checks, and in the end, rte_bitmap_free is called. Now, rte_bitmap_free is a dummy function, and it just checks if provided pointer to bitmap is valid or not. So, actual memory for subport is not freed. This patch fixes this by removing call to rte_bitmap_free, and instead calling rte_free. Fixes: d9213b829a31 ("sched: remove pipe params config from port level") Cc: stable@dpdk.org Signed-off-by: Hrvoje Habjanic Acked-by: Stephen Hemminger Acked-by: Jasvinder Singh --- lib/librte_sched/rte_sched.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_sched/rte_sched.c b/lib/librte_sched/rte_sched.c index c0983ddda4..f15a3b515d 100644 --- a/lib/librte_sched/rte_sched.c +++ b/lib/librte_sched/rte_sched.c @@ -888,7 +888,7 @@ rte_sched_subport_free(struct rte_sched_port *port, } } - rte_bitmap_free(subport->bmp); + rte_free(subport); } void -- 2.20.1