Sfoglia il codice sorgente

qcacld-3.0: Fix incorrect NULL pointer check for MAC layer

Propagate from qcacld-2.0 to qcacld-3.0

Currently there are some places where array name is compared to NULL
in MAC. Add fix to correct it.

CRs-Fixed: 1063255
Change-Id: I4f620feff23475d0b719fae7ef8e0ac1ab3b8908
Yingying Tang 8 anni fa
parent
commit
54ebfe43ab

+ 0 - 4
core/mac/src/pe/lim/lim_process_assoc_rsp_frame.c

@@ -533,10 +533,6 @@ lim_process_assoc_rsp_frame(tpAniSirGlobal mac_ctx,
 		lim_log(mac_ctx, LOGE,
 			FL("LFR3: Reassoc response packet header is NULL"));
 		return;
-	} else if (hdr->sa == NULL) {
-		lim_log(mac_ctx, LOGE,
-			FL("LFR3: Reassoc resp packet source address is NULL"));
-		return;
 	}
 
 	lim_log(mac_ctx, LOG1,

+ 4 - 5
core/mac/src/pe/lim/lim_process_deauth_frame.c

@@ -263,11 +263,10 @@ lim_process_deauth_frame(tpAniSirGlobal pMac, uint8_t *pRxPacketInfo,
 	 *     AP we're currently associated with (case a), then proceed
 	 *     with normal deauth processing.
 	 */
-	if (psessionEntry->limReAssocbssId != NULL) {
-		pRoamSessionEntry =
-			pe_find_session_by_bssid(pMac, psessionEntry->limReAssocbssId,
-						 &roamSessionId);
-	}
+	pRoamSessionEntry =
+		pe_find_session_by_bssid(pMac, psessionEntry->limReAssocbssId,
+							&roamSessionId);
+
 	if (lim_is_reassoc_in_progress(pMac, psessionEntry)
 	    || lim_is_reassoc_in_progress(pMac, pRoamSessionEntry)) {
 		if (!IS_REASSOC_BSSID(pMac, pHdr->sa, psessionEntry)) {