mimblewimble / rust-secp256k1-zkp

ZKP fork for rust-secp256k1, adds wrappers for range proofs, pedersen commitments, etc
Creative Commons Zero v1.0 Universal
57 stars 51 forks source link

refactor: swap (r,s) zero checking on signature #36

Closed garyyu closed 6 years ago

garyyu commented 6 years ago

This is the corresponding refactor of https://github.com/mimblewimble/secp256k1-zkp/pull/31 but in rust layer.

Please merging https://github.com/mimblewimble/secp256k1-zkp/pull/31 before merging this, otherwise the Travis-CI will fail.

Note: Without merging https://github.com/mimblewimble/secp256k1-zkp/pull/31, Travis-CI will fail on this PR. It's the expected failure.

yeastplume commented 6 years ago

the dependency is merged, but you'll also need to update the submodule branch before we can run the CI test again

garyyu commented 6 years ago

Submodule updated and Travis-CI passed.