Skip to content

Conversation

@mattsse
Copy link
Member

@mattsse mattsse commented Aug 10, 2022

Motivation

Closes #1943

there were some changes in revm re Bytecode and how accounts are handled, previously it was sufficient to just set the code_hash to KECCAK_EMPTY.
this is no longer the case, so #1943 resurfaced

Solution

  • also remove the code temporarily when impersonating an account
  • add test for gnosis safe contract

@onbjerg onbjerg added the T-bug Type: bug label Aug 10, 2022
@mattsse mattsse merged commit 4243e0a into foundry-rs:master Aug 10, 2022
iFrostizz pushed a commit to iFrostizz/foundry that referenced this pull request Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-bug Type: bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow impersonating contracts

2 participants