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

fuzz: adds structure aware target #5818

Closed
wants to merge 1 commit into from

Conversation

catenacyber
Copy link
Contributor

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

Describe changes:

  • adds structure aware fuzz target (aka fuzz suricata, not libpcap)

The point is to have a more efficient fuzzing discovering more code coverage and finding more bugs

To do so, it uses a fuzz-friendly format instead of libpcap
To test it you need the fuzzpcap dependency which can generate either generic pcap, or pcaps with only one tcp stream (computing sequence and acks automatically)
There is also a signature generation in this target, meant to produce more valid signatures.

Will work with oss-fuzz with https://github.com/catenacyber/oss-fuzz/tree/suricataware

so as not to fuzz libpcap
and generate structure aware signatures
@catenacyber catenacyber requested a review from a team as a code owner February 5, 2021 10:27
@codecov
Copy link

codecov bot commented Feb 5, 2021

Codecov Report

Merging #5818 (36333b2) into master (62e665c) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5818   +/-   ##
=======================================
  Coverage   72.38%   72.39%           
=======================================
  Files         604      604           
  Lines      179369   179370    +1     
=======================================
+ Hits       129837   129846    +9     
+ Misses      49532    49524    -8     
Flag Coverage Δ
suricata-verify 49.15% <ø> (+<0.01%) ⬆️
unittests 63.07% <ø> (ø)

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

@victorjulien victorjulien added the needs rebase Needs rebase to master label Mar 1, 2021
@catenacyber
Copy link
Contributor Author

Replaced by #5965

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs rebase Needs rebase to master
Development

Successfully merging this pull request may close these issues.

2 participants