Skip to content

fix(protocols): range-check option-list codes in MH, IPv4, TCP, DCCP, SCTP and PCAP-NG makers #2693

Closed

Description

@JarryShaw

Left out of #2692, which settles registry codes in the main make() fields. Option-list codes are still handled inconsistently across families:

  • Range-checked: HOPOPT/IPv6-Opts option types, HIP parameter codes, TLS extension types and HTTP/2 settings.
  • Not range-checked, so the code wraps: MH, IPv4, TCP, DCCP, SCTP and PCAP-NG. For example, MH writes option code 300 as 0x2c.

Under #2617, a code that does not fit its field raises ProtocolError. Route these through the #2692 resolver.

Also from that work: MH _make_msg_be's status is typed StatusCode, but its schema holds BindingError.

blocked until #2692 merges, since it edits the same makers.

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)
    breakingBreaks public-facing behaviour or API (apply alongside the type label)
    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

    Folded into #2692's fix round: its review asked for option-list codes to be range-checked at every tuple-list option loop, as HOPOPT does. #2692 will close this issue. It stays blocked until then.

  5. added a commit that references this issue on Nov 3, 2026
    e7bc62c
  6. JarryShaw commented on Nov 3, 2026

    @JarryShaw
    OwnerAuthor

    This folds into #2692, whose fix round now range-checks option-list codes in every family. #2692 closes it.

  7. 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 3, 2026
  8. moved this from Blocked to WIP in PyPCAPKiton Nov 3, 2026
  9. added a commit that references this issue on Nov 3, 2026
    3d1dea6
  10. removed
    wipWork in flight - a covering PR is open or an agent is actively on it
    on Nov 3, 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

    breakingBreaks public-facing behaviour or API (apply alongside the type label)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