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

fix: push0 should increment program counter by 1 instead of 2 #803

Merged
merged 4 commits into from
Jul 26, 2023

Conversation

vimpunk
Copy link
Contributor

@vimpunk vimpunk commented Jul 25, 2023

Description

The PR bumps the sputnikvm version, which contains a bugfix of the shanghai implementation.

Testing

Added test which validates EVM transaction created with parameter: evmVersion: 'shanghai'.

Cargo.lock Show resolved Hide resolved
@vimpunk vimpunk force-pushed the fix/shanghai-push0-invalid-pc branch from dc40a3b to 0ff258c Compare July 25, 2023 16:33
@vimpunk vimpunk changed the title fix: push0 should move program counter by 1 instead of 2 fix: push0 should increment program counter by 1 instead of 2 Jul 25, 2023
engine-tests/src/tests/sanity.rs Outdated Show resolved Hide resolved
engine-tests/src/tests/sanity.rs Outdated Show resolved Hide resolved
engine-tests/src/tests/sanity.rs Outdated Show resolved Hide resolved
Co-authored-by: Michael Birch <michael.birch@aurora.dev>
@aleksuss aleksuss added this pull request to the merge queue Jul 26, 2023
Merged via the queue into develop with commit 0ffcf0d Jul 26, 2023
20 checks passed
@aleksuss aleksuss deleted the fix/shanghai-push0-invalid-pc branch July 26, 2023 09:37
joshuajbouw pushed a commit that referenced this pull request Jul 27, 2023
## Description

The PR bumps the `sputnikvm` version, which contains a bugfix of the
`shanghai` implementation.

## Testing

Added test which validates EVM transaction created with parameter:
`evmVersion: 'shanghai'`.

---------

Co-authored-by: Oleksandr Anyshchenko <oleksandr.anyshchenko@aurora.dev>
Co-authored-by: Michael Birch <michael.birch@aurora.dev>
@joshuajbouw joshuajbouw mentioned this pull request Jul 27, 2023
joshuajbouw added a commit that referenced this pull request Jul 27, 2023
### Fixes

- Updated sputnikvm dependency with bugfix in the `shanghai`
implementation @mandreyel. #803

---------

Co-authored-by: mandreyel <mandreyel@aurora.dev>
Co-authored-by: Oleksandr Anyshchenko <oleksandr.anyshchenko@aurora.dev>
Co-authored-by: Michael Birch <michael.birch@aurora.dev>
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.

4 participants