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.

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

Genesis Block reward is treated as +balance for zeroAddress. #293

Open
aeharvlee opened this issue Mar 28, 2022 · 3 comments
Open

Genesis Block reward is treated as +balance for zeroAddress. #293

aeharvlee opened this issue Mar 28, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@aeharvlee
Copy link

aeharvlee commented Mar 28, 2022

Describe the bug

When running rosetta-cli for Ethereum Roopsten testnet or Mainnet, there was an reconciliation issue for 0x0000000000000000000000000000000000000000.

Error message:
Command Failed: reconciliation failure: inactive reconciliation error for 0x0000000000000000000000000000000000000000 at 10 (computed: 5000000000000000001ETH, live: 1ETH)

Cause

  • At the genesis block, there is a mining reward for 0x0000000000000000000000000000000000000000.
  • And that reward does not applied to on-chain balance of 0x0000000000000000000000000000000000000000.
  • But the problem is: Reconciler handles that genesis reward as +balance for 0x0000000000000000000000000000000000000000, so it triggers the error.

To Reproduce

  • Following this guide is enough to re-produce the problem.

Expected behavior

  • The first genesis block reward must not be calculated +balance for 0x0000000000000000000000000000000000000000 because it is not applied to on-chain state.

Additional context

  • OS: macOS 12.3
  • Go version: go1.17.7 darwin/arm64
  • Source code version of rosetta-cli: 18013bd
@aeharvlee aeharvlee added the bug Something isn't working label Mar 28, 2022
@aeharvlee
Copy link
Author

Currently, the cli user must set the startIndex as 1 to avoid Genesis Block mining reward issue.

@aeharvlee
Copy link
Author

aeharvlee commented Mar 29, 2022

In my opinion, below opotions should be considered.

  • Dose not track balances of ZeroAddress
  • Does not calculate Genesis mining reward as +balance

One of above patch should be applied to rosetta-sdk-go.

@shrimalmadhur
Copy link
Contributor

@aeharvlee you are right. We had this check before but somehow due to one another issue we changed it (coinbase/mesh-ethereum#73) . We will fix that soon. Thanks for raising the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

No branches or pull requests

2 participants