Re: 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.