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

decode/vntag: Add decoder for 802.1Qbh #6083

Closed
wants to merge 6 commits into from

Conversation

jlucovsky
Copy link
Contributor

Continuation of #6081

This PR adds support for decoding VNTAG packets (https://www.ieee802.org/1/files/public/docs2009/new-pelissier-vntag-seminar-0508.pdf)

Describe changes:

  • Correct build issue in unittest commit

suricata-verify-pr: 489
#suricata-verify-repo:
#suricata-verify-branch:
#suricata-update-pr:
#suricata-update-repo:
#suricata-update-branch:
#libhtp-pr:
#libhtp-repo:
#libhtp-branch:

@codecov
Copy link

codecov bot commented Apr 25, 2021

Codecov Report

Merging #6083 (18e1830) into master (d4554ec) will increase coverage by 0.00%.
The diff coverage is 84.78%.

@@           Coverage Diff           @@
##           master    #6083   +/-   ##
=======================================
  Coverage   77.00%   77.00%           
=======================================
  Files         612      613    +1     
  Lines      187684   187727   +43     
=======================================
+ Hits       144530   144565   +35     
- Misses      43154    43162    +8     
Flag Coverage Δ
fuzzcorpus 53.03% <5.55%> (+<0.01%) ⬆️
suricata-verify 50.22% <58.82%> (-0.03%) ⬇️
unittests 63.27% <84.44%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link
Member

@victorjulien victorjulien left a comment

Choose a reason for hiding this comment

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

Looking good. Some minor changes requested

rules/decoder-events.rules Show resolved Hide resolved
rules/decoder-events.rules Show resolved Hide resolved
src/decode-vntag.c Show resolved Hide resolved
@@ -1289,6 +1292,9 @@ static inline bool DecodeNetworkLayer(ThreadVars *tv, DecodeThreadVars *dtv,
DecodeEthernet(tv, dtv, p, data, len);
}
break;
case ETHERNET_TYPE_VNTAG:
Copy link
Member

Choose a reason for hiding this comment

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

wonder why codecov claims no coverage here

@jlucovsky
Copy link
Contributor Author

Continued in #6085

@jlucovsky jlucovsky closed this Apr 26, 2021
@jlucovsky jlucovsky deleted the vntag/2 branch June 30, 2021 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants