HomePhabricator

Also switch the (unused) verification code to low-s instead of even-s.
6fd7ef2bbf1fUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Also switch the (unused) verification code to low-s instead of even-s.

a81cd968 introduced a malleability breaker for signatures
(using an even value for S). In e0e14e43 this was changed to
the lower of two potential values, rather than the even one.
Only the signing code was changed though, the (for now unused)
verification code wasn't adapted.

Details

Provenance
Pieter Wuille <pieter.wuille@gmail.com>Authored on Feb 7 2014, 01:19
Pieter Wuille <sipa@ulyssis.org>Committed on Mar 10 2014, 19:38
schancelPushed on Jan 5 2018, 21:58
schancelPushed on Jan 5 2018, 21:39
schancelPushed on Jan 5 2018, 21:17
Parents
rSTAGINGa63f8b7b36e3: Merge pull request #3717 from djpnewton/wallet-txcount
Branches
Unknown
Tags
Unknown

Event Timeline

Pieter Wuille <sipa@ulyssis.org> committed rSTAGING6fd7ef2bbf1f: Also switch the (unused) verification code to low-s instead of even-s. (authored by Pieter Wuille <pieter.wuille@gmail.com>).Mar 10 2014, 19:38