Commit c1b5209e authored by Dan Carpenter's avatar Dan Carpenter Committed by Zheng Zengkai
Browse files

mfd: wm831x-auxadc: Prevent use after free in wm831x_auxadc_read_irq()



stable inclusion
from stable-5.10.20
commit b1b904813755d825eff31222d2d24b05638c0c52
bugzilla: 50608

--------------------------------

[ Upstream commit 26783d74 ]

The "req" struct is always added to the "wm831x->auxadc_pending" list,
but it's only removed from the list on the success path.  If a failure
occurs then the "req" struct is freed but it's still on the list,
leading to a use after free.

Fixes: 78bb3688 ("mfd: Support multiple active WM831x AUXADC conversions")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Acked-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
Signed-off-by: default avatarChen Jun <chenjun102@huawei.com>
Acked-by: default avatarXie XiuQi <xiexiuqi@huawei.com>
Signed-off-by: default avatarZheng Zengkai <zhengzengkai@huawei.com>
parent 9cf9b8b2
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment