tests: add new fuzzer #2965
Merged
tests: add new fuzzer #2965
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: David Korczynski <[email protected]>
DavidKorczynski
force-pushed
the
add-fixed-fuzzer
branch
from
8c75ea2 to
7abfdc5
Compare
|
Thank you! |
AdamKorcz
pushed a commit
to google/oss-fuzz
that referenced
this pull request
Dec 7, 2023
wader
reviewed
Dec 7, 2023
| "\"\") ) else . end )", | ||
| "[while(.<100; .*2)]", | ||
| "with_entries(.key |= \"KEY_\" + .)", | ||
| ". - [\"xml\", \"yaml\"]", |
There was a problem hiding this comment.
Are these generated from documentation or test? maybe could be?
There was a problem hiding this comment.
from the tests/*.test. This one specifically is:
Line 170 in 5029328
Specifically, it was generated using a simple scraping "capture each line that is after an empty line in some of the .test files" with a bit of sorting etc.
Please feel free to add any new ones as well! The more the merrier, and also it's not a big deal (for the fuzzer at least) if some of them are not valid.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds a new fuzzer that is similar in nature to
jq_fuzz_execute. The difference is that thejq_fuzz_executewill seed both thejqprogram and input with fuzz data wheras this newjq_fuzz_fixedfuzzer has a predefined list ofjqprogram and only the input will be seeded from the fuzzer. The benefit is that this fuzzer is closer to identifying issues that may happen in "generic jq programs" and much faster at exploring certain parts of the codebase. It does explore a subset of thejq_fuzz_execute, however, there is plenty of code to explore and this will be much more efficient at targeting some (relevant) parts of it.The
jq_progswas taken from the/tests/*.testfiles.From a local run, this drastically improves code coverage.