-
Notifications
You must be signed in to change notification settings - Fork 29
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
feat: Multi Asset Delegation pallet fuzzing #856
Conversation
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.
Need to also fuzz the precompile, great to see this!
2ffe570
to
7da7729
Compare
Working on it, will create a new PR for EVM Precompile fuzzing. |
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.
Can we add this to CI? Looks good, am curious how you're approaching the comment on checking math, etc.
Yup adding more checks now, maybe @1xstj could help guiding me on what are the important checks that needs to be added in the fuzzed code/state. |
a917a4c
to
03a1fa3
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.
LG 🚀
36b02b3
to
a5210c7
Compare
Summary of changes
Changes introduced in this pull request:
This pull request includes the addition of a new fuzzer module for the
pallet-multi-asset-delegation
. The most important changes include adding new dependencies, updating existing dependencies, and introducing a fuzzer for thepallet-multi-asset-delegation
module.Dependency Updates and Additions:
honggfuzz
dependency toCargo.toml
for fuzz testing.pallets/multi-asset-delegation/Cargo.toml
. [1] [2]Fuzzer Module Addition:
Cargo.toml
for thepallet-multi-asset-delegation-fuzzer
with dependencies and metadata.call.rs
file with the implementation of the fuzzer forpallet-multi-asset-delegation
.Codebase Enhancements:
mock
andmock_evm
modules to be included when thefuzzing
feature is enabled.BlueprintWhitelisted
event to useBlueprintId
instead ofu32
.Run the MAD Fuzzer
cargo install honggfuzz
cargo hfuzz run mad-fuzzer
Reference issue to close (if applicable)
There is no task for it right now so I will add the points to this PR.