On Wed, 2010-11-17 at 00:34 +0100, Felix Fietkau wrote: > On 2010-11-16 11:27 PM, Johannes Berg wrote: > > On Thu, 2010-11-11 at 15:07 +0100, Felix Fietkau wrote: > > > >> + if (info->attrs[NL80211_ATTR_MCAST_RATE]) > >> + ibss.mcast_rate = > >> + nla_get_u32(info->attrs[NL80211_ATTR_MCAST_RATE]); > > > > That should really check that the rate is supported, and maybe that it's > > a basic rate. > But what about band changes then? Yeah but what about them? You don't handle them anyway -- should maybe do a per-band setting? Basically right now you end up ignoring the rate if it's not available. johannes -- 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
