Staging: rtl8192e: off by one in rtl8192_get_channel_map()
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 21 Sep 2011 07:17:04 +0000 (10:17 +0300)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 27 Sep 2011 00:26:03 +0000 (17:26 -0700)
COUNTRY_CODE_MAX is not a valid country code.  We're off by one here.
This gets passed to Dot11d_Channelmap() where it's used as an offset
into the ChannelPlan[] array.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/rtl8192e/rtl_core.c

index 4c5d7abd9be0193b636f3af5c6aac7ce6f264289..5ad96649f4076586d8ebec77c97bd675a01bd29c 100644 (file)
@@ -1283,7 +1283,7 @@ static short rtl8192_get_channel_map(struct net_device *dev)
                return -1;
        }
 
-       if (priv->ChannelPlan > COUNTRY_CODE_MAX) {
+       if (priv->ChannelPlan >= COUNTRY_CODE_MAX) {
                printk(KERN_INFO "rtl819x_init:Error channel plan! Set to "
                       "default.\n");
                priv->ChannelPlan = COUNTRY_CODE_FCC;