Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2e3a4044 authored by Manjunathappa Prakash's avatar Manjunathappa Prakash Committed by snandini
Browse files

qcacmn: Bypass PN check for open security mode defrag frames 

Do not do PN check for open security mode defrag frames as it
is not applicable.

Change-Id: I7ed1073953c08b191c15c659a0d216eb7ed49b31
CRs-Fixed: 2753520
parent e247ff6f

dp/wifi3.0/dp_rx_defrag.c

+49 −22
Original line number Diff line number Diff line
@@ -826,6 +826,38 @@ static void dp_rx_frag_pull_hdr(qdf_nbuf_t nbuf, uint16_t hdrsize) 
		  __func__, (uint32_t)qdf_nbuf_len(nbuf), hdrsize);

}



/*

 * dp_rx_defrag_pn_check(): Check the PN of current fragmented with prev PN

 * @msdu: msdu to get the current PN

 * @cur_pn128: PN extracted from current msdu

 * @prev_pn128: Prev PN

 *

 * Returns: 0 on success, non zero on failure

 */

static int dp_rx_defrag_pn_check(qdf_nbuf_t msdu,

				 uint64_t *cur_pn128, uint64_t *prev_pn128)

{

	struct rx_pkt_tlvs *rx_pkt_tlv =

			(struct rx_pkt_tlvs *)qdf_nbuf_data(msdu);

	struct rx_mpdu_info *rx_mpdu_info_details =

	 &rx_pkt_tlv->mpdu_start_tlv.rx_mpdu_start.rx_mpdu_info_details;

	int out_of_order = 0;



	cur_pn128[0] = rx_mpdu_info_details->pn_31_0;

	cur_pn128[0] |=

		((uint64_t)rx_mpdu_info_details->pn_63_32 << 32);

	cur_pn128[1] = rx_mpdu_info_details->pn_95_64;

	cur_pn128[1] |=

		((uint64_t)rx_mpdu_info_details->pn_127_96 << 32);



	if (cur_pn128[1] == prev_pn128[1])

		out_of_order = (cur_pn128[0] <= prev_pn128[0]);

	else

		out_of_order = (cur_pn128[1] < prev_pn128[1]);



	return out_of_order;

}



/*

 * dp_rx_construct_fraglist(): Construct a nbuf fraglist

 * @peer: Pointer to the peer
@@ -844,35 +876,30 @@ dp_rx_construct_fraglist(struct dp_peer *peer, int tid, qdf_nbuf_t head, 
	qdf_nbuf_t rx_nbuf = msdu;

	struct dp_rx_tid *rx_tid = &peer->rx_tid[tid];

	uint32_t len = 0;

	uint64_t cur_pn128[2], prev_pn128[2];

	uint64_t cur_pn128[2] = {0, 0}, prev_pn128[2];

	int out_of_order = 0;

	int index;

	int needs_pn_check = 0;



	prev_pn128[0] = rx_tid->pn128[0];

	prev_pn128[1] = rx_tid->pn128[1];



	while (msdu) {

		struct rx_pkt_tlvs *rx_pkt_tlv =

				(struct rx_pkt_tlvs *)qdf_nbuf_data(msdu);

		struct rx_mpdu_info *rx_mpdu_info_details =

		 &rx_pkt_tlv->mpdu_start_tlv.rx_mpdu_start.rx_mpdu_info_details;

	index = hal_rx_msdu_is_wlan_mcast(msdu) ? dp_sec_mcast : dp_sec_ucast;

	if (qdf_likely(peer->security[index].sec_type != cdp_sec_type_none))

		needs_pn_check = 1;



		cur_pn128[0] = rx_mpdu_info_details->pn_31_0;

		cur_pn128[0] |=

			((uint64_t)rx_mpdu_info_details->pn_63_32 << 32);

		cur_pn128[1] = rx_mpdu_info_details->pn_95_64;

		cur_pn128[1] |=

			((uint64_t)rx_mpdu_info_details->pn_127_96 << 32);

	while (msdu) {

		if (qdf_likely(needs_pn_check))

			out_of_order = dp_rx_defrag_pn_check(msdu,

							     &cur_pn128[0],

							     &prev_pn128[0]);



		dp_debug("cur_pn128[0] 0x%llx cur_pn128[1] 0x%llx prev_pn128[0] 0x%llx prev_pn128[1] 0x%llx",

		if (qdf_unlikely(out_of_order)) {

			dp_info_rl("cur_pn128[0] 0x%llx cur_pn128[1] 0x%llx prev_pn128[0] 0x%llx prev_pn128[1] 0x%llx",

				   cur_pn128[0], cur_pn128[1],

				   prev_pn128[0], prev_pn128[1]);

		if (cur_pn128[1] == prev_pn128[1])

			out_of_order = (cur_pn128[0] <= prev_pn128[0]);

		else

			out_of_order = (cur_pn128[1] < prev_pn128[1]);



		if (out_of_order)

			return QDF_STATUS_E_FAILURE;

		}



		prev_pn128[0] = cur_pn128[0];

		prev_pn128[1] = cur_pn128[1];