diff options
author | Simon Wunderlich <simon.wunderlich@s2003.tu-chemnitz.de> | 2011-12-14 07:33:30 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2011-12-14 14:50:11 -0500 |
commit | cb71b8d80334add8991862f9fd36dc50874944ce (patch) | |
tree | d6a730afd5e4dc6743acd0bbc4166a66621f0ced /net/mac80211/ibss.c | |
parent | 38c9d6641ff0664911aebe4ba67124f28169a972 (diff) |
mac80211: free skb on error path of ieee80211_ibss_join()
Our new return also created a memleak. The skb should be freed before
returning an error.
Signed-off-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/ibss.c')
-rw-r--r-- | net/mac80211/ibss.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/mac80211/ibss.c b/net/mac80211/ibss.c index 0fc9752facea..fe0e91e0b8bc 100644 --- a/net/mac80211/ibss.c +++ b/net/mac80211/ibss.c | |||
@@ -995,6 +995,7 @@ int ieee80211_ibss_join(struct ieee80211_sub_if_data *sdata, | |||
995 | if (!ieee80211_set_channel_type(sdata->local, sdata, | 995 | if (!ieee80211_set_channel_type(sdata->local, sdata, |
996 | params->channel_type)) { | 996 | params->channel_type)) { |
997 | mutex_unlock(&sdata->u.ibss.mtx); | 997 | mutex_unlock(&sdata->u.ibss.mtx); |
998 | kfree_skb(skb); | ||
998 | return -EINVAL; | 999 | return -EINVAL; |
999 | } | 1000 | } |
1000 | } | 1001 | } |