- secp256k1_fe_sqrt now checks that the value it calculated is actually a square root.
- Add return values to secp256k1_fe_sqrt and secp256k1_ge_set_xo.
- Callers of secp256k1_ge_set_xo can use return value instead of explicit validity checks
- Add random value tests for secp256k1_fe_sqrt
secp256k1_fe_sqrt checks for success #19
pull peterdettman wants to merge 1 commits into bitcoin-core:master from peterdettman:checked-sqrt changing 6 files +84 −21-
peterdettman commented at 3:52 AM on May 21, 2014: contributor
-
09ca4f32e2
secp256k1_fe_sqrt checks for success
- secp256k1_fe_sqrt now checks that the value it calculated is actually a square root. - Add return values to secp256k1_fe_sqrt and secp256k1_ge_set_xo. - Callers of secp256k1_ge_set_xo can use return value instead of explicit validity checks - Add random value tests for secp256k1_fe_sqrt
-
sipa commented at 5:57 PM on May 23, 2014: contributor
ACK
- sipa merged this on Jun 2, 2014
- sipa closed this on Jun 2, 2014
- sipa referenced this in commit 7740eff28a on Jun 2, 2014
- peterdettman deleted the branch on Jun 22, 2014
- tecnovert referenced this in commit 809016c819 on Apr 18, 2022
- theStack referenced this in commit 37974c5e00 on Aug 7, 2023
- theStack referenced this in commit 197430f051 on Aug 17, 2023
- theStack referenced this in commit dc5514144f on Aug 17, 2023
- real-or-random referenced this in commit ee7aaf213e on Sep 14, 2023
