Skip to content

Conversation

@KelvinOmar98
Copy link

Changes:

  • Prevent IP.summary() from crashing on randomized destinations
  • Add UTScapy regression test covering this case

Fix #4883

@guedou guedou requested review from Copilot and removed request for Copilot January 6, 2026 10:48
@guedou
Copy link
Member

guedou commented Jan 6, 2026

Good catch!

@guedou guedou self-requested a review January 6, 2026 10:48
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.79%. Comparing base (40fc5ec) to head (140ba64).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4896      +/-   ##
==========================================
- Coverage   80.85%   80.79%   -0.06%     
==========================================
  Files         369      351      -18     
  Lines       90961    90682     -279     
==========================================
- Hits        73542    73270     -272     
+ Misses      17419    17412       -7     
Files with missing lines Coverage Δ
scapy/layers/inet.py 71.92% <100.00%> (+0.01%) ⬆️

... and 282 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@guedou guedou merged commit a5bc2bb into secdev:master Jan 6, 2026
30 checks passed
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.

TypeError: unhashable type: 'RandIP' when running pkt.summary() when pkt IP packet has RandIP

3 participants