Code Review Request: 7122707 update Security Providers to JDK8

Xuelei Fan xuelei.fan at oracle.com
Tue Sep 17 02:08:03 UTC 2013


On 9/17/2013 9:44 AM, Anthony Scarpino wrote:
> On 09/16/2013 06:27 PM, Xuelei Fan wrote:
>> src/macosx/classes/apple/security/AppleProvider.java
>> ====================================================
>> I'm not sure apple provider is using the JDK version as the provider
>> version.
>> -        super("Apple", 1.1, info);
>> +        super("Apple", 1.8d, info);
>>
>> src/share/classes/sun/security/jgss/wrapper/SunNativeProvider.java
>> ==================================================================
>> As above, I'm not sure native JGSS provider is using JDK version as the
>> provider version.
>> -        super(NAME, 1.0, INFO);
>> +        super(NAME, 1.8d, INFO);
>>
>> Should not a problem, but nice to get confirm from Max, and the
>> component owner of apple provider.
> 
> Some of us were talking locally about updating providers to the current
> JDK version even if they hadn't been modified.  So if there is a certain
> set that shouldn't be changed, then we should specify a criteria..
> 
I was just wondering why the version number start with 1.1 or 1.0. They
are not old providers for JDK 1.0 or 1.1.  I think above two providers
are introduced recently. Not sure about the history.  I prefer to be
consistent with JDK releases. But better to check with the owner.

Xuelei

> 
>>
>> src/share/classes/sun/security/pkcs11/SunPKCS11.java
>> ====================================================
>> -        super("SunPKCS11-Dummy", 1.7d, "SunPKCS11-Dummy");
>> +        super("SunPKCS11", 1.8d, "SunPKCS11");
>>
>> I think the real provider name should be "SunPKCS11-ConfigName".  I'm
>> not sure about this fix.  Vinnie may have comments here.
> 
> I forgot to put back the "Dummy" after I realized what it meant.  I'll
> update that.  thanks
> 
> Tony
> 
>>
>> Otherwise, looks fine to me.
>>
>> Xuelei
>>
>> On 9/17/2013 7:21 AM, Anthony Scarpino wrote:
>>> Hi,
>>>
>>> I'd like a review of these quick changes to the security providers.
>>> 7122707 Security Providers need to have their version numbers updated
>>> for JDK8
>>>
>>> http://cr.openjdk.java.net/~ascarpino/7122707/webrev.00/
>>>
>>> thanks
>>>
>>> Tony
>>
> 




More information about the security-dev mailing list