On 4/15/2024 1:26 PM, Christophe JAILLET wrote: > 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@xxxxxxxxxx> Acked-by: Jeff Johnson <quic_jjohnson@xxxxxxxxxxx>