[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240211150937.4058-1-d.dulov@aladdin.ru>
Date: Sun, 11 Feb 2024 07:09:37 -0800
From: Daniil Dulov <d.dulov@...ddin.ru>
To: Vinod Koul <vkoul@...nel.org>
CC: Daniil Dulov <d.dulov@...ddin.ru>, Bard Liao
<yung-chuan.liao@...ux.intel.com>, Pierre-Louis Bossart
<pierre-louis.bossart@...ux.intel.com>, Sanyog Kale
<sanyog.r.kale@...el.com>, <alsa-devel@...a-project.org>,
<linux-kernel@...r.kernel.org>, <lvc-project@...uxtesting.org>
Subject: [PATCH] soundwire: fix double free of pointer
If sdw_ml_sync_bank_switch() returns error not on the first iteration,
it leads to freeing prevously freed memory. So, set the pointer to NULL
after each successful bank switch.
Signed-off-by: Daniil Dulov <d.dulov@...ddin.ru>
---
drivers/soundwire/stream.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/soundwire/stream.c b/drivers/soundwire/stream.c
index 304ff2ee7d75..d650e6f0f8e7 100644
--- a/drivers/soundwire/stream.c
+++ b/drivers/soundwire/stream.c
@@ -833,6 +833,7 @@ static int do_bank_switch(struct sdw_stream_runtime *stream)
"multi link bank switch failed: %d\n", ret);
goto error;
}
+ bus->defer_msg.msg = NULL;
if (multi_link)
mutex_unlock(&bus->msg_lock);
--
2.25.1
Powered by blists - more mailing lists