diff options
| author |   Eyal Shapira <eyal@wizery.com> | 2012-03-19 12:06:27 +0200 |
|---|---|---|
| committer |   Luciano Coelho <coelho@ti.com> | 2012-04-10 12:42:49 +0300 |
| commit | 3eba4a0e6db9ca225bc0f3042d60fcfc86a30bc8 (patch) | |
| tree | 5c8fa54be9f81253a4063b339458cb9a5c5c02dd /drivers/net/wireless/wl12xx/main.c | |
| parent | wl12xx: free ap keys only in ap mode (diff) | |
| download | linux-dev-3eba4a0e6db9ca225bc0f3042d60fcfc86a30bc8.tar.xz linux-dev-3eba4a0e6db9ca225bc0f3042d60fcfc86a30bc8.zip | |
wl12xx: fix a memory leak of probereq template upon recovery
wlvif->probereq is zeroed when adding an interface but
the skb pointed to isn't freed when the interface is removed.
This would lead to a mem leak on every recovery.
Fix it by freeing the skb when removing the interface.
Signed-off-by: Eyal Shapira <eyal@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/wireless/wl12xx/main.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/wireless/wl12xx/main.c b/drivers/net/wireless/wl12xx/main.c index e21d21d7de8e..b560f2d2837b 100644 --- a/drivers/net/wireless/wl12xx/main.c +++ b/drivers/net/wireless/wl12xx/main.c @@ -2358,6 +2358,8 @@ deinit: wl1271_free_ap_keys(wl, wlvif); } + dev_kfree_skb(wlvif->probereq); + wlvif->probereq = NULL; wl12xx_tx_reset_wlvif(wl, wlvif); if (wl->last_wlvif == wlvif) wl->last_wlvif = NULL; |