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

From: Kalle Valo <[email-masked]>
Date: Wed, 9 Nov 2016 01:37:00 +0000 (UTC)

Johannes Thumshirn <[email-masked]> 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 <[email-masked]>
> Cc: Johannes Berg <[email-masked]>

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