Integrated: 8325162: Remove duplicate GCMParameters class

Daniel Jeliński djelinski at openjdk.org
Thu Feb 15 11:29:59 UTC 2024


On Mon, 12 Feb 2024 08:34:24 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

> Currently we have 2 GCMParameters classes in java.base, one in sun.security.util package, the other in com.sun.crypto.provider.
> 
> This PR merges the changes from the sun.security.util copy into com.sun.crypto.provider, and removes the sun.security.util class.
> 
> As far as I could tell, the sun.security.util class was added to avoid exporting com.sun.crypto.provider to jdk.crypto.cryptoki. The export was introduced later in [JDK-8259319](https://bugs.openjdk.org/browse/JDK-8259319), so there's no reason to keep both copies.
> 
> No new tests. Tier1-3 tests continue to pass.

This pull request has now been integrated.

Changeset: a0e5e16a
Author:    Daniel Jeliński <djelinski at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/a0e5e16afbd19f6396f0af2cba954225a357eca8
Stats:     148 lines in 2 files changed: 0 ins; 146 del; 2 mod

8325162: Remove duplicate GCMParameters class

Reviewed-by: weijun, valeriep

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

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



More information about the security-dev mailing list