mac80211: remove direct probe step before authentication
authorJohannes Berg <johannes.berg@intel.com>
Sat, 15 Aug 2015 19:39:54 +0000 (22:39 +0300)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 22 Sep 2015 13:21:23 +0000 (15:21 +0200)
commit46cad4b7a131a215159d889fa88d0dc71d581908
treec54192f676ee8c081722e07539625a155ddea585
parente3abc8ff0fc18b3925fd5d5c5fbd1613856f4e7c
mac80211: remove direct probe step before authentication

The direct probe step before authentication was done mostly for
two reasons:
 1) the BSS data could be stale
 2) the beacon might not have included all IEs

The concern (1) doesn't really seem to be relevant any more as
we time out BSS information after about 30 seconds, and in fact
the original patch only did the direct probe if the data was
older than the BSS timeout to begin with. This condition got
(likely inadvertedly) removed later though.

Analysing this in more detail shows that since we mostly use
data from the association response, the only real reason for
needing the probe response was that the code validates the WMM
parameters, and those are optional in beacons. As the previous
patches removed that behaviour, we can now remove the direct
probe step entirely.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/mlme.c