AztecProtocol / aztec-2.0

C++ elliptic curve library
134 stars 37 forks source link

A potential bug in scalar_multiplication.cpp #80

Open BoyuanFeng opened 2 years ago

BoyuanFeng commented 2 years ago

Hi,

Thanks for the great optimizations on batch affine!

When reading the implementation for add_affine_points_with_edge_cases(...), I found the following for loop which seems to iterate forever.

for (size_t i = (num_points)-2; i < num_points; i -= 2) { ... }

Any suggestions? Thanks!