RFR: 8325162: Remove duplicate GCMParameters class [v2]

Daniel Jeliński djelinski at openjdk.org
Tue Feb 13 20:58:17 UTC 2024


> 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.

Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:

  Remove the other GCMParameters

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17802/files
  - new: https://git.openjdk.org/jdk/pull/17802/files/2b076a96..d210a047

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17802&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17802&range=00-01

  Stats: 11 lines in 3 files changed: 0 ins; 4 del; 7 mod
  Patch: https://git.openjdk.org/jdk/pull/17802.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17802/head:pull/17802

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



More information about the security-dev mailing list