Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Feature]: Investigate Fuzzing #458

Open
2 of 4 tasks
0xNeshi opened this issue Dec 16, 2024 · 1 comment
Open
2 of 4 tasks

[Feature]: Investigate Fuzzing #458

0xNeshi opened this issue Dec 16, 2024 · 1 comment
Assignees
@0xNeshi
Labels
effort: high Large or difficult task. priority: 2 We will resolve this in a short timeframe. type: test Changes to the testing suite. uphill Some research/design is needed before this can be implemented.

Comments

@0xNeshi
Copy link
Collaborator

0xNeshi commented Dec 16, 2024 •
edited
Loading

What is the feature you would like to see?

  • find which, if any, contracts/functions are good candidates for fuzzing
  • determine whether we can use any existing library for fuzzing, or if we need to create this logic ourselves
  • implement fuzzing for a single function (to be used as template for others)

Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
@0xNeshi 0xNeshi added priority: 2 We will resolve this in a short timeframe. type: feature New feature request. type: test Changes to the testing suite. effort: high Large or difficult task. uphill Some research/design is needed before this can be implemented. labels Dec 16, 2024
@0xNeshi 0xNeshi self-assigned this Dec 16, 2024
@0xNeshi
Copy link
Collaborator Author

0xNeshi commented Dec 16, 2024 •
edited
Loading

Findings unrelated to fuzzing:
Erc721Enumerable::_add_token_to_owner_enumeration can panic if balance_of(to) == 0
Erc721Enumerable::_remove_token_from_all_tokens_enumeration can panic if self._all_tokens.len() == 0.

@0xNeshi 0xNeshi removed the type: feature New feature request. label Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@0xNeshi 0xNeshi

Labels
effort: high Large or difficult task. priority: 2 We will resolve this in a short timeframe. type: test Changes to the testing suite. uphill Some research/design is needed before this can be implemented.
Projects
Rust Contracts Stylus
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
1 participant
@0xNeshi