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