Skip to content
This repository has been archived by the owner on Nov 18, 2022. It is now read-only.

Fix address collision between contract and unused EOA #150

Merged
merged 7 commits into from
May 31, 2022

Conversation

magicalne
Copy link
Contributor

@magicalne magicalne commented May 9, 2022

Fix address collision between EOA account address and contract address created by create/create2.
We can overwrite the mapping if it's an EOA account and the nonce is 0.

@magicalne magicalne requested review from Flouse and jjyr May 9, 2022 04:04
@Flouse
Copy link
Collaborator

Flouse commented May 9, 2022

Please fix the integration-test in CI

c/polyjuice.h Outdated Show resolved Hide resolved
c/polyjuice.h Show resolved Hide resolved
c/polyjuice.h Show resolved Hide resolved
c/polyjuice.h Show resolved Hide resolved
@Flouse Flouse requested a review from TheWaWaR May 9, 2022 09:50
@magicalne magicalne force-pushed the fix_addr_collision branch from fe76abc to a5a0b95 Compare May 9, 2022 14:07
@Flouse Flouse changed the title Fix addr collision Fix address collision between contract and unused EOA May 10, 2022
@jjyr
Copy link
Collaborator

jjyr commented May 11, 2022

Please note the change godwokenrises/godwoken-scripts#119 (comment)

@magicalne magicalne force-pushed the fix_addr_collision branch from d4bade3 to 7d06ee1 Compare May 12, 2022 07:12
@magicalne magicalne force-pushed the fix_addr_collision branch from 7d06ee1 to 0106330 Compare May 12, 2022 07:24
@Flouse Flouse requested review from zeroqn and jjyr May 12, 2022 15:31
c/polyjuice.h Show resolved Hide resolved
Flouse
Flouse previously approved these changes May 13, 2022
c/polyjuice.h Show resolved Hide resolved
@Flouse
Copy link
Collaborator

Flouse commented May 13, 2022

Please bump the POLYJUICE_VERSION to "v1.1.6-beta"

Flouse
Flouse previously approved these changes May 15, 2022
jjyr
jjyr previously approved these changes May 16, 2022
zeroqn
zeroqn previously approved these changes May 16, 2022
c/polyjuice.h Outdated Show resolved Hide resolved
c/polyjuice.h Outdated Show resolved Hide resolved
@magicalne magicalne dismissed stale reviews from zeroqn and jjyr via 9dccdbf May 19, 2022 08:02
@Flouse Flouse changed the title Fix address collision between contract and unused EOA [DO NOT MERGE] Fix address collision between contract and unused EOA May 20, 2022
@Flouse Flouse changed the title [DO NOT MERGE] Fix address collision between contract and unused EOA Fix address collision between contract and unused EOA May 31, 2022
@Flouse Flouse merged commit 4d3985e into godwokenrises:main May 31, 2022
@magicalne magicalne deleted the fix_addr_collision branch September 23, 2022 09:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants