On Mon, 2015-03-23 at 14:07 -0500, Larry Finger wrote: > In this routine, kzalloc allocates a memory block. This allocation is > freed in the error paths, but not in the normal exit, thus the allocation > is leaked. > > The kmemleak facility was used to find the leak. >
Picked up - thanks. > Signed-off-by: Larry Finger <[email protected]> > Cc: Johannes Berg <[email protected]> > Cc: Emmanuel Grumbach <[email protected]> > Cc: Intel Linux Wireless <[email protected]> > --- > drivers/net/wireless/iwlwifi/iwl-drv.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/wireless/iwlwifi/iwl-drv.c > b/drivers/net/wireless/iwlwifi/iwl-drv.c > index 66ca000..aefdd9b 100644 > --- a/drivers/net/wireless/iwlwifi/iwl-drv.c > +++ b/drivers/net/wireless/iwlwifi/iwl-drv.c > @@ -1319,6 +1319,7 @@ static void iwl_req_fw_callback(const struct firmware > *ucode_raw, void *context) > op->name, err); > #endif > } > + kfree(pieces); > return; > > try_again:
