]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
mac80211: make ieee802_11_parse_elems return void
authorJohn W. Linville <linville@tuxdriver.com>
Fri, 26 Oct 2007 21:04:35 +0000 (17:04 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 16 Nov 2007 16:27:39 +0000 (08:27 -0800)
commitef0811638110d6309c4b498a33a9815d694fb6bc
treeb8370f3c01345ead30d1047b58608bac3db0e60d
parentf08fdbd7f5f6c0cafce7cdcda8df4e605d065cd6
mac80211: make ieee802_11_parse_elems return void

patch 67a4cce4a89718d252b61aaf58882c69c0e2f6e3 in mainline.

Some APs send management frames with junk padding after the last IE.
We already account for a similar problem with some Apple Airport
devices, but at least one device is known to send more than a single
extra byte.  The device in question is the Draytek Vigor2900:

http://www.draytek.com.au/products/Vigor2900.php

The junk in question looks like an IE that runs off the end of the
frame.  This cause us to return ParseFailed.  Since the frame in
question is an association response, this causes us to fail to associate
with this AP.

The return code from ieee802_11_parse_elems is superfluous.
All callers still check for the presence of the specific IEs that
interest them anyway.  So, remove the return code so the parse never
"fails".

Acked-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
net/mac80211/ieee80211_sta.c