On 07/09/2012 10:25, Johannes Berg wrote: > On Thu, 2012-09-06 at 19:07 +0200, Antonio Quartulli wrote: > >> - if (!(ifmgd->flags & IEEE80211_STA_MFP_ENABLED)) >> - IEEE80211_SKB_CB(skb)->flags |= >> - IEEE80211_TX_INTFL_DONT_ENCRYPT; > > >> + if (sdata->vif.type == NL80211_IFTYPE_STATION && >> + !(sdata->u.mgd.flags & IEEE80211_STA_MFP_ENABLED)) >> + IEEE80211_SKB_CB(skb)->flags |= >> + IEEE80211_TX_INTFL_DONT_ENCRYPT; > > It would seem that this should be > > if (sdata->vif.type != NL80211_IFTYPE_STATION || > !(flags & MFP_ENABLED)) > > ? Or just never set TX_INTFL_DONT_ENCRYPT at all and rely on ieee80211_tx_h_select_key() to do the right thing ? -- 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
