From: Felix Fietkau <nbd@openwrt.org>
Date: Wed, 13 Jul 2011 15:37:36 +0000 (+0000)
Subject: ath9k: add some more fixes to the mic failure handling rework patch
X-Git-Url: http://git.openwrt.org/?p=openwrt%2Fsvn-archive%2Farchive.git;a=commitdiff_plain;hb=24461f95140033b47d2da9866eef739096b4304a

ath9k: add some more fixes to the mic failure handling rework patch

SVN-Revision: 27607
---

diff --git a/package/mac80211/patches/550-ath9k_mmic_verify.patch b/package/mac80211/patches/550-ath9k_mmic_verify.patch
index 1918e11d7c..a6b2502bf2 100644
--- a/package/mac80211/patches/550-ath9k_mmic_verify.patch
+++ b/package/mac80211/patches/550-ath9k_mmic_verify.patch
@@ -24,12 +24,31 @@
  	if (!rx_stats->rs_datalen)
  		return false;
          /*
-@@ -850,25 +853,9 @@ static bool ath9k_rx_accept(struct ath_c
+@@ -838,6 +841,11 @@ static bool ath9k_rx_accept(struct ath_c
+ 	if (rx_stats->rs_more)
+ 		return true;
+ 
++	mic_error = is_valid_tkip && !ieee80211_is_ctl(fc) &&
++		!ieee80211_has_morefrags(fc) &&
++		!(le16_to_cpu(hdr->seq_ctrl) & IEEE80211_SCTL_FRAG) &&
++		(rx_stats->rs_status & ATH9K_RXERR_MIC);
++
+ 	/*
+ 	 * The rx_stats->rs_status will not be set until the end of the
+ 	 * chained descriptors so it can be ignored if rs_more is set. The
+@@ -845,30 +853,18 @@ static bool ath9k_rx_accept(struct ath_c
+ 	 * descriptors.
+ 	 */
+ 	if (rx_stats->rs_status != 0) {
+-		if (rx_stats->rs_status & ATH9K_RXERR_CRC)
++		if (rx_stats->rs_status & ATH9K_RXERR_CRC) {
+ 			rxs->flag |= RX_FLAG_FAILED_FCS_CRC;
++			mic_error = false;
++		}
  		if (rx_stats->rs_status & ATH9K_RXERR_PHY)
  			return false;
  
--		if (rx_stats->rs_status & ATH9K_RXERR_DECRYPT) {
-+		if (rx_stats->rs_status & ATH9K_RXERR_DECRYPT)
+ 		if (rx_stats->rs_status & ATH9K_RXERR_DECRYPT) {
  			*decrypt_error = true;
 -		} else if (rx_stats->rs_status & ATH9K_RXERR_MIC) {
 -			bool is_mc;
@@ -39,7 +58,7 @@
 -			 * decrypted using a valid TKIP key.
 -			 */
 -			is_mc = !!is_multicast_ether_addr(hdr->addr1);
- 
+-
 -			if (!ieee80211_is_ctl(fc) &&
 -			    !ieee80211_has_morefrags(fc) &&
 -			    !(le16_to_cpu(hdr->seq_ctrl) & IEEE80211_SCTL_FRAG) &&
@@ -47,11 +66,13 @@
 -				rxs->flag |= RX_FLAG_MMIC_ERROR;
 -			else
 -				rx_stats->rs_status &= ~ATH9K_RXERR_MIC;
--		}
++			mic_error = false;
+ 		}
++
  		/*
  		 * Reject error frames with the exception of
  		 * decryption and MIC failures. For monitor mode,
-@@ -886,6 +873,18 @@ static bool ath9k_rx_accept(struct ath_c
+@@ -886,6 +882,18 @@ static bool ath9k_rx_accept(struct ath_c
  			}
  		}
  	}
@@ -70,7 +91,7 @@
  	return true;
  }
  
-@@ -1940,6 +1939,9 @@ int ath_rx_tasklet(struct ath_softc *sc,
+@@ -1940,6 +1948,9 @@ int ath_rx_tasklet(struct ath_softc *sc,
  			sc->rx.rxotherant = 0;
  		}