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

Detect integers 6444 v1 #10089

Closed
wants to merge 2 commits into from

Conversation

catenacyber
Copy link
Contributor

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/6645
https://redmine.openinfosecfoundation.org/issues/6646

Both in tracking ticket https://redmine.openinfosecfoundation.org/issues/6644 for integers as first-class detection

Describe changes:

  • detect/integers: support hexadecimal notation for parsing
  • detect/integers: add mode for negated range

Commits from #10073 and #10083 now with their own ticket into the a tracking ticket for integers as first-class detection

catenacyber and others added 2 commits December 22, 2023 09:11
So that we can write enip.revision: 0x203

Ticket: 6645
@catenacyber
Copy link
Contributor Author

Oops, I had forgotten to push latest version

Copy link

codecov bot commented Dec 22, 2023

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (5cc872f) 82.19% compared to head (aa0d56e) 82.20%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #10089   +/-   ##
=======================================
  Coverage   82.19%   82.20%           
=======================================
  Files         975      975           
  Lines      271940   271988   +48     
=======================================
+ Hits       223523   223581   +58     
+ Misses      48417    48407   -10     
Flag Coverage Δ
fuzzcorpus 62.94% <73.33%> (+0.02%) ⬆️
suricata-verify 61.41% <73.33%> (-0.01%) ⬇️
unittests 62.85% <89.28%> (+<0.01%) ⬆️

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

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 17176

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 17178

This was referenced Jan 2, 2024
@catenacyber
Copy link
Contributor Author

Replaced by #10105

@catenacyber catenacyber closed this Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants