Code review request, JDK-8040062 Need to add new methods in BaseSSLSocketImpl
Sean Mullan
sean.mullan at oracle.com
Mon Apr 14 12:59:35 UTC 2014
Looks fine to me. Can you add a relates to link to JDK-8036979 and an
appropriate noreg label?
--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