Skip to content

ICMP support work for opte#369 #33

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.

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
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

FelixMcFelix
Copy link
Collaborator

@FelixMcFelix FelixMcFelix commented Apr 25, 2025

This PR defines additional ICMP and ICMP v6 message types for use within body transforms in OPTE -- primarily, those which can carry nested packets as their payloads. This includes a few Neighbor Discovery Protocol option classes.

To help handle ICMPv6/NDP options, the Ingot derive macro now allows for the var_len expression to be omitted (as in subparse packet definitions). This causes such headers to consume the remainder of the slice they are parsed from.

@FelixMcFelix FelixMcFelix marked this pull request as ready for review April 28, 2025 15:35
@FelixMcFelix FelixMcFelix requested a review from mkeeter April 28, 2025 16:57
pub const fn payload_is_packet(self) -> bool {
matches!(
self,
Self::DESTINATION_UNREACHABLE

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for covering the additional cases beyond DU here. Time exceeded will be important for traceroute to work properly.

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.

2 participants