Skip to content

fix(forge): do not use bytecode metadata in fuzz dict #10402

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

Merged
merged 1 commit into from
Apr 29, 2025

Conversation

grandizzy
Copy link
Collaborator

@grandizzy grandizzy commented Apr 28, 2025

Motivation

Solution

  • extract ignore_metadata_hash in commons, reuse when collecting push bytes into fuzz dict

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@grandizzy grandizzy changed the title fix(forge): do not use metadata in fuzzing fix(forge): do not use bytecode metadata in fuzz dict Apr 28, 2025
Base automatically changed from dani/metadata-hash-cleanup to master April 28, 2025 07:35
@grandizzy grandizzy force-pushed the grandizzy/issue-8115 branch from a946591 to d30e3e6 Compare April 28, 2025 07:38
@grandizzy grandizzy marked this pull request as ready for review April 28, 2025 08:02
@grandizzy grandizzy added T-bug Type: bug C-forge Command: forge labels Apr 28, 2025
@grandizzy grandizzy merged commit 96f6ffe into master Apr 29, 2025
22 checks passed
@grandizzy grandizzy deleted the grandizzy/issue-8115 branch April 29, 2025 02:00
@github-project-automation github-project-automation bot moved this to Done in Foundry Apr 29, 2025
@grandizzy grandizzy self-assigned this Apr 29, 2025
@grandizzy grandizzy moved this from Done to Completed in Foundry Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-forge Command: forge T-bug Type: bug
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

feat(fuzz): do not populate dictionary with bytecode metadata
3 participants