]> git.ipfire.org Git - thirdparty/linux.git/commit
net: ethernet: mtk_eth_soc: Fix use-after-free in metadata dst teardown
authorLorenzo Bianconi <lorenzo@kernel.org>
Tue, 2 Jun 2026 09:21:05 +0000 (11:21 +0200)
committerJakub Kicinski <kuba@kernel.org>
Thu, 4 Jun 2026 02:15:32 +0000 (19:15 -0700)
commit80df409e1a483676826a6c66e693dba6ac507751
treec407a812995141d8817a12eec0b7fa2419209a7f
parentb38cae85d1c45ff189d7ecb6ac36f41cdc3d84d0
net: ethernet: mtk_eth_soc: Fix use-after-free in metadata dst teardown

mtk_free_dev() calls metadata_dst_free() which frees the metadata_dst
with kfree() immediately, bypassing the RCU grace period.
In the RX path, skb_dst_set_noref() sets a non-refcounted pointer from
the skb to the metadata_dst. This function requires RCU read-side
protection and the dst must remain valid until all RCU readers complete.
Since metadata_dst_free() calls kfree() directly, a use-after-free can
occur if any skb still holds a noref pointer to the dst when the driver
tears it down.
Replace metadata_dst_free() with dst_release() which properly goes
through the refcount path: when the refcount drops to zero, it schedules
the actual free via call_rcu_hurry(), ensuring all RCU readers have
completed before the memory is freed.

Fixes: 2d7605a72906 ("net: ethernet: mtk_eth_soc: enable hardware DSA untagging")
Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Link: https://patch.msgid.link/20260602-airoha-mtk-metadata-uaf-fix-v1-2-3aaa99d83351@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/mediatek/mtk_eth_soc.c