Code Review Request, JDK-8167472, Chrome interop regression with JDK-8148516

Weijun Wang weijun.wang at oracle.com
Tue Oct 11 05:44:08 UTC 2016


Code change looks fine.

Or you can write if (isSupported && permits)....

Thanks
Max

On 10/11/2016 13:06, Xuelei Fan wrote:
> Hi,
>
> Please review this update:
>
>    http://cr.openjdk.java.net/~xuelei/8167472/webrev.00/
>
> The root cause is that in the update of JDK-8148516, we removed some EC
> curves from the default supported EC curves list.  However, while select
> the preferable curves requested by the client side, the server does not
> check whether the curves are supported or not.  Actually, only supported
> EC curves should be selected.
>
> I tested the patch with Chrome, and no new test is added.
>
> Thanks,
> Xuelei


More information about the security-dev mailing list