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

[core] Add a fuzzing regression test github action #674

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

ivmaykov
Copy link
Contributor

@ivmaykov ivmaykov commented Aug 1, 2023

While fuzzing subzero core before the nanopb 0.4.5 upgrade, the fuzzer found a bunch of inputs which triggered UBSAN.

This PR creates a regression test that runs the fuzzer with these inputs and verifies that none of them fail now that we're on nanopb 0.4.5.

If more bad inputs are discovered later, they can be added to this directory after the corresponding bugs are fixed.

While fuzzing subzero core before the nanopb 0.4.5 upgrade,
the fuzzer found a bunch of inputs which triggered UBSAN.

This PR creates a regression test that runs the fuzzer with these inputs
and verifies that none of them fail now that we're on nanopb 0.4.5.

If more bad inputs are discovered later, they can be added to this directory
after the corresponding bugs are fixed.
@ivmaykov ivmaykov force-pushed the fuzz-test-regression-action branch from de76935 to 41a3d26 Compare August 1, 2023 02:51
run: scripts/ubuntu_install_protobuf.sh

- name: Setup Clang
uses: egor-tensin/[email protected]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pin to sha?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if there is a way to do that for github actions? Also, none of our other workflows pin to a sha, they all use versions.

@ivmaykov ivmaykov merged commit f4f4d86 into square:master Aug 1, 2023
8 checks passed
@ivmaykov ivmaykov deleted the fuzz-test-regression-action branch August 1, 2023 16:22
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.

2 participants