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 conf 3591 v2 #9567

Closed
wants to merge 1 commit into from
Closed

Conversation

catenacyber
Copy link
Contributor

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

Describe changes:

  • Adds a new fuzz target which takes config + signature + pcap as an entry

Draft : expecting feedback if this looks a good approach

Still to do :

  • Make FuzzConfReload call more than app-layer stuff

#9531 with more config values (ssh hassh, ssl ja3)

@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #9567 (e3c132d) into master (9157070) will decrease coverage by 0.02%.
The diff coverage is 56.62%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9567      +/-   ##
==========================================
- Coverage   82.20%   82.19%   -0.02%     
==========================================
  Files         968      968              
  Lines      274275   274318      +43     
==========================================
+ Hits       225461   225468       +7     
- Misses      48814    48850      +36     
Flag Coverage Δ
fuzzcorpus 64.16% <51.80%> (+0.10%) ⬆️
suricata-verify 60.93% <75.55%> (-0.02%) ⬇️
unittests 62.87% <48.88%> (-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 16258

@catenacyber catenacyber added the needs rebase Needs rebase to master label Nov 23, 2023
@catenacyber
Copy link
Contributor Author

Closing as stale, did not get feedback 😢 (and low priority versus fixing current bugs found by fuzzing anyways)

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
2 participants