On Fri, 2012-04-20 at 17:43 +0530, Bala Shanmugam wrote: > Legacy rates are not validated while configuring > tx rateset using iw. So below cmd is accepted by nl80211. > sudo iw wlan2 set bitrates legacy-2.4 1 2 3 > > Validate legacy rates and return > error if any rate in the rateset is not valid. > > Signed-off-by: Bala Shanmugam <bkamatch@xxxxxxxxxxxxxxxx> > --- > net/wireless/nl80211.c | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c > index bcf6f70..f822e7c 100644 > --- a/net/wireless/nl80211.c > +++ b/net/wireless/nl80211.c > @@ -5577,6 +5577,9 @@ static int nl80211_set_tx_bitrate_mask(struct sk_buff *skb, > sband, > nla_data(tb[NL80211_TXRATE_LEGACY]), > nla_len(tb[NL80211_TXRATE_LEGACY])); > + if (mask.control[band].legacy == 0 && > + nla_len(tb[NL80211_TXRATE_LEGACY])) > + return -EINVAL; Please use proper indentation. 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
