RFR: 8295010: Reduce if required in EC limbs operations [v5]

Daniel Jeliński djelinski at openjdk.org
Tue Nov 22 08:43:26 UTC 2022


On Mon, 21 Nov 2022 16:19:00 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/util/math/intpoly/IntegerPolynomial.java line 561:
>> 
>>> 559:             // Reduce if required.
>>> 560:             // if (numAdds >= maxAdds) {
>>> 561:             if (numAdds > 32 - bitsPerLimb) {
>> 
>> if we allow this number of additions, `setProduct(SmallValue)` might overflow in the future. Currently it's safe - we only create a limited set of SmallValues, and they are all small enough to avoid this risk.
>> `getSmallValue` allows numbers up to `bitsPerLimb` bits. I think we can adjust that.
>
> Sorry, I did not get the ideas.  Did you meant to adjust the implementation of getSmallValue to allow 32 bits int value?

Apologies, I was looking at the wrong branch, didn't notice `setProduct(SmallValue)` also does a reduce if required.

-------------

PR: https://git.openjdk.org/jdk/pull/10624


More information about the security-dev mailing list