]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
can: hi311x: fix null pointer dereference when resuming from sleep before interface...
authorChen Yufeng <chenyufeng@iie.ac.cn>
Thu, 11 Sep 2025 15:08:20 +0000 (23:08 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 15 Oct 2025 09:56:24 +0000 (11:56 +0200)
commitd1fc4c041459e2d4856c1b2501486ba4f0cbf96b
tree9103aba529c8e7abea5b08f00db435c2227c4ccf
parentc1a6e988e555ebbba68e22cf1f1093b884b2dd8c
can: hi311x: fix null pointer dereference when resuming from sleep before interface was enabled

[ Upstream commit 6b696808472197b77b888f50bc789a3bae077743 ]

This issue is similar to the vulnerability in the `mcp251x` driver,
which was fixed in commit 03c427147b2d ("can: mcp251x: fix resume from
sleep before interface was brought up").

In the `hi311x` driver, when the device resumes from sleep, the driver
schedules `priv->restart_work`. However, if the network interface was
not previously enabled, the `priv->wq` (workqueue) is not allocated and
initialized, leading to a null pointer dereference.

To fix this, we move the allocation and initialization of the workqueue
from the `hi3110_open` function to the `hi3110_can_probe` function.
This ensures that the workqueue is properly initialized before it is
used during device resume. And added logic to destroy the workqueue
in the error handling paths of `hi3110_can_probe` and in the
`hi3110_can_remove` function to prevent resource leaks.

Signed-off-by: Chen Yufeng <chenyufeng@iie.ac.cn>
Link: https://patch.msgid.link/20250911150820.250-1-chenyufeng@iie.ac.cn
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/can/spi/hi311x.c