diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-02-22 08:32:20 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2013-02-22 14:50:47 -0500 |
commit | dc4a787c8f52f9d6e46156953f8014a3353bcbc7 (patch) | |
tree | d8b8551f0f18968fb79f4201e79952023fa33e9e /drivers/net/wireless | |
parent | 48f4d91679c0e0b26747f8adbb18e5212eb6d621 (diff) |
brcmfmac: fix missing unlock on error in brcmf_notify_vif_event()
Add the missing unlock before return from function brcmf_notify_vif_event()
in the error handling case.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless')
-rw-r--r-- | drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c b/drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c index cecc3eff72e9..2af9c0f0798d 100644 --- a/drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c +++ b/drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c | |||
@@ -4615,8 +4615,10 @@ static s32 brcmf_notify_vif_event(struct brcmf_if *ifp, | |||
4615 | switch (ifevent->action) { | 4615 | switch (ifevent->action) { |
4616 | case BRCMF_E_IF_ADD: | 4616 | case BRCMF_E_IF_ADD: |
4617 | /* waiting process may have timed out */ | 4617 | /* waiting process may have timed out */ |
4618 | if (!cfg->vif_event.vif) | 4618 | if (!cfg->vif_event.vif) { |
4619 | mutex_unlock(&event->vif_event_lock); | ||
4619 | return -EBADF; | 4620 | return -EBADF; |
4621 | } | ||
4620 | 4622 | ||
4621 | ifp->vif = vif; | 4623 | ifp->vif = vif; |
4622 | vif->ifp = ifp; | 4624 | vif->ifp = ifp; |