diff options
author | Daniel Drake <dsd@laptop.org> | 2012-08-01 22:35:36 +0200 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2012-08-02 19:51:47 +0200 |
commit | 8c1057e43c99e393fe2e14cca879affcb516d36b (patch) | |
tree | dca3305b4012a79d01e44e48f760aec6bb336caf /drivers/net/wireless/libertas | |
parent | rt2x00 : fix rt3290 resuming failed. (diff) | |
download | linux-8c1057e43c99e393fe2e14cca879affcb516d36b.tar.xz linux-8c1057e43c99e393fe2e14cca879affcb516d36b.zip |
libertas: fix two memory leaks
The if_sdio_card structure was never being freed, and neither
was the command structure used for association.
Signed-off-by: Daniel Drake <dsd@laptop.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/libertas')
-rw-r--r-- | drivers/net/wireless/libertas/cfg.c | 1 | ||||
-rw-r--r-- | drivers/net/wireless/libertas/if_sdio.c | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/wireless/libertas/cfg.c b/drivers/net/wireless/libertas/cfg.c index eb5de800ed90..1c10b542ab23 100644 --- a/drivers/net/wireless/libertas/cfg.c +++ b/drivers/net/wireless/libertas/cfg.c @@ -1254,6 +1254,7 @@ static int lbs_associate(struct lbs_private *priv, netif_tx_wake_all_queues(priv->dev); } + kfree(cmd); done: lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret); return ret; diff --git a/drivers/net/wireless/libertas/if_sdio.c b/drivers/net/wireless/libertas/if_sdio.c index 76caebaa4397..e970897f6ab5 100644 --- a/drivers/net/wireless/libertas/if_sdio.c +++ b/drivers/net/wireless/libertas/if_sdio.c @@ -1314,6 +1314,7 @@ static void if_sdio_remove(struct sdio_func *func) kfree(packet); } + kfree(card); lbs_deb_leave(LBS_DEB_SDIO); } |