Add SAE J1939 protocol support to Scapy contrib#4963
Add SAE J1939 protocol support to Scapy contrib#4963polybassa wants to merge 8 commits intosecdev:masterfrom
Conversation
|
@BenGardiner Please review. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4963 +/- ##
==========================================
+ Coverage 79.51% 80.27% +0.76%
==========================================
Files 381 383 +2
Lines 93716 94595 +879
==========================================
+ Hits 74521 75939 +1418
+ Misses 19195 18656 -539
🚀 New features to boost your workflow:
|
BenGardiner
left a comment
There was a problem hiding this comment.
You're gonna need a _Soft_J1939Socket too I think
22833a7 to
def7ba4
Compare
|
no soft socket? 😢 A lot of truck analysis happens on a windows platform because that is where it is easiest to use the RP1210 vehicle diagnostic adapters |
|
no worries. this will be the next PR in this series for J1939. I just want to keep the PRs somewhat "small". Next step is SoftSocket, followed by a J1939 JsonDB Parser. I'll fetch your changes from your open PR to the the SoftSocket. |
…, tighten ancillary checks, and extend interoperability tests
…rtions, and streamline subprocess communication
…h and clarify struct size details
…for TP.CM, RTS, CTS, ACK, and ABORT scenarios
… J1939_TP_CM, including dispatch hooks, field validations, and filter functionality
…tination addressing in send functions, and adjust thread handling for sniffing
|
@gpotter2 This PR is ready from my side. Any feedback? |
gpotter2
left a comment
There was a problem hiding this comment.
A minor comment, otherwise LGTM.
Thanks
| # --------------------------------------------------------------------------- | ||
|
|
||
| #: 64-bit NAME not used / not relevant | ||
| J1939_NO_NAME = getattr(socket, 'J1939_NO_NAME', 0) # 0 |
There was a problem hiding this comment.
Do not hesitate to set socket.XXX if it doesn't exist. It makes it easier to remove the compatibility when eventually we'll reach EOL, and we've been doing that elsewhere (mostly in Windows code)
No description provided.