fix: bullet rangeproofs batch verification will fail sometimes

mimblewimble / rust-secp256k1-zkp

ZKP fork for rust-secp256k1, adds wrappers for range proofs, pedersen commitments, etc

Creative Commons Zero v1.0 Universal

56 stars 51 forks source link

Closed garyyu closed 6 years ago

garyyu commented 6 years ago

fix the mistakes of "variables does not live long enough", which cause the test errors (from test cases in pr: https://github.com/mimblewimble/rust-secp256k1-zkp/pull/22 :

4 proofs batch verify failed
17 proofs batch verify failed
18 proofs batch verify failed
19 proofs batch verify failed
20 proofs batch verify failed

yeastplume commented 6 years ago

Looks like that makes sense... I can merge but it won't be usable in grin unless another tag is done. Do you need me to do that?