Skip to content

Fix ecdsa_raw verify and recover - #146

Open
wizardofozzie wants to merge 1 commit into
vbuterin:masterfrom
wizardofozzie:patch-19
Open

wizardofozzie wants to merge 1 commit into
vbuterin:masterfrom
wizardofozzie:patch-19

Conversation

@wizardofozzie

Copy link
Copy Markdown
Contributor

ecdsa v value fix (tests working!)

ecdsa v value fix (tests working!)
@wizardofozzie

Copy link
Copy Markdown
Contributor Author

v = 0 fails on ecdsa_raw_recover and ecdsa_raw_verify (False and ValueError, respectively) since v value is lost with DER encoding

@wizardofozzie wizardofozzie changed the title Update main.py Fix ecdsa_raw verify and recover Aug 1, 2016
@wizardofozzie

Copy link
Copy Markdown
Contributor Author

@vbuterin this code fixes failing tests

@reiven

reiven commented Oct 14, 2016

Copy link
Copy Markdown
Contributor

@vbuterin can this be merged, so tests stops failing? thanks

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