Re: [v2] cw1200: Don't leak memory if krealloc failes

From: Kalle Valo
Date: Tue Nov 08 2016 - 20:37:43 EST


Johannes Thumshirn <jthumshirn@xxxxxxx> wrote:
> The call to krealloc() in wsm_buf_reserve() directly assigns the newly
> returned memory to buf->begin. This is all fine except when krealloc()
> failes we loose the ability to free the old memory pointed to by
> buf->begin. If we just create a temporary variable to assign memory to
> and assign the memory to it we can mitigate the memory leak.
>
> Signed-off-by: Johannes Thumshirn <jthumshirn@xxxxxxx>
> Cc: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>

Patch applied to wireless-drivers-next.git, thanks.

9afdd6128c39 cw1200: Don't leak memory if krealloc failes

--
https://patchwork.kernel.org/patch/9358185/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches