Skip to content

fix(internet): a cut leaving only the type octet of a CALIPSO, SMF_DPD or MPL option logs a FieldValueError #2727

Closed

Description

@JarryShaw

Found by the Sonnet review of the #2694 fix. The same behaviour is on main at 01f063467, so this is not a regression.

A capture cut that leaves only the type octet of a CALIPSO, SMF_DPD or MPL option in HOPOPT or IPv6-Opts logs FieldValueError: … invalid <X> option length: 0. Inside IPv6 it logs at CRITICAL, plus a WARNING, though the layer is kept. Their length callbacks in the IPv6 option schemas reject the zero-filled length. A cut one octet later is clean.

Under the short-read ruling, a cut should be recorded quietly through Schema's built-in, so a cut at 3 octets should behave like a cut at 4. The pins in test_ipv6_option_helpers_1519_unit, test_ipv6_extension_unit and test_dead_branches_2318_unit hold the current error and will move with the fix.

This is blocked until #2694 merges, because it reworks the same option areas.

Activity

  1. added this to the PyPCAPKit v1.6 milestone on Nov 3, 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 3, 2026
  3. moved this to Blocked in PyPCAPKiton Nov 3, 2026
  4. JarryShaw commented on Nov 3, 2026

    @JarryShaw
    OwnerAuthor

    #2694 (#2731) has merged as ab7be2fb0, so this is unblocked. A worker goes on it next. 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 3, 2026
  6. moved this from Blocked to WIP in PyPCAPKiton Nov 3, 2026
  7. JarryShaw commented on Nov 3, 2026

    @JarryShaw
    OwnerAuthor

    The PR is #2737.

  8. 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

    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