[PATCH] Fix off by one error in channel scanning

Ryan Mallon ryan at bluewatersys.com
Thu Mar 6 22:48:21 EST 2008


Hey,

One of our customers noted that the Libertas driver doesn't seem to 
work with channels 4 and 9. I started digging through the code and 
found an off-by-one in wlan_scan_channel_list. We are using a 
reasonably old version of the Libertas source, so I'm not sure that
the following patch will apply cleanly, or is even relevant any more, 
but I though I would post it in case it still fixes a bug ;-).

Signed-off-by: Ryan Mallon <ryan at bluewatersys.com>

--- a/drivers/net/wireless/libertas/scan.c      2007-08-08 12:21:44.000000000 +1200
+++ b/drivers/net/wireless/libertas/scan.c      2008-03-07 16:41:56.000000000 +1300
@@ -676,7 +676,7 @@
        }

 done:
-       priv->adapter->last_scanned_channel = ptmpchan->channumber;
+       priv->adapter->last_scanned_channel = ptmpchan->channumber - 1;

        /* Tell userspace the scan table has been updated */
        memset(&wrqu, 0, sizeof(union iwreq_data));







More information about the libertas-dev mailing list