Skip to content
This repository has been archived by the owner on Feb 14, 2025. It is now read-only.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ESultanik ESultanik

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

No branches or pull requests
1 participant
@ESultanik