[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230807075118.128122-1-xiafukun@huawei.com>
Date: Mon, 7 Aug 2023 15:51:18 +0800
From: Xia Fukun <xiafukun@...wei.com>
To: <peter.ujfalusi@...ux.intel.com>, <broonie@...nel.org>,
<pierre-louis.bossart@...ux.intel.com>,
<ranjani.sridharan@...ux.intel.com>,
<yung-chuan.liao@...ux.intel.com>, <cujomalainey@...omium.org>,
<noah.klayman@...el.com>, <daniel.baluta@....com>,
<rander.wang@...el.com>
CC: <linux-kernel@...r.kernel.org>, <xiafukun@...wei.com>
Subject: [PATCH] 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: 12c41c779fad ("ASoC: SOF: Refactor rx function for fuzzing")
Signed-off-by: Xia Fukun <xiafukun@...wei.com>
---
sound/soc/sof/ipc3.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/sof/ipc3.c b/sound/soc/sof/ipc3.c
index 2c5aac31e8b0..580960ff273d 100644
--- a/sound/soc/sof/ipc3.c
+++ b/sound/soc/sof/ipc3.c
@@ -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;
--
2.17.1
Powered by blists - more mailing lists