Unverified Commit 41aad090 authored by Xia Fukun's avatar Xia Fukun Committed by Mark Brown
Browse files

ASoC: SOF: Fix incorrect use of sizeof in sof_ipc3_do_rx_work() 



Here hdr is a pointer, and we should measure the size of
struct sof_ipc_cmd_hdr.

Fixes: 12c41c77 ("ASoC: SOF: Refactor rx function for fuzzing")
Signed-off-by: default avatarXia Fukun <xiafukun@huawei.com>
Reviewed-by: default avatarCurtis Malainey <cujomalainey@chromium.org>
Link: https://lore.kernel.org/r/20230807075118.128122-1-xiafukun@huawei.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 1f566435

sound/soc/sof/ipc3.c

+1 −1
Original line number Diff line number Diff line
@@ -1001,7 +1001,7 @@ void sof_ipc3_do_rx_work(struct snd_sof_dev *sdev, struct sof_ipc_cmd_hdr *hdr, 


	ipc3_log_header(sdev->dev, "ipc rx", hdr->cmd);



	if (hdr->size < sizeof(hdr) || hdr->size > SOF_IPC_MSG_MAX_SIZE) {

	if (hdr->size < sizeof(*hdr) || hdr->size > SOF_IPC_MSG_MAX_SIZE) {

		dev_err(sdev->dev, "The received message size is invalid: %u\n",

			hdr->size);

		return;