Skip to content

[ndp] neighsol(): optional strict validation of IPv6 Hop Limit (==255) per RFC 4861 #4829

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
itchenfei wants to merge 10 commits into
base: master
Choose a base branch
from
Open

[ndp] neighsol(): optional strict validation of IPv6 Hop Limit (==255) per RFC 4861 #4829

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
0f62e2e
ndp: add optional strict HLIM==255 validation in neighsol() per RFC 4861
itchenfei Aug 27, 2025
d1794d6
ndp: neighsol(): use '==' in BPF filter for HLIM==255
itchenfei Aug 27, 2025
ea43bb1
fix: update neighsol() to use srp and add stop_filter for HLIM valida…
itchenfei Aug 27, 2025
bdeada0
fix: enhance neighsol() to return None for empty responses and valida…
itchenfei Aug 27, 2025
6efa1bb
fix: simplify HLIM validation in neighsol() by removing ICMPv6ND_NA c…
itchenfei Aug 27, 2025
c588598
refactor: convert neighsol() from srp to sniff-based implementation
itchenfei Aug 29, 2025
5238209
fix: delete unused import srp
itchenfei Aug 29, 2025
08dd8c2
fix: correct chainCC parameter usage in neighsol() sniff implementati…
itchenfei Aug 29, 2025
ce406ff
fix: Revert to srp1 with BPF filter
itchenfei Aug 30, 2025
d05c2fc
ICMPv6ND_NA: validate hlim==255 in answers() per RFC 4861
guedou Feb 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions scapy/layers/inet6.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2258,6 +2258,10 @@ def hashret(self):
return bytes_encode(self.tgt) + self.payload.hashret()

def answers(self, other):
if not isinstance(self.underlayer, IPv6):
return False
if self.underlayer.hlim != 255:
return False
return isinstance(other, ICMPv6ND_NS) and self.tgt == other.tgt

# associated possible options : target link-layer option, Redirected header
Expand Down