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