-
Notifications
You must be signed in to change notification settings - Fork 899
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
dag: improve fuzzer #4200
dag: improve fuzzer #4200
Conversation
Signed-off-by: AdamKorcz <adam@adalogics.com>
@AdamKorcz you can run |
Signed-off-by: Philippe Scorsolini <p.scorsolini@gmail.com>
@AdamKorcz looks like the fuzzing is actually failing, I think it's related to the test code and not an finding from the fuzzing, could you have a look? |
Does this still exist in the Makefile? |
56f957d
to
9a1454a
Compare
Signed-off-by: AdamKorcz <adam@adalogics.com>
9a1454a
to
7591bb0
Compare
@AdamKorcz yes, maybe you'll have to init the build submodule first |
Co-authored-by: Philippe Scorsolini <p.scorsolini@gmail.com> Signed-off-by: AdamKorcz <44787359+AdamKorcz@users.noreply.github.com>
Description of your changes
Adds more calls to the DAG fuzzer to improve coverage.
I have:
make reviewable
to ensure this PR is ready for review.backport release-x.y
labels to auto-backport this PR if necessary.How has this code been tested
go test -fuzz=FuzzDag