Skip to content

v1.18 Backports 2026-01-22#43923

Merged
giorio94 merged 1 commit intov1.18from
pr/v1.18-backport-2026-01-22-12-27
Jan 22, 2026
Merged

v1.18 Backports 2026-01-22#43923
giorio94 merged 1 commit intov1.18from
pr/v1.18-backport-2026-01-22-12-27

Conversation

@giorio94
Copy link
Copy Markdown
Member

Once this PR is merged, a GitHub action will update the labels of these PRs:

 43868

[ upstream commit b7c4147 ]

The previous commit updating short ICMP error packet NAT
handling improperly computed the length of a short packet
by neglecting to account for the ihl field of the IP header
representing 32 bit words rather than bytes.  This caused
a reversion of #33844.

Furthermore, the change in short packet detection depends on
the incoming bpf_context structure having the "len" field set
correctly, which the BPF tests for that condition did not set.

This corrects the comparison for the inner L4 packet length
and updates the short ICMP error packet BPF tests to ensure
the ctx->len field is set like the kernel does.

Fixes: 1a018d56d623 ("bpf: Refine inner packet L4 checksum detection")

Signed-off-by: Bill Reese <ReeseW@computer.org>
Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 added kind/backports This PR provides functionality previously merged into master. backport/1.18 This PR represents a backport for Cilium 1.18.x of a PR that was merged to main. labels Jan 22, 2026
@giorio94
Copy link
Copy Markdown
Member Author

/test

@giorio94 giorio94 marked this pull request as ready for review January 22, 2026 13:13
@giorio94 giorio94 requested a review from a team as a code owner January 22, 2026 13:13
@giorio94 giorio94 enabled auto-merge January 22, 2026 13:13
@giorio94 giorio94 added this pull request to the merge queue Jan 22, 2026
Merged via the queue into v1.18 with commit e0cc2f8 Jan 22, 2026
315 of 317 checks passed
@giorio94 giorio94 deleted the pr/v1.18-backport-2026-01-22-12-27 branch January 22, 2026 14:07
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/1.18 This PR represents a backport for Cilium 1.18.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants