diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-12-08 01:49:03 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2011-12-13 15:30:25 -0500 |
commit | fb03c5eb8c0bbf4561cb5aa72e0a9546e9574661 (patch) | |
tree | ad6dd8a8329c282b780a1f74ca27f100061aa276 /net/mac80211 | |
parent | cc78d6b16a6853a3f6c014a6173df41d80f65a35 (diff) |
mac80211: unlock on error path in ieee80211_ibss_join()
We recently introduced a new return here but it needs an unlock first.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211')
-rw-r--r-- | net/mac80211/ibss.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/mac80211/ibss.c b/net/mac80211/ibss.c index 3f830ac159e5..0fc9752facea 100644 --- a/net/mac80211/ibss.c +++ b/net/mac80211/ibss.c | |||
@@ -993,8 +993,10 @@ int ieee80211_ibss_join(struct ieee80211_sub_if_data *sdata, | |||
993 | if (params->channel_fixed) { | 993 | if (params->channel_fixed) { |
994 | sdata->local->oper_channel = params->channel; | 994 | sdata->local->oper_channel = params->channel; |
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 | return -EINVAL; | 998 | return -EINVAL; |
999 | } | ||
998 | } | 1000 | } |
999 | 1001 | ||
1000 | if (params->ie) { | 1002 | if (params->ie) { |