Re: [dpdk-dev] [PATCH] eal: fix memory leak when saving arguments

2021-09-16 Thread David Marchand
On Wed, Sep 1, 2021 at 11:59 AM Bruce Richardson wrote: > > On Wed, Sep 01, 2021 at 09:38:47AM +, Conor Walsh wrote: > > This patch fixes a memleak which was reported in Bugzilla within the > > eal_save_args function. This was caused by the function mistakenly > > adding -- to the eal args ins

Re: [dpdk-dev] [PATCH] eal: fix memory leak when saving arguments

2021-09-01 Thread Bruce Richardson
On Wed, Sep 01, 2021 at 09:38:47AM +, Conor Walsh wrote: > This patch fixes a memleak which was reported in Bugzilla within the > eal_save_args function. This was caused by the function mistakenly > adding -- to the eal args instead of breaking beforehand. > > Bugzilla ID: 722 > Fixes: 293c53d

[dpdk-dev] [PATCH] eal: fix memory leak when saving arguments

2021-09-01 Thread Conor Walsh
This patch fixes a memleak which was reported in Bugzilla within the eal_save_args function. This was caused by the function mistakenly adding -- to the eal args instead of breaking beforehand. Bugzilla ID: 722 Fixes: 293c53d8b23c ("eal: add telemetry callbacks") Reported-by: Zhihong Peng Signed

Re: [dpdk-dev] [PATCH] eal: fix memory leak in uevent parse and process

2020-10-20 Thread David Marchand
On Sat, May 23, 2020 at 12:52 PM wangyunjian wrote: > > From: Yunjian Wang > > When the memory for uevent.devname is allocated in dev_uev_parse(). It > is not freed when parse the subsystem layer fails in dev_uev_parse(). > And Before return, it is also not freed in dev_uev_handler(). These > cau

Re: [dpdk-dev] [PATCH] eal: fix memory leak in uevent parse and process

2020-07-17 Thread wangyunjian
Is there any ideas on this bug? Ping for review, please. Yunjian > -Original Message- > From: wangyunjian > Sent: Saturday, May 23, 2020 6:52 PM > To: dev@dpdk.org > Cc: Lilijun (Jerry) ; xudingke > ; wangyunjian ; > sta...@dpdk.org > Subject: [dpdk-dev] [PATCH]

[dpdk-dev] [PATCH] eal: fix memory leak in uevent parse and process

2020-05-23 Thread wangyunjian
From: Yunjian Wang When the memory for uevent.devname is allocated in dev_uev_parse(). It is not freed when parse the subsystem layer fails in dev_uev_parse(). And Before return, it is also not freed in dev_uev_handler(). These cause a memory leak. Fixes: 0d0f478d0483 ("eal/linux: add uevent par

Re: [dpdk-dev] [PATCH] eal: fix memory leak

2018-05-02 Thread Thomas Monjalon
02/05/2018 13:09, Olivier Matz: > On Wed, May 02, 2018 at 10:26:32AM +, Jianfeng Tan wrote: > > params is not freed if pthread_create() fails. The fix is > > straight-forward. > > > > Fixes: 3d09a6e26d8b ("eal: fix threads block on barrier") > > > > Reported-by: Olivier Matz > > Signed-off-b

Re: [dpdk-dev] [PATCH] eal: fix memory leak

2018-05-02 Thread Olivier Matz
On Wed, May 02, 2018 at 10:26:32AM +, Jianfeng Tan wrote: > params is not freed if pthread_create() fails. The fix is > straight-forward. > > Fixes: 3d09a6e26d8b ("eal: fix threads block on barrier") > > Reported-by: Olivier Matz > Signed-off-by: Jianfeng Tan Reviewed-by: Olivier Matz

[dpdk-dev] [PATCH] eal: fix memory leak

2018-05-02 Thread Jianfeng Tan
params is not freed if pthread_create() fails. The fix is straight-forward. Fixes: 3d09a6e26d8b ("eal: fix threads block on barrier") Reported-by: Olivier Matz Signed-off-by: Jianfeng Tan --- lib/librte_eal/common/eal_common_thread.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) di

Re: [dpdk-dev] [PATCH] eal: fix memory leak in memzone when no room in config

2018-01-12 Thread Thomas Monjalon
21/12/2017 19:07, Anatoly Burakov: > We check if there's space in config after we allocated the memzone, > but if there isn't, we never free it back. This patch adds memzone > free if there's no room in memzone config. > > Fixes: ff909fe21f0a ("mem: introduce memzone freeing") > Cc: sergio.gonzale

[dpdk-dev] [PATCH] eal: fix memory leak in memzone when no room in config

2017-12-21 Thread Anatoly Burakov
We check if there's space in config after we allocated the memzone, but if there isn't, we never free it back. This patch adds memzone free if there's no room in memzone config. Fixes: ff909fe21f0a ("mem: introduce memzone freeing") Cc: sergio.gonzalez.mon...@intel.com Cc: sta...@dpdk.org Signed-o