Skip to content

Commit 620d880

Browse files
wbowlingopsiff
authored andcommitted
net: skbuff: preserve shared-frag marker during coalescing
netdev inclusion category: bugfix CVE: CVE-2026-46300 skb_try_coalesce() can attach paged frags from @from to @to. If @from has SKBFL_SHARED_FRAG set, the resulting @to skb can contain the same externally-owned or page-cache-backed frags, but the shared-frag marker is currently lost. That breaks the invariant relied on by later in-place writers. In particular, ESP input checks skb_has_shared_frag() before deciding whether an uncloned nonlinear skb can skip skb_cow_data(). If TCP receive coalescing has moved shared frags into an unmarked skb, ESP can see skb_has_shared_frag() as false and decrypt in place over page-cache backed frags. Propagate SKBFL_SHARED_FRAG when skb_try_coalesce() transfers paged frags. The tailroom copy path does not need the marker because it copies bytes into @to's linear data rather than transferring frag descriptors. Fixes: cef401d ("net: fix possible wrong checksum generation") Fixes: f4c50a4034e6 ("xfrm: esp: avoid in-place decrypt on shared skb frags") Signed-off-by: William Bowling <vakzz@zellic.io> Reviewed-by: Eric Dumazet <edumazet@google.com> Tested-by: Jiayuan Chen <jiayuan.chen@linux.dev> Link: https://patch.msgid.link/20260513041635.1289541-1-vakzz@zellic.io Signed-off-by: Jakub Kicinski <kuba@kernel.org> (cherry picked from commit f84eca5817390257cef78013d0112481c503b4a3) Signed-off-by: Wentao Guan <guanwentao@uniontech.com> (cherry picked from commit f786284) Signed-off-by: Wentao Guan <guanwentao@uniontech.com>
1 parent 0f2becd commit 620d880

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

net/core/skbuff.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6155,6 +6155,8 @@ bool skb_try_coalesce(struct sk_buff *to, struct sk_buff *from,
61556155
from_shinfo->frags,
61566156
from_shinfo->nr_frags * sizeof(skb_frag_t));
61576157
to_shinfo->nr_frags += from_shinfo->nr_frags;
6158+
if (from_shinfo->nr_frags)
6159+
to_shinfo->flags |= from_shinfo->flags & SKBFL_SHARED_FRAG;
61586160

61596161
if (!skb_cloned(from))
61606162
from_shinfo->nr_frags = 0;

0 commit comments

Comments
 (0)