wifi: ath11k: Fix error handling in ath11k_wmi_p2p_noa_event()
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Mon, 15 Apr 2024 20:26:41 +0000 (22:26 +0200)
committerKalle Valo <quic_kvalo@quicinc.com>
Thu, 18 Apr 2024 15:20:08 +0000 (18:20 +0300)
commit6ef5b4c9598c928b3e2c4c4b543c81331941f136
tree24bc415a623198db8ee710ecca60cee7fd11a54a
parent43528ae9d9bfbf398dfd1cfedc8eecb953f6fde0
wifi: ath11k: Fix error handling in ath11k_wmi_p2p_noa_event()

if (noa_descriptors > WMI_P2P_MAX_NOA_DESCRIPTORS), there is a mix of
return and goto. In such a case, 'td' should be freed to avoid a memory
leak.

While at it, change ath11k_wmi_p2p_noa_event() to return void.
'0' was returned in all cases, even in case of error and the only caller
does not handle the return value.
This is also more consistent with most of functions called from
ath11k_wmi_tlv_op_rx().

Fixes: 2408379f15a1 ("wifi: ath11k: implement handling of P2P NoA event")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://msgid.link/07f1fc75b2d5b4173ae1b7bb1da5be7f6fc608c8.1713212781.git.christophe.jaillet@wanadoo.fr
drivers/net/wireless/ath/ath11k/wmi.c