On Wed, Apr 17, 2013 at 05:39:23PM +0200, Felix Fietkau wrote: > On 2013-04-17 5:23 PM, Karl Beldan wrote: > > On Tue, Apr 16, 2013 at 01:38:44PM +0200, Felix Fietkau wrote: > >> @@ -692,16 +692,38 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx) > >> */ > >> rate_control_get_rate(tx->sdata, tx->sta, &txrc); > >> > >> - if (unlikely(info->control.rates[0].idx < 0)) > >> - return TX_DROP; > >> + if (tx->sta) > >> + ratetbl = rcu_dereference(tx->sta->sta.rates); > >> + > >> + if (unlikely(info->control.rates[0].idx < 0)) { > >> + if (ratetbl) { > >> + struct ieee80211_tx_rate rate = { > >> + .idx = ratetbl->rate[0].idx, > >> + .flags = ratetbl->rate[0].flags, > >> + .count = ratetbl->rate[0].count > >> + }; > >> + > >> + if (ratetbl->rate[0].idx < 0) > >> + return TX_DROP; > >> + > >> + tx->rate = rate; > > tx->rate = ratetbl->rate[0]; > No, the data structure in ratetbl->rate is not compatible. > Oops, yes. Karl -- 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