[PATCH] b43: fix memory leak

Rafał Miłecki zajec5 at gmail.com
Thu Feb 18 05:40:20 PST 2016


On 18 February 2016 at 13:34, Sudip Mukherjee
<sudipm.mukherjee at gmail.com> wrote:
> From: Sudip Mukherjee <sudip at vectorindia.org>
>
> On error we jumped to the label bcma_out and returned the error code but
> we missed freeing dev.

What if b43_one_core_attach fails? Won't we miss kfree then as well?
Shouldn't you simply improve error path and maybe use an additional
label? Also may we be missing a proper free in b43_bcma_remove as
well?

Your patch subject could also be a bit more descriptive (e.g. what
memory leak exactly is it).



More information about the b43-dev mailing list