From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D1313521222 for ; Tue, 8 Sep 2026 22:54:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788908052; cv=none; b=AU5hGdsskY2OaAX+kF4pHZwD7X0GEwNV9cRMi4UtdxJMVawmAMU5bnAXwlRhbviBFTocWdFpanTXLFJOxKZ4aXaEIXw9qZGk6uyGURoWJSGxWHsPSCAWmUHJSF6vN1umPOvGUrdjwfQPigzreo3hjM5p2dmyUVMKFIonJbz5xoM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788908052; c=relaxed/simple; bh=P4pzRqX9+JiI+AdjvA0X0fHe/uENMGgcd93q8wEoCpk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=eK3lD16ne2trrDgAQAT8XGzaxAD9vON73wPsPVMBV/KMhHav4KE1cwt9iDBnWr2p+SevWL32AsX9SkTG8933ljUvFlz5jSZVPx20IVgNx1I287MSHW51TvoHosdqIClgUR9gtKDzJzboyg7kV2kd+f/jCIUoXyQYgiIqr+SPWzc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=gKQfx1Im; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="gKQfx1Im" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4804F1F00A3F; Tue, 8 Sep 2026 22:54:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1788908050; bh=sfMJPRdH5JWN0GgCB8QRkD9u/i036gsKf9rk+j/rNXA=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=gKQfx1Imn4IqrY7Q2KXFpyWROj41Z/+klxDb1aNdPwfiIT/6FYuhubhZMrlqxnXFA EVY1+ImaGyuPbIHduT3JDuc3n0xmCowd2MxPA6EOgYmR6lUELYW8ifqOpdvcU64RTM YXBlLqCiwrj2cESqo6Cf7MEri4n14oy2XC/VkZxNM16cvsTWZETRrbbSjLpyGSommm mKS7AVDioqe+kH9ZeZhJD3QgmPVEM2KiIrI768jpZyjVS2Zx6Y/0zOJDxdfWF+2rFV e+ygmp6Np1kcrddZ1pYq40BBlrcUtDuVbd06d4gHQNVH2XJXIKJJ2bUMXoH5YdmLri tk6cy2el53xkg== From: Sasha Levin To: stable@vger.kernel.org Cc: Eric Dumazet , Paolo Abeni , Sasha Levin Subject: [PATCH 5.15.y 2/3] ipv4: frags: remove ipq_put() Date: Tue, 8 Sep 2026 18:54:07 -0400 Message-ID: <20260908225408.538794-2-sashal@kernel.org> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260908225408.538794-1-sashal@kernel.org> References: <2026090319-antiques-palm-c286@gregkh> <20260908225408.538794-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Eric Dumazet [ Upstream commit a2fb987c0ecf0498cc17056339cb11d128c46ab7 ] Replace ipq_put() with inet_frag_putn() Signed-off-by: Eric Dumazet Link: https://patch.msgid.link/20250312082250.1803501-3-edumazet@google.com Signed-off-by: Paolo Abeni For the stable backport, retain the existing ip_expire() error and skb freeing paths. Also brace the multicast delivery conditional so that the dependent fix can add its non-local skb orphaning without conflict. Stable-dep-of: e36ce6e78fe3 ("ip: orphan prefetched skbs before multicast forwarding") Signed-off-by: Sasha Levin --- net/ipv4/ip_fragment.c | 15 ++++----------- net/ipv6/ip6_input.c | 4 ++-- 2 files changed, 6 insertions(+), 13 deletions(-) diff --git a/net/ipv4/ip_fragment.c b/net/ipv4/ip_fragment.c index 11dedc82b98da..34e85c552ac36 100644 --- a/net/ipv4/ip_fragment.c +++ b/net/ipv4/ip_fragment.c @@ -102,14 +102,6 @@ static void ip4_frag_free(struct inet_frag_queue *q) inet_putpeer(qp->peer); } - -/* Destruction primitives. */ - -static void ipq_put(struct ipq *ipq) -{ - inet_frag_putn(&ipq->q, 1); -} - /* Kill ipq entry. It is not destroyed immediately, * because caller (and someone more) holds reference count. */ @@ -138,6 +130,7 @@ static void ip_expire(struct timer_list *t) struct net *net; struct ipq *qp; int err; + int refs = 1; qp = container_of(frag, struct ipq, q); net = qp->q.fqdir->net; @@ -195,7 +188,7 @@ static void ip_expire(struct timer_list *t) out_rcu_unlock: rcu_read_unlock(); kfree_skb(head); - ipq_put(qp); + inet_frag_putn(&qp->q, refs); } /* Find the correct entry in the "incomplete datagrams" queue for @@ -484,14 +477,14 @@ int ip_defrag(struct net *net, struct sk_buff *skb, u32 user) /* Lookup (or create) queue header */ qp = ip_find(net, ip_hdr(skb), user, vif); if (qp) { - int ret; + int ret, refs = 1; spin_lock(&qp->q.lock); ret = ip_frag_queue(qp, skb); spin_unlock(&qp->q.lock); - ipq_put(qp); + inet_frag_putn(&qp->q, refs); return ret; } diff --git a/net/ipv6/ip6_input.c b/net/ipv6/ip6_input.c index cf577cc3b48cf..94d2ab8689d88 100644 --- a/net/ipv6/ip6_input.c +++ b/net/ipv6/ip6_input.c @@ -552,9 +552,9 @@ int ip6_mc_input(struct sk_buff *skb) /* unknown RA - process it normally */ } - if (deliver) + if (deliver) { skb2 = skb_clone(skb, GFP_ATOMIC); - else { + } else { skb2 = skb; skb = NULL; } -- 2.53.0