DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [Bug 484] rte_sched_subport_free function does not free subport memory
@ 2020-05-22 13:26 bugzilla
  0 siblings, 0 replies; only message in thread
From: bugzilla @ 2020-05-22 13:26 UTC (permalink / raw)
  To: dev

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: dev@dpdk.org
          Reporter: hrvoje.habjanic@zg.ht.hr
  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.

-- 
You are receiving this mail because:
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-05-22 13:26 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-22 13:26 [dpdk-dev] [Bug 484] rte_sched_subport_free function does not free subport memory bugzilla

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git