Thread (4 messages) flat view 4 messages, 4 authors, 2021-09-05

Re: [PATCH] iwlwifi: pnvm: Fix a memory leak in 'iwl_pnvm_get_from_fs()'

From: Coelho, Luciano <hidden>
Date: 2021-09-03 08:55:37
Also in: kernel-janitors, lkml, netdev

On Fri, 2021-09-03 at 11:19 +0300, Dan Carpenter wrote:
On Thu, Sep 02, 2021 at 10:38:11PM +0200, Christophe JAILLET wrote:
quoted
A firmware is requested but never released in this function. This leads to
a memory leak in the normal execution path.

Add the missing 'release_firmware()' call.
Also introduce a temp variable (new_len) in order to keep the value of
'pnvm->size' after the firmware has been released.

Fixes: cdda18fbbefa ("iwlwifi: pnvm: move file loading code to a separate function")
Signed-off-by: Christophe JAILLET <redacted>
Reviewed-by: Dan Carpenter <redacted>
Acked-by: Luca Coelho <redacted>

Kalle, can you please queue this for v5.15? I'll assign it to you. 
Thanks! 

--
Cheers,
Luca.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help