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

Sean Mullan mullan at openjdk.org
Wed Nov 13 20:27:01 UTC 2024


On Wed, 13 Nov 2024 15:12:24 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Sean Mullan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove osName and osArch fields from Config.
>>   Remove getValue method from P11KeyAgreement.
>
> 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.

Good spot; I removed them - I'm ok with making small changes as long as they are within the scope of the code being modified.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22077#discussion_r1841105708


More information about the security-dev mailing list