Integrated: 8279903: Redundant modulo operation in ECDHKeyAgreement

John Jiang jjiang at openjdk.java.net
Thu Jan 13 02:21:24 UTC 2022


On Wed, 12 Jan 2022 06:57:45 GMT, John Jiang <jjiang at openjdk.org> wrote:

> In class `sun.security.ec.ECDHKeyAgreement`, the last `mod()` in the below line looks redundant,
> 
> BigInteger lhs = y.modPow(BigInteger.valueOf(2), p).mod(p);
> 
> I think this tiny change just be a code cleanup, so no test for it and label `noreg-cleanup` is added for this JBS issue.

This pull request has now been integrated.

Changeset: 48519480
Author:    John Jiang <jjiang at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/485194805966e8dbb76473fa26276e5ba26d8097
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

8279903: Redundant modulo operation in ECDHKeyAgreement

Reviewed-by: weijun, xuelei

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

PR: https://git.openjdk.java.net/jdk/pull/7042



More information about the security-dev mailing list