Skip to content

Fix fragmentation cache#322

Open
TheSableCZ wants to merge 5 commits intomasterfrom
fix-fragmentation-cache
Open

Fix fragmentation cache#322
TheSableCZ wants to merge 5 commits intomasterfrom
fix-fragmentation-cache

Conversation

@TheSableCZ
Copy link
Contributor

@TheSableCZ TheSableCZ commented Feb 17, 2026

Summary

Several fixes related to fragmentation cache and parser issues.

Details

  • Fragmentation stats were incorrectly computed because recursion in put_pkt method.
  • The subsequent fragments (not the first ones) were filled with random ports because the parser continued parsing the L4 layer.
  • Packet data were not reset correctly in parser.

✅ Checks

  • Relevant documentation was updated (if needed)
  • CI pipeline passes (build, tests, lint, static analysis)
  • New code follows clang-tidy rules
  • The change is clearly related to a specific issue / requirement
  • Documentation (e.g. Doxygen) is updated if needed
  • The PR is rebased on the latest master
  • No unrelated changes are included

TheSableCZ and others added 5 commits February 11, 2026 17:24
This change allows extracting side-effect parts which must be called once.
Fixes some members not resetting.
Fixes inconsistent creation of fragmentation key and subsequent
incorrect indexing in fragmentation table which was caused by not
resetting ip addresses.
Fixes incorrect marking some packets as fragments which was caused by
not resetting frag_id and frag_off.
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.

1 participant

Comments