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

smb: add smb.version keyword 5075 v3 #9451

Closed
wants to merge 4 commits into from

Conversation

jmtaylor90
Copy link
Contributor

Make sure these boxes are signed before submitting your Pull Request -- thank you.

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/5075

Describe changes:

  • used initial authors commits
  • removed unnecessary NULL checks
  • removed unnecessary debug related function calls

Provide values to any of the below to override the defaults.

To use a pull request use a branch name like pr/N where N is the
pull request number.

Alternatively, SV_BRANCH may also be a link to an
OISF/suricata-verify pull-request.

SV_REPO=
SV_BRANCH=https://github.com/OISF/suricata-verify/pull/1380
SU_REPO=
SU_BRANCH=
LIBHTP_REPO=
LIBHTP_BRANCH=

@jmtaylor90 jmtaylor90 changed the title Detect smb version 5075 v3 smb: add smb.version keyword 5075 v3 Sep 6, 2023
@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Merging #9451 (a9ad3bb) into master (185f605) will decrease coverage by 0.01%.
The diff coverage is 90.21%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9451      +/-   ##
==========================================
- Coverage   82.16%   82.16%   -0.01%     
==========================================
  Files         968      969       +1     
  Lines      274204   274288      +84     
==========================================
+ Hits       225302   225361      +59     
- Misses      48902    48927      +25     
Flag Coverage Δ
fuzzcorpus 64.06% <26.82%> (-0.03%) ⬇️
suricata-verify 60.87% <87.80%> (-0.02%) ⬇️
unittests 62.88% <36.95%> (-0.01%) ⬇️

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

@catenacyber
Copy link
Contributor

Looks like rebase was not clean enough as CI is red for commit check : we want Suricata to build at every commit to run git bisect

Could you fix this @jmtaylor90 ?

@jmtaylor90
Copy link
Contributor Author

Yeah, will take a look today and get this sorted. Thanks!

@jmtaylor90
Copy link
Contributor Author

@catenacyber should I add the change you made in #9462 here as well?

@catenacyber
Copy link
Contributor

should I add the change you made in #9462 here as well?

Not until it is merged, and you may get merged first ;-)

@jmtaylor90
Copy link
Contributor Author

continued in #9469

@jmtaylor90 jmtaylor90 closed this Sep 8, 2023
@jmtaylor90 jmtaylor90 deleted the detect-smb-version-5075-v3 branch September 8, 2023 16:08
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.

3 participants