The original implementation assumes that r1 * z0 always be greater than r0 * z1 in L192. However it could be wrong if maximumMSB is greater than 80 and all variables are scaled by maximumMSB.
Mitigation
commit 44320a8
The mitigation removed scaling function. all r0, r1, z0, z1 will be used for calculation without any scaling.
The original issue was resolved.
Lines of code
Vulnerability details
Comments
The original implementation assumes that
r1 * z0
always be greater thanr0 * z1
in L192. However it could be wrong ifmaximumMSB
is greater than80
and all variables are scaled bymaximumMSB
.Mitigation
commit 44320a8 The mitigation removed scaling function. all
r0
,r1
,z0
,z1
will be used for calculation without any scaling. The original issue was resolved.Conclusion
Confirmed