Skip to content

Fix smart contract addressing and storage #177

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tinkerrc
Copy link

@tinkerrc tinkerrc commented Jun 3, 2025

This commit fixes the following issues:

  • New contract addresses are generated with a random nonce instead of 0. This allows an account owner to deploy multiple contracts without address conflict.
  • When storing, GlobalView passes both the contract address and the key argument to the underlying storage as key instead of just the key argument. This allows different instances of the same contract to store separate state values under the same state name.
  • Update eEVM dependency.
  • In contract_tools, I added the missing ":f:" to getopt_long. This allows users to pass config file to contract_tools.
  • Remove the merge conflict markers from start_contract_service.sh.
  • clang-format fixed some formatting issues.

This commit fixes the following issues:

- New contract addresses are generated with a random nonce instead of 0.
  This allows an account owner to deploy multiple contracts without
  address conflict.
- When storing, GlobalView passes both the contract address and the key
  argument to the underlying storage as key instead of just the key
  argument. This allows different instances of the same contract to
  store separate state values under the same state name.
- Update eEVM dependency.
- In contract_tools, I added the missing ":f:" to getopt_long. This
  allows users to pass config file to contract_tools.
- Remove the merge conflict markers from start_contract_service.sh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant