coinbase / kryptology

Apache License 2.0
847 stars 123 forks source link

fix: make schnorr zkp work with all curves #61

Closed shahar-utila closed 1 year ago

shahar-utila commented 2 years ago

Now schnorr zkp works only with curves that are defined over fields of 256 bits since C is generated using SetBytes() on a hash of 256 bits. This PR fixes it by using Scalar.Hash instead (doesn't rely on the size of the field).

type=routine risk=low impact=sev5