]> git.itanic.dy.fi Git - linux-stable/commitdiff
Merge patch series "can: j1939: avoid possible use-after-free when j1939_can_rx_regis...
authorMarc Kleine-Budde <mkl@pengutronix.de>
Mon, 5 Jun 2023 06:27:23 +0000 (08:27 +0200)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Mon, 5 Jun 2023 06:27:23 +0000 (08:27 +0200)
Fedor Pchelkin <pchelkin@ispras.ru> says:

The patch series fixes a possible racy use-after-free scenario
described in 2/2: if j1939_can_rx_register() fails then the concurrent
thread may have already read the invalid priv structure.

The 1/2 makes j1939_netdev_lock a mutex so that access to
j1939_can_rx_register() can be serialized without changing GFP_KERNEL
to GFP_ATOMIC inside can_rx_register(). This seems to be safe.

Note that the patch series has been tested only via Syzkaller and not
with a real device.

Link: https://lore.kernel.org/r/20230526171910.227615-1-pchelkin@ispras.ru
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>

Trivial merge