-
Notifications
You must be signed in to change notification settings - Fork 274
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
Add support for LINUX_SLL2 #728
Add support for LINUX_SLL2 #728
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just a couple nits. Targeting 4.5 relase.
/* $Id$ */ | ||
|
||
/* | ||
* Copyright (c) 2001-2010 Aaron Turner <aturner at synfin dot net> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also need `* Copyright (c) 2013-2023 Fred Klassen - AppNeta
@@ -0,0 +1,30 @@ | |||
# $Id:$ | |||
# START OF: dlt_linuxsll2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need copyright stuff here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left out this one, since there's no Makefile.am
to copy+paste Copyright stuff from.
4f4a812
to
744466f
Compare
@btriller
|
Appears this is working. Nevermind.
|
744466f
to
ad31b98
Compare
I may need to undo the force push. History gets messed up if you cherry-pick or force push. Only merge and rebase are safe. |
I pulled this PR down to test it since I had a Linux cooked capture v2 which I needed to convert to Ethernet via tcprewrite (and my usual install was failing with the error obvserved in #727) However with this PR Build compiled, the usual Linux Cooked Capture (v1) to Ethernet conversion command produced a strange output file: I can mitigate this nicely if I force to user DLT, force to ethernet DTL and then supply the raw hex I want (source mac, dest mac, 0800 in hex with commas). I can't share the file but from what I can gleam by comparing the two different output files:
I am not sure if this problem is this PR specifically but since the base tcprewrite can't read SLLv2 I figured I would report it under this PR. (I can split this into another issue if needed.) Since I can't share my files I tried to run this same test on the example file provided in the feature request it also fails.
If you run the alt command I supplied it comes out nicely
Build and Platform Info
|
Is this PR still valid ? This could be a very great PLUS for this application :) |
I think there are still some things missing. IIRC I only implemented the tcpreplay part. |
Thanks @btriller .
|
I'll pull this into a working branch to preserve history, and see if I can fix this. |
PR #728 cleanup and completion of SLL v2 implementation
PR #728 cleanup and completion of SLL v2 implementation
Thank you so much @btriller for this patch! It works well! |
fixes #727