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.

Sign up for GitHub

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

Jump to bottom

Reproduce the Constantinople sstore gas refund bug #31

Closed
ESultanik opened this issue Oct 29, 2018 · 2 comments
Closed

Reproduce the Constantinople sstore gas refund bug #31

ESultanik opened this issue Oct 29, 2018 · 2 comments
Assignees
@ESultanik
Labels
enhancement New feature or request

Comments

@ESultanik
Copy link
Member

Automatically reproduce this bug using differential testing openethereum/parity-ethereum#9746
Create a minimal example that demonstrates it.
@ESultanik ESultanik added the enhancement New feature or request label Oct 29, 2018
@ESultanik ESultanik self-assigned this Oct 29, 2018
@ESultanik
Copy link
Member Author

Block on #32 then include this test as an example: https://gist.github.com/holiman/0399d4962b40a1df5d493b2cc08fd2f2#file-sstorecalltoselfsubrefundbelowzero-json

ESultanik added a commit that referenced this issue Oct 31, 2018
@ESultanik
Update the default Echidna contract with another storage variable (#31)
56e109e
@ESultanik
Copy link
Member Author

We can reproduce the bug now, but this issue will now block on #23 so we can load a custom contract to do it. It should also block on #34 to see the output of the Differential Tester.

ESultanik added a commit that referenced this issue Nov 1, 2018
@ESultanik
Refine a differential test error message (#31)
17ba67b
ESultanik added a commit that referenced this issue Nov 1, 2018
@ESultanik
Improved differential test logging (#31)
2626d92
ESultanik added a commit that referenced this issue Nov 1, 2018
@ESultanik
Added the Constantinople gas usage example (#31)
b2bdd53
ESultanik added a commit that referenced this issue Nov 1, 2018
@ESultanik
Added a README for the Constantinople example (#31)
8e3d830
ESultanik added a commit that referenced this issue Nov 1, 2018
@ESultanik
Added a note about how to run the example (#31)
be35666
@ESultanik ESultanik mentioned this issue Nov 1, 2018
Merged
ESultanik added a commit that referenced this issue Nov 1, 2018
@ESultanik
Added a note about client versions for reproducing the bug (#31)
4ff151b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ESultanik ESultanik

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ESultanik