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

Add ClusterFuzzLite set up #154

Merged
merged 4 commits into from
Jan 4, 2024

Conversation

DavidKorczynski
Copy link
Contributor

@DavidKorczynski DavidKorczynski commented Dec 21, 2023

Description

Add continuous fuzzing set up by way of ClusterFuzzLite. This will run the attached fuzzer for each PR such 180 seconds and if any issues are found these will be flagged.

Test Steps

To reproduce this by way of OSS-Fuzz (which ClusterFuzzLite will be using):

git clone https://github.com/google/oss-fuzz
git clone https://github.com/DavidKorczynski/coreJSON corejson
cd corejson
git checkout clusterfuzz-initial

# Build the fuzzers in .clusterfuzzlite
python3 ../oss-fuzz/infra/helper.py build_fuzzers --external $PWD

# Run the fuzzer for 10 seconds
python3 ../oss-fuzz/infra/helper.py run_fuzzer --external $PWD validate_fuzzer -- -max_total_time=10

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

This is adding essentially another test and not a feature.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Signed-off-by: David Korczynski <david@adalogics.com>
n9wxu
n9wxu previously approved these changes Dec 26, 2023
ActoryOu
ActoryOu previously approved these changes Dec 27, 2023
Copy link
Member

@ActoryOu ActoryOu left a comment

Choose a reason for hiding this comment

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

Really appreciate this change. Approve in advance.
One minor comment is - could you help to add the test steps in readme file to help people reproduce?

Signed-off-by: David Korczynski <david@adalogics.com>
@DavidKorczynski DavidKorczynski dismissed stale reviews from ActoryOu and n9wxu via 2cb3dd2 December 27, 2023 09:24
Signed-off-by: David Korczynski <david@adalogics.com>
@DavidKorczynski
Copy link
Contributor Author

One minor comment is - could you help to add the test steps in readme file to help people reproduce?

Yes, done!

Signed-off-by: David Korczynski <david@adalogics.com>
@ActoryOu ActoryOu merged commit 0423144 into FreeRTOS:main Jan 4, 2024
11 checks passed
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.

3 participants