RFR: 8287178: IntegerModuloP::multiplicativeInverse returns 0 for 0 [v2]

Weijun Wang weijun at openjdk.java.net
Thu Jun 9 23:14:49 UTC 2022


On Thu, 9 Jun 2022 22:29:36 GMT, Jamil Nimeh <jnimeh at openjdk.org> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   verb
>
> src/java.base/share/classes/sun/security/util/math/IntegerModuloP.java line 161:
> 
>> 159:         // This method is used in 2 cases:
>> 160:         // 1. To calculate the inverse of a number in ECDSAOperations,
>> 161:         //    this number must be non zero (modulus p).
> 
> Should that be "modulo p"?  Other than that it looks good to me.

Yes, it's a verb here.

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

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



More information about the security-dev mailing list