Adapt @peterdettman's verify function (taking the magnitude leniency factor 2 into account), and use it as consistency check everywhere.
Add secp256k1_fe_verify in field_5x52_impl and use it everywhere #44
pull sipa wants to merge 2 commits into bitcoin-core:master from sipa:verify changing 2 files +71 −12-
sipa commented at 5:10 PM on July 15, 2014: contributor
-
sipa commented at 1:53 PM on July 16, 2014: contributor
@peterdettman Any comments?
-
Test demonstrating discrepancy in sqr output 59447da38d
-
Correct secp256k1_fe_verify and use it everywhere 21f81a8469
-
sipa commented at 7:08 PM on July 17, 2014: contributor
Rebased.
- sipa merged this on Jul 23, 2014
- sipa closed this on Jul 23, 2014
- sipa referenced this in commit 7d1956ff8d on Jul 23, 2014
- sipa cross-referenced this on Sep 2, 2014 from issue Test demonstrating discrepancy in sqr output by peterdettman
Contributors