diff options
author | Johannes Berg <johannes@sipsolutions.net> | 2008-02-25 04:11:50 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2008-03-06 15:30:43 -0500 |
commit | ff59dc76e6e996092e4e11f4a6a370702428ead5 (patch) | |
tree | 438af93a51f1276171530c64a569376d0fba1ecf /net | |
parent | 2f5ce793c0817d8d38f1c7ad23945607d57e47d6 (diff) |
mac80211: add missing "break" statement in mesh code
This inserts a missing break statement which, if hit, would cause
the code to fall-through and unlock a spinlock twice. Noticed via
sparse's "lock count wrong in basic block" warning and careful
code inspection.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Cc: Luis Carlos Cobo <luisca@cozybit.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/mac80211/mesh_plink.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/mac80211/mesh_plink.c b/net/mac80211/mesh_plink.c index 0b0e8d7eb9c7..c2f4fe7a30b6 100644 --- a/net/mac80211/mesh_plink.c +++ b/net/mac80211/mesh_plink.c | |||
@@ -691,6 +691,7 @@ void mesh_rx_plink_frame(struct net_device *dev, struct ieee80211_mgmt *mgmt, | |||
691 | spin_unlock_bh(&sta->plink_lock); | 691 | spin_unlock_bh(&sta->plink_lock); |
692 | mesh_plink_frame_tx(dev, PLINK_CLOSE, sta->addr, llid, | 692 | mesh_plink_frame_tx(dev, PLINK_CLOSE, sta->addr, llid, |
693 | plid, reason); | 693 | plid, reason); |
694 | break; | ||
694 | case OPN_ACPT: | 695 | case OPN_ACPT: |
695 | if (del_timer(&sta->plink_timer)) | 696 | if (del_timer(&sta->plink_timer)) |
696 | sta_info_put(sta); | 697 | sta_info_put(sta); |