There's no value in setting a flag that will never be checked after this point, this seems to be legacy code -- I think previously the flag was used to check whether to encrypt the frame or not. Now, however, the flag need not be set, and setting it actually interferes if the frame will be processed again later. Signed-off-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> --- net/mac80211/tx.c | 3 --- 1 file changed, 3 deletions(-) --- wireless-testing.orig/net/mac80211/tx.c 2010-01-16 20:18:50.000000000 +0100 +++ wireless-testing/net/mac80211/tx.c 2010-01-16 20:19:05.000000000 +0100 @@ -553,9 +553,6 @@ ieee80211_tx_h_select_key(struct ieee802 } } - if (!tx->key || !(tx->key->flags & KEY_FLAG_UPLOADED_TO_HARDWARE)) - info->flags |= IEEE80211_TX_INTFL_DONT_ENCRYPT; - return TX_CONTINUE; } -- 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
