Skip to content

Comments

update storage trie type (#1070 #1071)#1096

Merged
gurukamath merged 1 commit intoethereum:forks/praguefrom
SamWilsn:update-trie-type
Jan 29, 2025
Merged

update storage trie type (#1070 #1071)#1096
gurukamath merged 1 commit intoethereum:forks/praguefrom
SamWilsn:update-trie-type

Conversation

@SamWilsn
Copy link
Contributor

What was wrong?

Related to Issue #1070 #1071

@SamWilsn SamWilsn requested a review from gurukamath January 25, 2025 18:49
Copy link
Contributor

@gurukamath gurukamath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes to src/ethereum_optimized/state_db.py will also need to be ported over I think

@gurukamath gurukamath merged commit ad81be4 into ethereum:forks/prague Jan 29, 2025
5 checks passed
SamWilsn added a commit to SamWilsn/eth1.0-specs that referenced this pull request Oct 16, 2025
Tests the case where a large base[^1] is added to 96. Both Geth and
Nethermind correctly revert the precompile call with an out-of-gas
error, but EELS crashes with an `OverflowError`.

[^1]: `0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffa0`

Reported-by: Omik
danceratopz pushed a commit to danceratopz/execution-specs that referenced this pull request Oct 22, 2025
Tests the case where a large base[^1] is added to 96. Both Geth and
Nethermind correctly revert the precompile call with an out-of-gas
error, but EELS crashes with an `OverflowError`.

[^1]: `0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffa0`

Reported-by: Omik
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.

2 participants