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