diff mbox

cfg80211: fix missing break in NL8211_CHAN_WIDTH_80P80 case

Message ID 1468781727-9906-1-git-send-email-colin.king@canonical.com (mailing list archive)
State Accepted
Delegated to: Johannes Berg
Headers show

Commit Message

Colin King July 17, 2016, 6:55 p.m. UTC 
From: Colin Ian King <colin.king@canonical.com>

The switch on chandef->width is missing a break on the
NL8211_CHAN_WIDTH_80P80 case; currently we get a WARN_ON when
center_freq2 is non-zero because of the missing break.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 net/wireless/chan.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Johannes Berg Aug. 2, 2016, 7:44 a.m. UTC | #1 
On Sun, 2016-07-17 at 19:55 +0100, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
> 
> The switch on chandef->width is missing a break on the
> NL8211_CHAN_WIDTH_80P80 case; currently we get a WARN_ON when
> center_freq2 is non-zero because of the missing break.
> 
Applied, thanks.

johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/net/wireless/chan.c b/net/wireless/chan.c
index b0e11b6..0f50622 100644
--- a/net/wireless/chan.c
+++ b/net/wireless/chan.c
@@ -513,6 +513,7 @@  static bool cfg80211_chandef_dfs_available(struct wiphy *wiphy,
 		r = cfg80211_get_chans_dfs_available(wiphy,
 						     chandef->center_freq2,
 						     width);
+		break;
 	default:
 		WARN_ON(chandef->center_freq2);
 		break;