Commit 1af83148 authored by Felix Fietkau's avatar Felix Fietkau Committed by Kalle Valo
Browse files

mt76: check qos ack policy before reordering packets 



Do not attempt to reorder packets not part of a BA session

Signed-off-by: default avatarFelix Fietkau <nbd@nbd.name>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
parent 18efed59

drivers/net/wireless/mediatek/mt76/agg-rx.c

+8 −1
Original line number Diff line number Diff line
@@ -147,12 +147,13 @@ mt76_rx_aggr_check_ctl(struct sk_buff *skb, struct sk_buff_head *frames) 
void mt76_rx_aggr_reorder(struct sk_buff *skb, struct sk_buff_head *frames)

{

	struct mt76_rx_status *status = (struct mt76_rx_status *) skb->cb;

	struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;

	struct mt76_wcid *wcid = status->wcid;

	struct ieee80211_sta *sta;

	struct mt76_rx_tid *tid;

	bool sn_less;

	u16 seqno, head, size;

	u8 idx;

	u8 ackp, idx;



	__skb_queue_tail(frames, skb);
@@ -165,6 +166,12 @@ void mt76_rx_aggr_reorder(struct sk_buff *skb, struct sk_buff_head *frames) 
		return;

	}



	/* not part of a BA session */

	ackp = *ieee80211_get_qos_ctl(hdr) & IEEE80211_QOS_CTL_ACK_POLICY_MASK;

	if (ackp != IEEE80211_QOS_CTL_ACK_POLICY_BLOCKACK &&

	    ackp != IEEE80211_QOS_CTL_ACK_POLICY_NORMAL)

		return;



	tid = rcu_dereference(wcid->aggr[status->tid]);

	if (!tid)

		return;