Code review request, JDK-8017157, catch more exception in test RejectClientRenego

Vincent Ryan vincent.x.ryan at oracle.com
Thu Jun 20 14:17:38 UTC 2013


Your change looks fine to me.

On 20 Jun 2013, at 10:42, Xuelei Fan wrote:

> Hi,
> 
> Please review this test only update:
> http://cr.openjdk.java.net/~xuelei/8017157/webrev.00/
> 
> In test RejectClientRenego.java, we tried to catch SSLHandshakeException
> in both client and server. However, the client thread may throw
> SocketException when it receives server handshake failure messages.
> 
> This update loosens the exception check by ignoring IOException rather
> than SSLHandshakeException:
> 
> -        } catch (SSLHandshakeException she) {
> +        } catch (IOException ioe) {
> 
> Thanks,
> Xuelei




More information about the security-dev mailing list