ath10k: fix memory overwrite of the WoWLAN wakeup packet pattern
authorWen Gong <quic_wgong@quicinc.com>
Mon, 10 Jan 2022 14:24:13 +0000 (16:24 +0200)
committerKalle Valo <quic_kvalo@quicinc.com>
Tue, 11 Jan 2022 14:30:14 +0000 (16:30 +0200)
commite3fb3d4418fce5484dfe7995fcd94c18b10a431a
tree2e3805682c3d0cdd132e9a6b1e7dcfa7574b6ffc
parent3d38faef0de1756994b3d95e47b2302842f729e2
ath10k: fix memory overwrite of the WoWLAN wakeup packet pattern

In function ath10k_wow_convert_8023_to_80211(), it will do memcpy for
the new->pattern, and currently the new->pattern and new->mask is same
with the old, then the memcpy of new->pattern will also overwrite the
old->pattern, because the header format of new->pattern is 802.11,
its length is larger than the old->pattern which is 802.3. Then the
operation of "Copy frame body" will copy a mistake value because the
body memory has been overwrite when memcpy the new->pattern.

Assign another empty value to new_pattern to avoid the overwrite issue.

Tested-on: QCA6174 hw3.2 SDIO WLAN.RMH.4.4.1-00049

Fixes: fa3440fa2fa1 ("ath10k: convert wow pattern from 802.3 to 802.11")
Signed-off-by: Wen Gong <quic_wgong@quicinc.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20211222031347.25463-1-quic_wgong@quicinc.com
drivers/net/wireless/ath/ath10k/wow.c