Nothing is wrong with the current implementation. But it looks out of place because we generally prefer bit operations because they're faster and are more likely constant-time.
Backport of secp256k1#882
Differential D9388
[SECP256K1] Use bit ops instead of int mult for constant-time logic in gej_add_ge Fabien on Apr 8 2021, 14:55. Authored by Tags None Subscribers None
Details
Nothing is wrong with the current implementation. But it looks out of place because we generally prefer bit operations because they're faster and are more likely constant-time. Backport of secp256k1#882 ninja check-secp256k1
Diff Detail
|