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

clusterfuzzlite support #87

Merged
merged 7 commits into from
Oct 4, 2023
Merged

clusterfuzzlite support #87

merged 7 commits into from
Oct 4, 2023

Conversation

jmartins-ledger
Copy link
Contributor

@jmartins-ledger jmartins-ledger commented Jun 26, 2023

  • Fixed boilerplate fuzzer
  • Added clusterfuzzlite support
  • PR fuzzing for address/undefined/memory sanitizers (5 minutes)
  • Batch fuzzing for address/undefined/memory sanitizers + Corpus prunning + code coverage report (5 minutes, At 12:00 PM, only on Saturday)

@xchapron-ledger
Copy link
Contributor

@jmartins-ledger Thanks! I'm wondering, is there a way to factorize the 3 differents fuzz workflows? They looks very alike.
Also I'm wondering if there is a way to avoid the https://github.com/LedgerHQ/app-boilerplate-fuzz-storage repo? I'm not sure to understand how it will work to push artifacts? Is there a way these artifacts can be pushed as a branch on this repo or somethink like that?

@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (5c618a7) 80.43% compared to head (50b8186) 80.43%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #87   +/-   ##
=======================================
  Coverage   80.43%   80.43%           
=======================================
  Files           3        3           
  Lines          46       46           
=======================================
  Hits           37       37           
  Misses          9        9           
Flag Coverage Δ
unittests 80.43% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jmartins-ledger
Copy link
Contributor Author

@xchapron-ledger I merged two of them into one, since they act like cron tasks. Right now there are 2 workflows, one for cron tasks and one for PR's. I removed the external repo dependency, the artifacts can be downloaded in the github action "menu". So each fuzzing campaign takes 5min.

.github/workflows/cflite_cron.yml Show resolved Hide resolved
.clusterfuzzlite/Dockerfile Outdated Show resolved Hide resolved
fuzzing/README.md Outdated Show resolved Hide resolved
@cedelavergne-ledger cedelavergne-ledger merged commit 93adb68 into master Oct 4, 2023
38 checks passed
@cedelavergne-ledger cedelavergne-ledger deleted the cflite-support branch October 4, 2023 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants