Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Ci fuzz serde v8 #11764

Closed
wants to merge 2 commits into from
Closed

Conversation

catenacyber
Copy link
Contributor

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

Describe changes:

  • ci: move from cifuzz to clusterfuzzlite to better support fuzzing on main7
  • rust: revert update to serde (because oss-fuzz uses an old nightly compiler)

#11757 with no wrong changes to Cargo.toml

for oss-fuzz/cifuzz compatibility
As they use an old nightly compiler
To better support main7 CI fuzzing
@catenacyber catenacyber requested review from jasonish and a team as code owners September 12, 2024 14:33
This was referenced Sep 12, 2024
Copy link

codecov bot commented Sep 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.54%. Comparing base (31bed10) to head (506f256).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11764      +/-   ##
==========================================
- Coverage   82.62%   82.54%   -0.09%     
==========================================
  Files         919      919              
  Lines      248979   248979              
==========================================
- Hits       205722   205515     -207     
- Misses      43257    43464     +207     
Flag Coverage Δ
fuzzcorpus 60.32% <ø> (-0.54%) ⬇️
livemode 18.71% <ø> (+<0.01%) ⬆️
pcap 44.14% <ø> (+<0.01%) ⬆️
suricata-verify 61.90% <ø> (+0.01%) ⬆️
unittests 58.99% <ø> (+<0.01%) ⬆️

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

@catenacyber
Copy link
Contributor Author

Continued in #11779

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant