On Fri, 2013-03-22 at 14:27 +0100, Karl Beldan wrote:
> > > And, oops typo, I meant :
> > >
> > > + !(info->flags & IEEE80211_TX_CTL_INJECTED && info->control.rates[0] != -1))
> > >
> > Well actually this won't work anyway because control.rates[] isn't
> > initialized to -1.
> >
> Of course we would set info->control.rates[0] to -1 by default in
> ieee80211_parse_tx_radiotap and set it to something else if we parse a
> IEEE80211_RADIOTAP_{RATE,MCS} (i.e what I was referring to in 1b).
That still wouldn't work though, there are other paths leading to point
you're looking at. You'd have to set it always, which would probably
work but is tricky to ensure at the right points.
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
