Skip to content

LLT-7321: backport block packets with wrong src ip from entering tunnel - #1795

Draft
mathiaspeters wants to merge 3 commits into
release/v7.0from
LLT-7321_backport_block_wrong_src_ip
Draft

LLT-7321: backport block packets with wrong src ip from entering tunnel#1795
mathiaspeters wants to merge 3 commits into
release/v7.0from
LLT-7321_backport_block_wrong_src_ip

Conversation

@mathiaspeters

@mathiaspeters mathiaspeters commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Problem

Fix was merged to main, but we need to backport it to v7.0 releases

Solution

Do backport, but skip some unimportant commits to keep hotfix small. All skipped commits relate to natlab tests and are non-critical. Additionally, had to slightly adjust natlab tests to use standard setup_mesh_nodes and setup_connections instead of using the newly introduced factories, as well as creating the exit stack in the test instead of providing it as a fixture.

Skipped commits:

  • 7625b7a: Hardcode tunnel local ipv4 - 10.5.0.2
  • 370bef7: Add *.so files to .gitignore
  • 2e0bf2c: Add libfirewall pytest mark
  • 59139df: Fix tcpdump syntax error from unjoined filter predicates
  • e0064db: Fail fast in netcat helpers when process exits early

☑️ Definition of Done checklist

  • Commit history is clean (requirements)
  • README.md is updated
  • Functionality is covered by unit or integration tests

@mathiaspeters
mathiaspeters requested a review from a team as a code owner June 2, 2026 18:12
@tomaszpatejko

Copy link
Copy Markdown
Contributor

+1

lcruz99 added 2 commits June 3, 2026 15:27
When a VPN exit node is active, outbound packets whose source IP
is not one of the known tunnel addresses must be rejected. Pre-VPN
sockets bound to a stale local IP would otherwise leak through the
tunnel and poison the exit node's NAT table, causing return traffic
to be misrouted.
@mathiaspeters
mathiaspeters force-pushed the LLT-7321_backport_block_wrong_src_ip branch from 39486d4 to c3a6bcb Compare June 3, 2026 13:27
@mathiaspeters
mathiaspeters marked this pull request as draft June 4, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants