ath9k: Fix potential out of bounds in ath9k_htc_txcompletion_cb()
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 13 Aug 2020 14:12:53 +0000 (17:12 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Mon, 17 Aug 2020 10:24:01 +0000 (13:24 +0300)
commit2705cd7558e718a7240c64eb0afb2edad5f8c190
treede8a157cc13b4ce74f54695659e36c40f6612029
parent5024f21c159f8c1668f581fff37140741c0b1ba9
ath9k: Fix potential out of bounds in ath9k_htc_txcompletion_cb()

The value of "htc_hdr->endpoint_id" comes from skb->data so Smatch marks
it as untrusted so we have to check it before using it as an array
offset.

This is similar to a bug that syzkaller found in commit e4ff08a4d727
("ath9k: Fix use-after-free Write in ath9k_htc_rx_msg") so it is
probably a real issue.

Fixes: fb9987d0f748 ("ath9k_htc: Support for AR9271 chipset.")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200813141253.GA457408@mwanda
drivers/net/wireless/ath/ath9k/htc_hst.c