Tim - do you mind reviewing this one? It's the BadHankshakeTest failure
on windows-x64 that we've discussed in the past but didn't manage to get
to the finish line. Another bug (and patch)[1] was submitted by Kees
Cook on exactly the same issue, but on Ubunutu with OpenJDK compiled
with an option to detect bugs like this. The issue as you know, arises
when the debuggee is connected to something that isn't a debugger and
the handshake fails. The transport code writes the error message into a
buffer on the stack that isn't sized correctly and so can trash the
stack. In the proposed change I have over-sized the buffer in case the
error message is tweaked in the future. I've also fixed a typo in the
error message and some local clean-up (more could be done). The webrev
is here:
http://cr.openjdk.java.net/~alanb/6432567/webrev.00/
Once we agree the changes, then this is something we can push to 6open
aswell.
Thanks,
Alan.
[1] https://bugs.openjdk.java.net/show_bug.cgi?id=100103