From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f42.google.com (mail-pj1-f42.google.com [209.85.216.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F2D013546EC for ; Fri, 31 Jul 2026 03:24:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785468260; cv=none; b=kJJFrOJksPxCpFOBI1RgT9RyLwQvQQSFuEXnVeoZyLyegqpVVccptOZlgYd1D84Om7+4t7D2XmJafId5DCAV2fLSV5z/wZ2OqeZgLA+rpu0/uK7nvtJYi/EnTe2fb9YOYTfwZo0N79F9o4SM8LPVt5VQvnSk8RcHLnjqch5O818= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785468260; c=relaxed/simple; bh=RTtDn8xLk4cqizbu/eCai5rh4lCOJEd4o/Ihvcc9P4k=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=CvFDxnPN1SNwW91R9t9v3RSHX/6DlxmN9xIfqE+dF0XV3gaMhIyFwxtdWbkoqgBCfifOOWa0fS2gegB23O9Cojv0pS9Wv7IRYv4AbOUVho1S2e32A8J2ygx3q3GlU2IzGDJYekygFupT7+EKyvsi3BR4qt3/g2+NKaN+paq7v5M= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=SmdZ0yvB; arc=none smtp.client-ip=209.85.216.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="SmdZ0yvB" Received: by mail-pj1-f42.google.com with SMTP id 98e67ed59e1d1-3856d6fbcb3so409122a91.2 for ; Thu, 30 Jul 2026 20:24:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785468258; x=1786073058; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=XgtOPZlcFx69TNmsMfc9ttGlQcFwyZj8UdAlpvl+1TI=; b=SmdZ0yvBbtiIENE8Z2WPaMJSFNd7b40OaqtYAM//jmikx8JOAOlxmfRSSJ39sNy+HH yQOsEU73smnNdL0JJD6sL9UMLjBQ7GGAow/S3+dmuSsaLaBLiCzJHhywoN4EiefcU+DZ 2mgWj6QxrWfeMQENNNKcRxq7yfsY1j++glUG6U6Olzjm2tOKL+UYmtku/YON12jfbOxl 2RaE1LfUo++hyYfxnpr0bSXi2y/oKAxkoU20c480Qp+9mJu8eGZ1QKxZps816r+28fDH AKqIpaygnjxvMOVEcWt1AVMgSpsUjoS2ZH9T2UbqNMw4JLtCtstLBhh4SDeqoprjWIGL WygA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785468258; x=1786073058; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=XgtOPZlcFx69TNmsMfc9ttGlQcFwyZj8UdAlpvl+1TI=; b=Atlyp/bG4YDQgK+TS4M6y/RMxdYc84oQrQMX3jT7OFREP+W5Nmpvdiz2gRk/eIHl7H 2PUS7gPG4z4QdBhYBkjKxfGV1U7EcR7Qf3YNZJgbQGRuRagWpO/qxP3F55ppBXCyFGAP 1i5jnps5jbqRYSecblbuyioptFf97POlplYQc+Rj53vrTYNd5PKMWgcz61SF+2vmboap DyHP4rpE9eIZ9yOIUTzqehwOq+YHSgCx+9KYh4qx6RsbymlPK1AUKs1lYJro6G0D8uK8 icJDIuS1hR3fz1ZgDhTV4gu4qDauFhUfG01tDai8s4286HhAbMs7zDUxNtFwomNW39eQ R7KA== X-Forwarded-Encrypted: i=1; AHgh+RqxmPDpHhsjY/oenTtyb/n8X6F2Oh9UNUy5rRJrcCOvdSvTNCQtPmHgo8hMhM8YrQs04lu9JDxGAscWPJc=@vger.kernel.org X-Gm-Message-State: AOJu0Yz+Uq4OnVDaCEVygtztodFGGmgpWs21Ljdf05KcZYkQ4XhBlmNG nHDAd/sbZRukTDtMItqUDQiz/O/r5GwMOQ++HVnVGXSfYc4B0+XS/xqg X-Gm-Gg: AR+sD11KBKP6pMr3Q6ivngUQmAUUzkOkmD5Xsc7h5L2bH3HrQeidBaZl3qpr8dZUzkB P3eVpQTknt6P96sdvz89kn+2KqwhOai4hrmRPJmxwS9QK1Cigvd1YF00oxLpMmyCzzAqu/gRMjK xYEWJ9HC13fibMU6+NCZbzxbbIr2WN0t4LCtjlAoefCAzsAgLOIISXJNS4eNlzahXr0IQQK4mO5 +ME9Xdgw3MwFocHIX3mDia8saVgbZASj5I82AkziKvdIQYtyfwnjILbfMJlaLwG0cxDYDLyGrGk fKjPVbhnC+r8pnN3toiMdQUkjao5gvFxxoAKfmfLbRAOmT2YH8vREl6f7WGJ6RRwpE2pMAxBDr1 LB2HAs8EMX3oeb+OGjmPKjJm10ILH54afgKJj030YDoPdcyEQSysZolruP8kUF60LIjMdstaNNf 94ZfxudoPI21FC3eqOwS0h7ftDxoTjAyAUWCCk6p5b/kCNJAwKqay2cJHdlC4Z2IRBy3zSX1Owo esRx+6tDMD5q5Ciz0KnHHAWZhJzNOX9jG4MbY3riBaRXd3B X-Received: by 2002:a17:90b:4d8e:b0:38e:3a8:2374 with SMTP id 98e67ed59e1d1-38fb149bb44mr351802a91.30.1785468258030; Thu, 30 Jul 2026 20:24:18 -0700 (PDT) Received: from localhost.localdomain ([202.8.105.119]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2d047c42b4bsm869275ad.78.2026.07.30.20.24.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 30 Jul 2026 20:24:17 -0700 (PDT) From: Sun Jian To: netdev@vger.kernel.org Cc: Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend , Stanislav Fomichev , Kuniyuki Iwashima , Hangbin Liu , Krishna Kumar , Samiullah Khawaja , Martin Karsten , Lorenzo Bianconi , =?UTF-8?q?Toke=20H=C3=B8iland-J=C3=B8rgensen?= , linux-kernel@vger.kernel.org, bpf@vger.kernel.org, Sun Jian , maciej.fijalkowski@intel.com, stable@vger.kernel.org Subject: [PATCH net v2 1/2] net: fix skb length accounting after generic XDP frag adjustment Date: Thu, 30 Jul 2026 20:23:56 -0700 Message-ID: <20260731032357.6114-2-sun.jian.kdev@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260731032357.6114-1-sun.jian.kdev@gmail.com> References: <20260731032357.6114-1-sun.jian.kdev@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Generic XDP exposes non-linear skb fragments through an xdp_buff. If an XDP program adjusts the fragment area, bpf_prog_run_generic_xdp() copies xdp_frags_size back to skb->data_len but leaves skb->len containing the old fragment contribution. After a fragment shrink, this makes skb_headlen() larger than the actual linear area. In the reproduced UDP receive path, __skb_datagram_iter() copied 1024 bytes past the actual linear tail to userspace, starting at struct skb_shared_info. The copied bytes included the affected skb's nr_frags, xdp_frags_size and a kernel pointer from skb_shinfo(skb)->frags[0]. Real packet data was displaced by the same amount and truncated at the end. Subtract the old data_len before replacing it and add the new data_len afterwards, keeping skb->len and skb->data_len synchronized. A 60000-byte UDP datagram on a veth pair with MTU 64000 was shortened by 1024 bytes from its fragment area. Before the fix, all 10 runs produced corrupted payloads. After the fix, all 10 runs matched the expected payload exactly. Fixes: e6d5dbdd20aa ("xdp: add multi-buff support for xdp running in generic mode") Cc: stable@vger.kernel.org Link: https://lore.kernel.org/bpf/al9T9Eto%2FhRIzP5W@boxer/ Signed-off-by: Sun Jian --- net/core/dev.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index 5933c5dab09e..5c37cf6c4aa1 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -5517,12 +5517,16 @@ u32 bpf_prog_run_generic_xdp(struct sk_buff *skb, struct xdp_buff *xdp, } /* XDP frag metadata (e.g. nr_frags) are updated in eBPF helpers - * (e.g. bpf_xdp_adjust_tail), we need to update data_len here. + * (e.g. bpf_xdp_adjust_tail). Remove the old fragment contribution + * from skb->len before updating data_len, then add the new one back. */ - if (xdp_buff_has_frags(xdp)) + skb->len -= skb->data_len; + if (xdp_buff_has_frags(xdp)) { skb->data_len = skb_shinfo(skb)->xdp_frags_size; - else + skb->len += skb->data_len; + } else { skb->data_len = 0; + } /* check if XDP changed eth hdr such SKB needs update */ eth = (struct ethhdr *)xdp->data; -- 2.43.0