Code review request, JDK-8040062 Need to add new methods in BaseSSLSocketImpl

Xuelei Fan xuelei.fan at oracle.com
Mon Apr 14 13:03:19 UTC 2014


On 4/14/2014 8:59 PM, Sean Mullan wrote:
> Looks fine to me. Can you add a relates to link to JDK-8036979 and an
> appropriate noreg label?
> 
Made the update in JBS.

Thanks for the review.

Xuelei

> --Sean
> 
> On 04/14/2014 06:04 AM, Xuelei Fan wrote:
>> Hi,
>>
>> Please review the fix for JDK-8040062:
>>
>>      http://cr.openjdk.java.net/~xuelei/8040062/webrev.00/
>>
>> In JDK-8036979, there are news methods are added to java.net.Socket.
>> These methods need to be overridden in SSL socket implementation,
>> sun/security/ssl/BaseSSLSocketImpl.java.
>>
>> No new regression test.  The existing test:
>>
>>     sun/security/ssl/SSLSocketImpl/CheckMethods.java
>>
>> can be used to verify this fix.
>>
>> Thanks,
>> Xuelei
>>




More information about the security-dev mailing list