Review request for 6432567: PIT : com/sun/jdi/BadHandshakeTest.java fails due to java.net.ConnectException
Alan Bateman
Alan.Bateman at Sun.COM
Thu Aug 27 21:50:33 PDT 2009
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.
More information about the serviceability-dev
mailing list