Hi Simon,
Nice catch!
Looks good.
Copyright comment needs an update.
Thanks,
Serguei
On 12/4/18 08:42, Daniel D. Daugherty wrote:
On 12/4/18 11:02 AM, Severin Gehwolf wrote:
Hi,
On Tue, 2018-12-04 at 10:08 -0500, Daniel D. Daugherty wrote:
On 12/4/18 9:57 AM, Simon Tooke wrote:
In one fatal error code path, snprintf() is given its output buffer
asan
input string.
src/jdk.jdwp.agent/share/native/libjdwp/debugInit.c line 645:
(void)snprintf(buf, sizeof(buf), "JDWP %s", buf);
Proposed fix is of course:
(void)snprintf(buf, sizeof(buf), "JDWP %s", msg);
This was found by compiling with GCC 8.1
Bug: https://bugs.openjdk.java.net/browse/JDK-8214061
webrev:
http://cr.openjdk.java.net/~sgehwolf/webrevs/stooke/JDK-8214061/01/webrev/
src/jdk.jdwp.agent/share/native/libjdwp/debugInit.c
No comments.
Thumbs up.
Can this be considered trivial enough to only require one reviewer?
Yes. This is trivial so only one (R)eviewer is needed.
FWIW, this looks good to me too, but I'm not a Reviewer.
That's okay. You can still be listed as a (r)eviewer. So now it
has 2 folks... :-)
Dan
Thanks,
Severin
