Skip to content

fix(protocols): malformed MH messages and MPTCP DSS options do not rebuild as captured #2669

Closed

Description

@JarryShaw

Some complete but malformed MH messages and TCP MPTCP DSS options do not rebuild as captured. One example is an MH Home_Test whose length field is too small: it rebuilds as 16 of its 24 octets. The owner ruled that every parse→rebuild round trip must match byte for byte. The #2660 worker found this through a one-octet mutation search over complete packets. It has not been reproduced independently, and #2660 did not change it.

To do:

  • Reproduce on main with one-octet mutations of complete MH messages and of TCP options carrying MPTCP DSS.
  • Find why a length field shorter than the octets present loses the remainder.
  • Fix it so the rebuild writes the octets as captured, with the extra octets kept where the schema keeps trailing data.

Blocked on #2614's transport and internet PRs, which own tcp.py and mh.py. Unblocks when both merge.

Activity

  1. added this to the PyPCAPKit v1.6 milestone on Nov 2, 2026
  2. added
    bugIssues reporting a defect (set by the bug report template; a default, not an assessment)
    blockedDeferred pending another issue or decision; see the last comment for what unblocks it
    on Nov 2, 2026
  3. moved this to Blocked in PyPCAPKiton Nov 2, 2026
  4. JarryShaw commented on Nov 2, 2026

    @JarryShaw
    OwnerAuthor

    #2677 and #2679 have merged, so this is unblocked. A worker is on it now. Moved from blocked to wip.

  5. added
    wipWork in flight - a covering PR is open or an agent is actively on it
    and removed
    blockedDeferred pending another issue or decision; see the last comment for what unblocks it
    on Nov 2, 2026
  6. moved this from Blocked to WIP in PyPCAPKiton Nov 2, 2026
  7. JarryShaw commented on Nov 2, 2026

    @JarryShaw
    OwnerAuthor

    The PR is #2683. A Sonnet review is running, and it is also checking the mobility-option and cut-MH side findings before I file them.

  8. removed
    wipWork in flight - a covering PR is open or an agent is actively on it
    on Nov 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugIssues reporting a defect (set by the bug report template; a default, not an assessment)

    Projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions