From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1794273718; b=sRpFwngyi5NB062qLdJTq3p1EALyNK+TGObwPEz1WZ4PJm2r5WtK5uiF5rbXM17m0PAE EPpm4WqKB/H5sB1uSpRDemfmvrt9f9Fp7+BqxxRC1d2m8x+ORlLZMOokx2IvO49O7oCka 1GOBYtKBfZyik4W3LV+L3GNvKfUtdJziRrw0+1y8v4KyZyco0axRXJKKNcabY7Gwnfyh6 hFTB5VIeth35EsCfIlJPaK1EnqTP7pEjNHS50y64HL1k8Fl0Oa4TtxeDhc+3RLcPNpYdV q51V4z1+hi3MD8AQ1hzWFtxjsNOyvl7lAnCGUnCYvNTLPql5omKOfmMOk42dO3RN//A== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1794273718; h=Received:DKIM-Signature:From:To:Cc:Subject:Date:Message-ID:X-Mailer: MIME-Version:Content-Type:Content-Transfer-Encoding; bh=5PxlOgGx+Ud4oNJy7EHYMB+HyReKdWfCyMbnmfETT34=; b=gtz7SpjDjO5bF8mTGRzEhVWDbfs83g52HhKR+Fo3pWHm2iNGiDLPUQohGFO4lehkH99f rEuhaLQiFAwrJF+y6yTgLdjosDsne3NG/RplSIJx5M/SJEA49HG8JPgYAhzrEP9YrGoEO XEb9NxGh4P8RZmKrMBSHVMr3YM0A0I6hF2G44Pfco+qZJKlzoAW9tXVxl7qa3defDdamn wIzEi/cnni5wiq5ltDJP8qu4wEcB9Cwn0in4KE+jrdfk4Dc4AcWzunEUdmc80ycSY+xxu UDpQAfHWj4dhp60/iQHPyoRdjDHhuzuFy4qvon62YnfHGcyMVppIMkwxIOO3ryEIDfQ== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=QaSIC3GQ; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8ECF860A263E for ; Tue, 10 Nov 2026 01:21:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=QaSIC3GQ Received: by smtp.kernel.org (Postfix) with ESMTPSA id 79CD51F008BD; Tue, 10 Nov 2026 01:21:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794273718; bh=5PxlOgGx+Ud4oNJy7EHYMB+HyReKdWfCyMbnmfETT34=; h=From:To:Cc:Subject:Date; b=QaSIC3GQYJSRf3dRSTy8R0h7BoBegkT8NZy6bDQlq8ywjWrQlWM031+b9RFM2LG+j gAyGSas1TvjgD4GcfyUTsQrU/Kcxr6hyYEMw3EvcEPPOcymsPq0b4yo+4+RXAeibHj cIvbzA9Pod0rDFOoxN9uPjKV1ZCxJ4S15t4DUY+G9uWX8f34vbAXu9iwW3CK1gEnVr eQITHhpjHxyiDmdhU+68TQc/XDlxwZzcpEq5L/eV5rXEdvMTobKaKf+OWngDLrPavB 1ChdJIwMpTA6EaO+SglPs+S5AJK6FUcH+6Nh8z//qlmMS6xI9tUEdRFujb6xpwTVJC sXduQtX8KS6tw== From: Sasha Levin To: edumazet@kernel.org, kylebot@openai.com, fw@strlen.de, idosch@nvidia.com, jiayuan.chen@linux.dev, wang.zhan@smartx.com, pabeni@redhat.com Cc: stable@vger.kernel.org Subject: FAILED: patch "[PATCH] inet: frags: limit the truesize of a reassembly queue" failed to apply to 5.15-stable tree Date: Mon, 9 Nov 2026 20:21:56 -0500 Message-ID: <20261109202156-failed-5.15-3ae20e4697eb@kernel.org> X-Mailer: git-send-email 2.53.0 X-Mailing-List: stable@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The patch below does not apply to the 5.15-stable tree. If someone wants it applied there, or to any other stable or longterm tree, then please email the backport, including the original git commit id to . To reproduce the conflict and resubmit, you may use the following commands: git fetch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ linux-5.15.y git checkout FETCH_HEAD git cherry-pick -x 3ae20e4697eb969104fe4b1e98a675f0100382b6 # git commit -s git send-email --to '' --in-reply-to '20261109202156-failed-5.15-3ae20e4697eb@kernel.org' --subject-prefix 'PATCH 5.15.y' 'HEAD^..' Thanks, Sasha ------------------ original commit in Linus's tree ------------------ >From 3ae20e4697eb969104fe4b1e98a675f0100382b6 Mon Sep 17 00:00:00 2001 From: Eric Dumazet Date: Fri, 30 Oct 2026 09:12:25 +0100 Subject: [PATCH] inet: frags: limit the truesize of a reassembly queue The truesize of fragments in a reassembly queue is only bounded by the per netns ipfrag_high_thresh sysctl, which has no upper limit. inet_frag_reasm_finish() sums the truesize of all fragments in an int. If this sum reaches 2^32, it wraps, and the final sk_wmem_alloc adjustment of the socket owning the reassembled skb can cancel its own reference, leading to a use-after-free of this socket. This can be triggered from an unprivileged user/network namespace, raising ipfrag_high_thresh and using padded TUN packets, which keep a large truesize after being trimmed by ip_rcv_core(). Track the sum of the truesize of fragments in each queue, and refuse a fragment that would make it bigger than INT_MAX / 2. Callers kill the queue on this error. This bound is far above the needs of legitimate traffic. Fixes: 18685451fc4e ("inet: inet_defrag: prevent sk release while still in use") Cc: stable@vger.kernel.org Reported-by: Kyle Zeng Cc: Florian Westphal Signed-off-by: Eric Dumazet Reviewed-by: Ido Schimmel Reviewed-by: Jiayuan Chen Tested-by: Wang Zhan Link: https://patch.msgid.link/20261030081225.2575462-1-edumazet@kernel.org Signed-off-by: Paolo Abeni --- include/net/inet_frag.h | 9 +++++++++ net/ipv4/inet_fragment.c | 5 +++++ 2 files changed, 14 insertions(+) diff --git a/include/net/inet_frag.h b/include/net/inet_frag.h index b02d2472b003e..c1fb419a9cf03 100644 --- a/include/net/inet_frag.h +++ b/include/net/inet_frag.h @@ -79,6 +79,7 @@ struct frag_v6_compare_key { * @tstamp_type: stamp has a mono delivery time (EDT) * @flags: fragment queue flags * @max_size: maximum received fragment size + * @truesize: sum of the truesize of queued fragments * @fqdir: pointer to struct fqdir * @rcu: rcu head for freeing deferall */ @@ -100,6 +101,7 @@ struct inet_frag_queue { u8 tstamp_type; __u8 flags; u16 max_size; + unsigned int truesize; struct fqdir *fqdir; struct rcu_head rcu; }; @@ -172,6 +174,13 @@ extern const u8 ip_frag_ecn_table[16]; #define IPFRAG_DUP 1 #define IPFRAG_OVERLAP 2 #define IPFRAG_MIXED_SECPATH 3 +#define IPFRAG_TOOBIG 4 + +/* Max sum of the truesize of fragments in a queue. This keeps the sums + * done in inet_frag_reasm_prepare() and inet_frag_reasm_finish() well + * below INT_MAX. Legitimate traffic stays far below this limit. + */ +#define INET_FRAG_MAX_TRUESIZE (INT_MAX / 2) int inet_frag_queue_insert(struct inet_frag_queue *q, struct sk_buff *skb, int offset, int end); void *inet_frag_reasm_prepare(struct inet_frag_queue *q, struct sk_buff *skb, diff --git a/net/ipv4/inet_fragment.c b/net/ipv4/inet_fragment.c index c18da91c1f031..0b8d22ffafca6 100644 --- a/net/ipv4/inet_fragment.c +++ b/net/ipv4/inet_fragment.c @@ -336,6 +336,7 @@ void inet_frag_queue_flush(struct inet_frag_queue *q, q->rb_fragments = RB_ROOT; q->fragments_tail = NULL; q->last_run_head = NULL; + q->truesize = 0; } EXPORT_SYMBOL(inet_frag_queue_flush); @@ -446,6 +447,9 @@ int inet_frag_queue_insert(struct inet_frag_queue *q, struct sk_buff *skb, if (last && !secpath_equal(last, skb)) return IPFRAG_MIXED_SECPATH; + if (skb->truesize > INET_FRAG_MAX_TRUESIZE - q->truesize) + return IPFRAG_TOOBIG; + /* An IP fragment is never a GSO packet, but an untrusted source * (virtio_net_hdr) may have attached GSO metadata to it. Do not let * that reach the reassembled skb, whose head keeps the first @@ -510,6 +514,7 @@ int inet_frag_queue_insert(struct inet_frag_queue *q, struct sk_buff *skb, if (offset) nf_reset_ct(skb); + q->truesize += skb->truesize; return IPFRAG_OK; } EXPORT_SYMBOL(inet_frag_queue_insert); -- 2.53.0