Skip to content
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

Extended SDP (J3400 RP) decoding support #64

Open
JeremyWhaling opened this issue May 8, 2024 · 2 comments
Open

Extended SDP (J3400 RP) decoding support #64

JeremyWhaling opened this issue May 8, 2024 · 2 comments

Comments

@JeremyWhaling
Copy link
Contributor

I'm starting work on adding support for extended SDP as defined in J3400 recommended practice. My plan is just to add to the SDP table and other various parts of the Lua script without any major structural changes. I wanted to point this out in case there's changes to the Lua script as part of the pending pull request (#62).

@chardin-cpi
Copy link
Collaborator

chardin-cpi commented May 10, 2024

time wise - will likely rebase the other PR on this change when it is ready - talked to @jhart-cpi about how we need to pass the grammar for 20 so we will need to modify the Lua but keep going with this change since we will try to avoid the conflicts as possible.

@JeremyWhaling
Copy link
Contributor Author

Sounds good. My main modifications are to the SDP table, adding some helper functions for things like inlet port type, and within the dissector function itself I am differentiating the logic from "regular" SDP by looking at payload length (so the sections under if payload_type_name == "SDP REQUEST" and "SDP RESPONSE" will grow). Unlike the previous "SDP with EMSP", the SDP header is the same (0x9000 / 0x9001), so this seems to be the only way to attack this (maybe something to bring up in committee).

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

No branches or pull requests

2 participants