]> git.ipfire.org Git - thirdparty/linux.git/commit
tipc: fix UAF in tipc_l2_send_msg()
authorEric Dumazet <edumazet@google.com>
Fri, 12 Jun 2026 13:59:49 +0000 (13:59 +0000)
committerJakub Kicinski <kuba@kernel.org>
Mon, 15 Jun 2026 19:50:29 +0000 (12:50 -0700)
commitf4c3d89fc986b0da196ddfc6cfe0ea5d5d08bec6
treea19812911428662103dcf8175face6f83be2a3ca
parent62821d481975d830ed67f30d0cf2b2036bf3b5ca
tipc: fix UAF in tipc_l2_send_msg()

Syzbot reported a slab-use-after-free in ipvlan_hard_header() when
called from tipc_l2_send_msg().

The root cause is that tipc_disable_l2_media() calls synchronize_net()
while b->media_ptr is still valid. This allows concurrent RCU readers
to obtain the device pointer after synchronize_net() has finished.
The pointer is cleared later in bearer_disable(), but without any
subsequent synchronization, allowing the device to be freed while
still in use by readers.

Fix this by clearing b->media_ptr in tipc_disable_l2_media() before
calling synchronize_net().

This is safe to do now because the call order in bearer_disable()
was reversed in 0d051bf93c06 ("tipc: make bearer packet filtering generic")
to call tipc_node_delete_links() (which needs the pointer) before
disable_media().

Fixes: 282b3a056225 ("tipc: send out RESET immediately when link goes down")
https: //lore.kernel.org/netdev/6a2c1007.428ffe26.258b27.015d.GAE@google.com/T/#u
Reported-by: syzbot+64ec81389cbad56a8c35@syzkaller.appspotmail.com
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Jon Maloy <jmaloy@redhat.com>
Reviewed-by: Tung Nguyen <tung.quang.nguyen@est.tech>
Link: https://patch.msgid.link/20260612135949.4010482-1-edumazet@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/tipc/bearer.c