]> git.baikalelectronics.ru Git - kernel.git/commit
rndis_wlan: fix matching bssid check in rndis_check_bssid_list()
authorJussi Kivilinna <jussi.kivilinna@mbnet.fi>
Sat, 9 Jun 2012 15:45:20 +0000 (18:45 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 11 Jun 2012 18:29:15 +0000 (14:29 -0400)
commitc2ed4d6255964f51c35221d5d25c6e594daaadac
tree0c7b860942f9b0ad4da5b7260595f7327bd3aff7
parent4d753b4b158201a5be9adc72a3855bced3ed85bb
rndis_wlan: fix matching bssid check in rndis_check_bssid_list()

rndis_check_bssid_list() originally tried to check if bssid->mac and
match_bssid are equal using compare_ether_addr() when it should use
!compare_ether_addr(). This check was added by commit
c20591b74b95164f810670085985825e865de186 as part of workaround for
hardware issue.

Commit 29178e2a80df24b6ca48e61f4a361a9f7e72fbbd that replaced
compare_ether_addr with ether_addr_equal relieved that this compare
to be inverse of what it should be.

Compare was added as response to hardware bug, where bssid-list does
not contain BSSID and other information of currently connected AP
(spec insists that device must provide this information in the list
when connected). Lack bssid-data on current connection then causes
WARN_ON somewhere in cfg80211. Workaround was to check if bssid-list
returns current bssid and if it does not, manually construct bssid
information in other ways. And this workaround worked, with inverse
check. Which must mean that when hardware is experiencing the problem,
it's actually returning empty bssid-list and this check didn't make
any difference for workaround.

However inverse check causes workaround be activated when bssid-list
returns only entry, currently connected BSSID. That does not cause
problems in itself, just slightly more inaccurate information in
scan-list.

Cc: Joe Perches <joe@perches.com>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Jussi Kivilinna <jussi.kivilinna@mbnet.fi>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/rndis_wlan.c