Skip to content

Conversation

@vedhavyas
Copy link
Contributor

@vedhavyas vedhavyas commented Oct 27, 2025

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:

@immunefi-magnus
Copy link

🛡️ Immunefi PR Reviews

We 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:

🔗 Send this PR in for review

Once submitted, we'll take care of assigning a reviewer and follow up here.

Copy link

@cursor cursor bot left a 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

teor2345
teor2345 previously approved these changes Oct 27, 2025
Copy link
Member

@teor2345 teor2345 left a 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.)

@vedhavyas
Copy link
Contributor Author

Output directory containing crashes saved has changed since I have updated the script to prebuilt the binary.
Should be fixed now.
Previous run went fine and hopefully run will give all green

@vedhavyas vedhavyas requested a review from teor2345 October 27, 2025 06:43
@vedhavyas vedhavyas enabled auto-merge October 27, 2025 07:04
teor2345
teor2345 previously approved these changes Oct 27, 2025
@vedhavyas vedhavyas added this pull request to the merge queue Oct 27, 2025
@vedhavyas vedhavyas removed this pull request from the merge queue due to a manual request Oct 27, 2025
@teor2345
Copy link
Member

Note: This PR should pass CI once we update it to a main that includes #3719.

@vedhavyas
Copy link
Contributor Author

Should be good to review now.
Updated script to print the stack traces if any crashes are found

@vedhavyas vedhavyas enabled auto-merge November 3, 2025 03:49
@vedhavyas vedhavyas requested a review from teor2345 November 3, 2025 04:29
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.

4 participants