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

add test for returndatasize after ecrec #346

Closed
wants to merge 1 commit into from

Conversation

cdetrio
Copy link
Member

@cdetrio cdetrio commented Oct 6, 2017

Minimal test case for consensus issue was discovered by fuzz testing. Fixed in Parity in openethereum/parity-ethereum#6652

@winsvega
Copy link
Collaborator

please refill with latest develop

@cdetrio
Copy link
Member Author

cdetrio commented Oct 10, 2017

will there be any change with latest develop? other than the filledwith metadata

@winsvega
Copy link
Collaborator

hash of the test filler required

@pirapira
Copy link
Member

@cdetrio otherwise, when testeth:develop runs this test, it reports errors about missing fillerhash. testeth:develop tries to make sure the test is up-to-date with the filler, by comparing the filler contents against a hash recorded in the filled test.

@winsvega
Copy link
Collaborator

#362

@winsvega winsvega closed this Oct 19, 2017
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.

3 participants