Review request for 6432567: PIT : com/sun/jdi/BadHandshakeTest.java fails due to java.net.ConnectException

[Alan Bateman]

Kelly O'Hair wrote:
> This:
>    expected = strlen(hello);
> Should probably be:
>    expected = (int)strlen(hello);
Yes, it should.

>
> And I would almost think that this:
>    char msg[100];
> should be:
>    char msg[80+expected*2];
> and snprintf() used rather than sprintf? But not a big deal.
Make sense, and using snprintf would avoid the a similar issue in the 
future, if the message was changed to be much longer.

>
> Looks good otherwise.  
Thanks Kelly, Tim,  - I'll sort this out Monday (I'm not around Friday).

-Alan.