RFR: 8344120: Remove Security Manager dependencies from jdk.crypto.cryptoki module

Roger Riggs rriggs at openjdk.org
Wed Nov 13 15:21:13 UTC 2024


On Wed, 13 Nov 2024 14:44:06 GMT, Sean Mullan <mullan at openjdk.org> wrote:

> Now that JEP 486 has been integrated, the `jdk.crypto.cryptoki` implementation dependencies on `System.getSecurityManager` and `AccessController.doPrivileged` can be removed.

LGTM

src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/Config.java line 72:

> 70:         }
> 71:         osName = System.getProperty("os.name");
> 72:         osArch = System.getProperty("os.arch");

osName and osArch seem to be unused; they could be removed but might be out of scope for this PR.

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

Marked as reviewed by rriggs (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22077#pullrequestreview-2433552784
PR Review Comment: https://git.openjdk.org/jdk/pull/22077#discussion_r1840546297


More information about the security-dev mailing list