commit 2e44dd4a0f55e107fe29b7b5db14c872341ce2f7
author Sravan Kumar Kairam <sgoud@codeaurora.org> Mon Feb 13 18:31:09 2017 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Tue Mar 14 22:52:30 2017 -0700
tree 9a60d1e2aad89d6eeac87e44644f4f2a67bd83bc
parent d607ed59f72801fd37646e29070ab0505060b543

wlan: Drop packets other than EAPOL frames during pre assoc

Currently while pre assoc EAPOL caching there may be frames other than
EAPOL reaching host and at host those packets are neither dropped or
forwarded to kernel. As memory is not returned it will leads to RX low
resource at host.

In this change drop or return the memory for the packets other than
EAPOL frames.

Change-Id: I2bf955dfb5b6687bfbcbe48a1978a1f054307626
CRs-Fixed: 2005933

CORE/TL/src/wlan_qct_tl.c

diff --git a/CORE/TL/src/wlan_qct_tl.c b/CORE/TL/src/wlan_qct_tl.c
index 2454477..3be2835 100755
--- a/CORE/TL/src/wlan_qct_tl.c
+++ b/CORE/TL/src/wlan_qct_tl.c
@@ -6116,6 +6116,11 @@
                "%s: Cache Eapol frame", __func__));
       pTLCb->vosEapolCachedFrame = vosTempBuff;
    }
+   else {
+      TLLOG1(VOS_TRACE( VOS_MODULE_ID_TL, VOS_TRACE_LEVEL_INFO,
+               "%s: Drop duplicate EAPOL frame", __func__));
+      vos_pkt_return_packet(vosTempBuff);
+   }
 }
 
 /*==========================================================================
@@ -6182,7 +6187,9 @@
   v_S7_t              currentAvgRSSI = 0;
   v_U8_t              ac;
 #endif
+  uint8_t            ucMPDUHLen;
   uint16_t           seq_no;
+  uint16_t           usEtherType = 0;
 
   /*- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -*/
 
@@ -6363,6 +6370,7 @@
       ucSTAId = (v_U8_t)WDA_GET_RX_STAID( pvBDHeader );
       ucTid   = (v_U8_t)WDA_GET_RX_TID( pvBDHeader );
       uDPUSig = WDA_GET_RX_DPUSIG(pvBDHeader);
+      ucMPDUHLen = (uint8_t)WDA_GET_RX_MPDU_HEADER_LEN(pvBDHeader);
       seq_no = (uint16_t)WDA_GET_RX_REORDER_CUR_PKT_SEQ_NO(pvBDHeader);
 
       TLLOG2(VOS_TRACE( VOS_MODULE_ID_TL, VOS_TRACE_LEVEL_INFO_HIGH,
@@ -6391,9 +6399,21 @@
       /* Pre assoc cache eapol */
       if (pTLCb->preassoc_caching)
       {
-        TLLOG1(VOS_TRACE( VOS_MODULE_ID_TL, VOS_TRACE_LEVEL_INFO,
-               "WLAN TL:TL preassoc_caching is enabled seq No: %d", seq_no));
-         WLANTL_CacheEapol(pTLCb, vosTempBuff);
+         WLANTL_GetEtherType(pvBDHeader,vosTempBuff, ucMPDUHLen, &usEtherType);
+         if (WLANTL_LLC_8021X_TYPE != usEtherType)
+         {
+            VOS_TRACE(VOS_MODULE_ID_TL, VOS_TRACE_LEVEL_INFO,
+                      "%s: RX Frame not EAPOL EtherType %d",
+                      __func__, usEtherType);
+            vos_pkt_return_packet(vosTempBuff);
+         }
+         else
+         {
+            WLANTL_CacheEapol(pTLCb, vosTempBuff);
+            TLLOG1(VOS_TRACE( VOS_MODULE_ID_TL, VOS_TRACE_LEVEL_INFO,
+                "WLAN TL:TL preassoc_caching is enabled seq No: %d", seq_no));
+         }
+
          vosTempBuff = vosDataBuff;
          continue;
       }