Code Review Request, 8154344 sun/security/pkcs11/KeyAgreement/SupportedDHKeys.java fails on solaris

Xuelei Fan xuelei.fan at oracle.com
Tue May 10 08:23:51 UTC 2016


On 5/10/2016 2:37 PM, Wang Weijun wrote:
> 
>> On May 10, 2016, at 7:58 AM, Xuelei Fan <xuelei.fan at oracle.com> wrote:
>>
>> On 5/10/2016 12:16 AM, Michael StJohns wrote:
>>> You're going to hate me, but shouldn't this be checking the Provider
>>> rather than the OS when trying to determine which test to ignore?
>>>
>> It's more accurate to check the provider, too.
>>
>> -   if (!System.getProperty("os.name").equals("SunOS")) {
>> +   if (provider.getName().equals("SunPKCS11-Solaris") &&
>> +         !System.getProperty("os.name").equals("SunOS")) {
> 
> There is SunPKCS11-Solaris not on SunOS?
> 
SunPKPCS11-NSS may be on SunOS.

Xuelei



More information about the security-dev mailing list