Thread (5 messages) 5 messages, 3 authors, 2017-02-25

[PATCH] staging: bcm2835: Fix a memory leak in error handling path

From: Dan Carpenter <hidden>
Date: 2017-02-25 04:13:37
Also in: kernel-janitors, linux-media, lkml

On Fri, Feb 24, 2017 at 10:38:38PM +0100, Stefan Wahren wrote:
quoted
Dan Carpenter [off-list ref] hat am 24. Februar 2017 um 20:57 geschrieben:


On Fri, Feb 24, 2017 at 01:37:30PM +0100, Stefan Wahren wrote:
quoted
Hi Christophe,

Am 19.02.2017 um 11:34 schrieb Christophe JAILLET:
quoted
If 'kzalloc()' fails, we should release resources allocated so far, just as
done in all other cases in this function.

Signed-off-by: Christophe JAILLET <redacted>
---
Not sure that the error handling path is correct.
Is 'gdev[0]' freed? Should it be?
Yes, but I already sent a patch to fix this and your leak as well and
Greg merged it.
My leak? I'm confused.
The one you're fixing I mean.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help