cfg80211: off by one in nl80211_trigger_scan()
authorDan Carpenter <error27@gmail.com>
Fri, 29 Jul 2011 08:52:18 +0000 (11:52 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 1 Aug 2011 17:46:46 +0000 (13:46 -0400)
The test is off by one so we'd read past the end of the
wiphy->bands[] array on the next line.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/wireless/nl80211.c

index 28d2aa1..e83e7fe 100644 (file)
@@ -3464,7 +3464,7 @@ static int nl80211_trigger_scan(struct sk_buff *skb, struct genl_info *info)
                                    tmp) {
                        enum ieee80211_band band = nla_type(attr);
 
-                       if (band < 0 || band > IEEE80211_NUM_BANDS) {
+                       if (band < 0 || band >= IEEE80211_NUM_BANDS) {
                                err = -EINVAL;
                                goto out_free;
                        }