Re: [PATCH 01/20] baseband/acc100: fix a memory leak in acc100 queue setup

2022-06-24 Thread David Marchand
On Tue, Feb 22, 2022 at 7:18 PM Weiguo Li wrote: > > We allocated memory for 'q', we don't free it when null check for 'd' fails > and it will lead to memory leak. > We can move null check for 'd' ahead of the memory allocation to fix it. > > Fixes: 060e76729302 ("baseband/acc100: add queue config

RE: [PATCH 01/20] baseband/acc100: fix a memory leak in acc100 queue setup

2022-02-23 Thread Chautru, Nicolas
> From: Weiguo Li > > We allocated memory for 'q', we don't free it when null check for 'd' fails > and > it will lead to memory leak. > We can move null check for 'd' ahead of the memory allocation to fix it. > > Fixes: 060e76729302 ("baseband/acc100: add queue configuration") > > Signed-off-

[PATCH 01/20] baseband/acc100: fix a memory leak in acc100 queue setup

2022-02-22 Thread Weiguo Li
We allocated memory for 'q', we don't free it when null check for 'd' fails and it will lead to memory leak. We can move null check for 'd' ahead of the memory allocation to fix it. Fixes: 060e76729302 ("baseband/acc100: add queue configuration") Signed-off-by: Weiguo Li --- drivers/baseband/ac