On Friday 13 June 2008 16:50:44 Vladimir Koutny wrote: > In IBSS mode prior to join/creation of new IBSS it is possible that > a frame from unknown station is received and an ibss_add_sta() is > called. This will cause a warning in rate_lowest_index() since the > list of supported rates of our station is not initialized yet. > > The fix is to add ibss stations with a rate we received that frame > at; this single-element set will be extended later based on beacon > data. Also there is no need to store stations from a foreign IBSS. makes sense. Acked-by: Bruno Randolf <br1@xxxxxxxxxxx> > - sta->supp_rates[local->hw.conf.channel->band] = > - sdata->u.sta.supp_rates_bits[local->hw.conf.channel->band]; > + if (supp_rates) > + sta->supp_rates[band] = supp_rates; > + else > + sta->supp_rates[band] = sdata->u.sta.supp_rates_bits[band]; i'm not sure about that last line. i know, it was like that before - but i think we shouldn't assume another station supports the same rates as we do in any case. we normally get the supported rates of another station thru the beacons we receive. or from received data packets (thanks to this patch). bruno -- 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
