-
Notifications
You must be signed in to change notification settings - Fork 255
SRLabs: Introduce fuzzing harness for pallet-domains #3717
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
base: main
Are you sure you want to change the base?
Conversation
staking fuzzing harness
🛡️ Immunefi PR ReviewsWe noticed that your project isn't set up for automatic code reviews. If you'd like this PR reviewed by the Immunefi team, you can request it manually using the link below: Once submitted, we'll take care of assigning a reviewer and follow up here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment @cursor review or bugbot run to trigger another review on this PR
…uzzer for 10 mins excluding build time
db89852 to
63d4275
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me as long as CI passes reliably after multiple runs.
(Since it's non-deterministic, one run isn't enough to show the outcome is stable.)
|
Output directory containing crashes saved has changed since I have updated the script to prebuilt the binary. |
|
Note: This PR should pass CI once we update it to a main that includes #3719. |
|
Should be good to review now. |
This PR introduces a fuzzing harness for pallet-domains, encoding invariants and valid user-processes when fuzzing for maximum impact.
This is port of the pr #3693 but added few more refactoring on top. I have preserved the commits before my changes as is so that reviewers have easier time reviewing remaining commits instead of all commits again.
Had to create a new PR since I do not have write access to that original PR fork.
Code contributor checklist: