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

Wang Weijun weijun.wang at oracle.com
Wed May 11 04:42:44 UTC 2016


This looks good. You might need to add a noreg-self to the bug.

Thanks
Max

> On May 11, 2016, at 12:31 PM, Xuelei Fan <xuelei.fan at oracle.com> wrote:
> 
> On 5/10/2016 4:23 PM, Xuelei Fan wrote:
>> 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.
>> 
> 
> Update the block to:
> -   if (provider.getName().equals("SunPKCS11-Solaris") &&
> -      !System.getProperty("os.name").equals("SunOS")) {
> +   if (!provider.getName().equals("SunPKCS11-Solaris")) {
> 
> This update will be integrated with JDK-8156716.  As this is a simple
> fix, will not open another thread any more.
> 
> Thanks,
> Xuelei




More information about the security-dev mailing list