Commit 1990af8d authored by Johannes Berg's avatar Johannes Berg Committed by David S. Miller

[MAC80211]: consolidate decryption more

Currently, we have three RX handlers doing the decryption.
This patch changes it to have only one handler doing
everything, thereby getting rid of many duplicate checks.
Signed-off-by: default avatarJohannes Berg <johannes@sipsolutions.net>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
--
 net/mac80211/rx.c |   46 ++++++++++++----------------------------------
 1 files changed, 12 insertions(+), 34 deletions(-)
parent 70f08765
...@@ -419,7 +419,7 @@ ieee80211_rx_h_check(struct ieee80211_txrx_data *rx) ...@@ -419,7 +419,7 @@ ieee80211_rx_h_check(struct ieee80211_txrx_data *rx)
static ieee80211_txrx_result static ieee80211_txrx_result
ieee80211_rx_h_load_key(struct ieee80211_txrx_data *rx) ieee80211_rx_h_decrypt(struct ieee80211_txrx_data *rx)
{ {
struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) rx->skb->data; struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) rx->skb->data;
int keyidx; int keyidx;
...@@ -456,7 +456,7 @@ ieee80211_rx_h_load_key(struct ieee80211_txrx_data *rx) ...@@ -456,7 +456,7 @@ ieee80211_rx_h_load_key(struct ieee80211_txrx_data *rx)
return TXRX_CONTINUE; return TXRX_CONTINUE;
/* /*
* No point in finding a key if the frame is neither * No point in finding a key and decrypting if the frame is neither
* addressed to us nor a multicast frame. * addressed to us nor a multicast frame.
*/ */
if (!(rx->flags & IEEE80211_TXRXD_RXRA_MATCH)) if (!(rx->flags & IEEE80211_TXRXD_RXRA_MATCH))
...@@ -507,42 +507,21 @@ ieee80211_rx_h_load_key(struct ieee80211_txrx_data *rx) ...@@ -507,42 +507,21 @@ ieee80211_rx_h_load_key(struct ieee80211_txrx_data *rx)
if (rx->key) { if (rx->key) {
rx->key->tx_rx_count++; rx->key->tx_rx_count++;
/* TODO: add threshold stuff again */ /* TODO: add threshold stuff again */
} } else {
return TXRX_CONTINUE;
}
static ieee80211_txrx_result
ieee80211_rx_h_wep_weak_iv_detection(struct ieee80211_txrx_data *rx)
{
if (!rx->sta || !(rx->fc & IEEE80211_FCTL_PROTECTED) ||
(rx->fc & IEEE80211_FCTL_FTYPE) != IEEE80211_FTYPE_DATA ||
!rx->key || rx->key->conf.alg != ALG_WEP ||
!(rx->flags & IEEE80211_TXRXD_RXRA_MATCH))
return TXRX_CONTINUE;
/* Check for weak IVs, if hwaccel did not remove IV from the frame */
if (!(rx->u.rx.status->flag & RX_FLAG_IV_STRIPPED) ||
!(rx->u.rx.status->flag & RX_FLAG_DECRYPTED))
if (ieee80211_wep_is_weak_iv(rx->skb, rx->key))
rx->sta->wep_weak_iv_count++;
return TXRX_CONTINUE;
}
static ieee80211_txrx_result
ieee80211_rx_h_decrypt(struct ieee80211_txrx_data *rx)
{
if (!(rx->fc & IEEE80211_FCTL_PROTECTED))
return TXRX_CONTINUE;
if (!rx->key) {
if (net_ratelimit()) if (net_ratelimit())
printk(KERN_DEBUG "%s: RX protected frame," printk(KERN_DEBUG "%s: RX protected frame,"
" but have no key\n", rx->dev->name); " but have no key\n", rx->dev->name);
return TXRX_DROP; return TXRX_DROP;
} }
/* Check for weak IVs if possible */
if (rx->sta && rx->key->conf.alg == ALG_WEP &&
((rx->fc & IEEE80211_FCTL_FTYPE) == IEEE80211_FTYPE_DATA) &&
(!(rx->u.rx.status->flag & RX_FLAG_IV_STRIPPED) ||
!(rx->u.rx.status->flag & RX_FLAG_DECRYPTED)) &&
ieee80211_wep_is_weak_iv(rx->skb, rx->key))
rx->sta->wep_weak_iv_count++;
switch (rx->key->conf.alg) { switch (rx->key->conf.alg) {
case ALG_WEP: case ALG_WEP:
return ieee80211_crypto_wep_decrypt(rx); return ieee80211_crypto_wep_decrypt(rx);
...@@ -551,6 +530,7 @@ ieee80211_rx_h_decrypt(struct ieee80211_txrx_data *rx) ...@@ -551,6 +530,7 @@ ieee80211_rx_h_decrypt(struct ieee80211_txrx_data *rx)
case ALG_CCMP: case ALG_CCMP:
return ieee80211_crypto_ccmp_decrypt(rx); return ieee80211_crypto_ccmp_decrypt(rx);
case ALG_NONE: case ALG_NONE:
WARN_ON(1);
return TXRX_CONTINUE; return TXRX_CONTINUE;
} }
...@@ -1348,8 +1328,6 @@ ieee80211_rx_handler ieee80211_rx_handlers[] = ...@@ -1348,8 +1328,6 @@ ieee80211_rx_handler ieee80211_rx_handlers[] =
ieee80211_rx_h_if_stats, ieee80211_rx_h_if_stats,
ieee80211_rx_h_passive_scan, ieee80211_rx_h_passive_scan,
ieee80211_rx_h_check, ieee80211_rx_h_check,
ieee80211_rx_h_load_key,
ieee80211_rx_h_wep_weak_iv_detection,
ieee80211_rx_h_decrypt, ieee80211_rx_h_decrypt,
ieee80211_rx_h_sta_process, ieee80211_rx_h_sta_process,
ieee80211_rx_h_defragment, ieee80211_rx_h_defragment,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment