RFR: 8284855: Update needed to Cleaners added to jdk.crypto.cryptoki [v3]

Brent Christian bchristi at openjdk.java.net
Fri Apr 15 18:51:38 UTC 2022


On Fri, 15 Apr 2022 15:00:18 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

> > CleanerFactory is in java.base module, and does not export to jdk.crypto.cryptoki module.  I'm not sure if adding modules dependency is good or not.

It seems fine to me to export the common Cleaner to `jdk.crypto.cryptoki`. It's already [exported for desktop and Panama](https://github.com/bchristi-git/jdk/blob/master/src/java.base/share/classes/module-info.java#L226), and I see that [com.sun.crypto.provider](https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/com/sun/crypto/provider/DESKey.java#L84) already uses it.

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

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



More information about the security-dev mailing list