Re: [PATCH net] slip: fix use-after-free in sl_sync()

From
Jakub Kicinski <kuba@kernel.org>
Date
2026-08-20 20:59:22
Also in
lkml
Reply to
Aleksandr Khromov, 2026-08-17
On Mon, 17 Aug 2026 10:09:52 +0300 Aleksandr Khromov wrote:
slip_devs[] stores bare net_device pointers and takes no reference on
them.  sl_sync() and sl_alloc() walk that table from slip_open() under
rtnl_lock(), while an entry is dropped by sl_free_netdev(), which
sl_setup() installs as dev->priv_destructor.
Would it not be a much simpler fix to move the deinit to ndo_uninit
if we need rtnl_lock?
-- 
pw-bot: cr
Thread (2 messages) 2 messages, 2 authors, last reply 2026-08-20
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help