mac80211: remove useless setting of IEEE80211_TX_INTFL_DONT_ENCRYPT
authorJohannes Berg <johannes@sipsolutions.net>
Sun, 17 Jan 2010 00:47:57 +0000 (01:47 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 19 Jan 2010 21:25:17 +0000 (16:25 -0500)
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@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/tx.c

index a74ab797fed92f61af9680e531ae2ad0de12acc6..9afbee0d53c0c6bed2956e4f4ec2808367b3e3a5 100644 (file)
@@ -553,9 +553,6 @@ ieee80211_tx_h_select_key(struct ieee80211_tx_data *tx)
                }
        }
 
-       if (!tx->key || !(tx->key->flags & KEY_FLAG_UPLOADED_TO_HARDWARE))
-               info->flags |= IEEE80211_TX_INTFL_DONT_ENCRYPT;
-
        return TX_CONTINUE;
 }