Skip to contentSkip to conversation

rte_sched_subport_free function does not free subport memory

dpdk-dev [email protected] 1 participant 1 message May 22, 2020

lore
Share & tools
lore
#0 lore raw

https://bugs.dpdk.org/show_bug.cgi?id=484

            Bug ID: 484
           Summary: rte_sched_subport_free function does not free subport
                    memory
           Product: DPDK
           Version: 19.11
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Severity: major
          Priority: Normal
         Component: other
          Assignee: [email protected]
          Reporter: [email protected]

Target Milestone: ---

Created attachment 105
--> https://bugs.dpdk.org/attachment.cgi?id=105&action=edit
Patch to fix rte_sched_subport_free leak

Hi.

It would seem that rte_sched_subport_free does not properly free allocated subport memory.

In function rte_sched_subport_free (lib/librte_sched/rte_sched.c, line 865), 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, and this is shown as memory leak.

Fix is quite simple, rte_bitmap_free should be replaced with rte_free, and argument should be changed to subport pointer.

Regards,

H.

Signature · 2 lines
You are receiving this mail because:
You are the assignee for the bug.