]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
veth: convert frag_list skbs before running XDP
authorMatt Fleming <mfleming@cloudflare.com>
Wed, 22 Jul 2026 19:19:25 +0000 (20:19 +0100)
committerJakub Kicinski <kuba@kernel.org>
Sat, 25 Jul 2026 00:02:10 +0000 (17:02 -0700)
commitd0d6415963040c401e7a7e4e482a698ba52448cb
tree14b5a094187ef6fb7dc1b1be105ebe2148c389ae
parent817ff6efdb7f484ea547218e11e17d8e43daa3b4
veth: convert frag_list skbs before running XDP

A frag_list skb can reach veth with data_len set but nr_frags zero.
veth_convert_skb_to_xdp_buff() only converts skbs that are shared,
locked, have frags[], or do not have enough headroom. It later uses
skb_is_nonlinear() to decide whether to set XDP_FLAGS_HAS_FRAGS and
xdp_frags_size.

That exposes frag_list data to XDP as if it were stored in frags[], but
frags[] is empty. AF_XDP copy mode can then trust the bogus XDP fragment
metadata, walk an empty fragment entry, and crash in memcpy() from
__xsk_rcv().

Route non-linear skbs through skb_pp_cow_data() before exposing them to
XDP, and only advertise XDP frags when the resulting skb has frags[].
skb_copy_bits() already handles frag_list input, and skb_pp_cow_data()
builds frags[] output with skb_add_rx_frag(), which is the
representation XDP multi-buffer expects.

Fixes: 718a18a0c8a6 ("veth: Rework veth_xdp_rcv_skb in order to accept non-linear skb")
Cc: stable@vger.kernel.org
Signed-off-by: Matt Fleming <mfleming@cloudflare.com>
Reviewed-by: Toke Høiland-Jørgensen <toke@toke.dk>
Acked-by: Lorenzo Bianconi <lorenzo@kernel.org>
Link: https://patch.msgid.link/20260722191925.2192070-1-matt@readmodwrite.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/veth.c
net/core/skbuff.c