wifi: ath11k: Fix qmi_msg_handler data structure initialization
authorRahul Bhattacharjee <quic_rbhattac@quicinc.com>
Fri, 21 Oct 2022 09:01:26 +0000 (14:31 +0530)
committerKalle Valo <quic_kvalo@quicinc.com>
Wed, 2 Nov 2022 15:53:41 +0000 (17:53 +0200)
qmi_msg_handler is required to be null terminated by QMI module.
There might be a case where a handler for a msg id is not present in the
handlers array which can lead to infinite loop while searching the handler
and therefore out of bound access in qmi_invoke_handler().
Hence update the initialization in qmi_msg_handler data structure.

Tested-on: IPQ8074 hw2.0 AHB WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1

Signed-off-by: Rahul Bhattacharjee <quic_rbhattac@quicinc.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20221021090126.28626-1-quic_rbhattac@quicinc.com
drivers/net/wireless/ath/ath11k/qmi.c

index 145f20a..72cfc13 100644 (file)
@@ -3090,6 +3090,9 @@ static const struct qmi_msg_handler ath11k_qmi_msg_handlers[] = {
                        sizeof(struct qmi_wlfw_fw_init_done_ind_msg_v01),
                .fn = ath11k_qmi_msg_fw_init_done_cb,
        },
+
+       /* end of list */
+       {},
 };
 
 static int ath11k_qmi_ops_new_server(struct qmi_handle *qmi_hdl,