On Friday, October 14, 2011 12:45:32 AM Jouni Malinen wrote:
> On Thu, Oct 13, 2011 at 09:08:49PM +0200, Christian Lamparter wrote:
> > 802.11n-2009 extends the supported rates element with a
> > magic value which can be used to prevent legacy stations
> > from joining the BSS.
>
> Well, it can be used to try to make legacy stations not attempt
> connection, but no guarantees on them actually checking whether they
> support all the "basic rates".. For example, where is mac80211 (or
> wpa_supplicant) doing that check? ;-)
Actually, you have already implemented the check elsewhere :)
>
> > diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
> > @@ -1463,6 +1463,38 @@ ieee80211_rx_mgmt_disassoc(struct ieee80211_sub_if_data *sdata,
> > +static void ieee80211_get_rates(struct ieee80211_supported_band *sband,
>
> > + for (i = 0; i < supp_rates_len; i++) {
> > + int rate = (supp_rates[i] & 0x7f) * 5;
> > + bool is_basic = !!(supp_rates[i] & BSS_MEMBERSHIP_SELECTOR);
>
> This looks a bit odd since the BSS_MEMBERSHIP_SELECTOR is not exactly
> same as basic rate indicator even through they share the same bit. We
> used to have the magic 0x80 value here which could actually look less
> confusing than the mixing of basic and BSS membership terms.
>
> > + if (rate > 110)
> > + *have_higher_than_11mbit = true;
>
> While this is not really introduced by this patch, this looks quite
> bogus since the higher-than-11Mbps is then used to figure out whether
> this was a 802.11g network. That is not correct since a network with a
> single supported rate 6 Mbps should also get that behavior.. More robust
> mechanism would be to check for any OFDM rate being listed.
>
> > + ieee80211_get_rates(sband, elems.supp_rates, elems.supp_rates_len,
> > + &rates, &basic_rates, &have_higher_than_11mbit);
>
> > + ieee80211_get_rates(sband, elems.ext_supp_rates,
> > + elems.ext_supp_rates_len, &rates, &basic_rates,
> > + &have_higher_than_11mbit);
>
> Yay for getting rid of the duplicated loop :-).
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
