
<div dir="ltr">Hi Staffan,<div><br></div><div>this looks fine now, thanks!</div><div><br></div><div>..Thomas</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 10, 2015 at 3:42 PM, Staffan Larsen <span dir="ltr"><<a href="mailto:staffan.larsen@oracle.com" target="_blank">staffan.larsen@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi Thomas,Â <div><br><div><span class=""><blockquote type="cite"><div>On 10 dec. 2015, at 15:27, Thomas StÃ¼fe <<a href="mailto:thomas.stuefe@gmail.com" target="_blank">thomas.stuefe@gmail.com</a>> wrote:</div><br><div><div dir="ltr">Hi Staffan,<div><br></div><div>thanks!</div><div><br></div><div>It also just occurred to me that strerror is not considered threadsafe. Maybe strerror_r() would a better option (depending on the version you use you would have to check the return value for NULL, though).</div></div></div></blockquote><div><br></div></span><div>Updated here:Â <a href="http://cr.openjdk.java.net/~sla/JDK-8145099/webrev.03/" target="_blank">http://cr.openjdk.java.net/~sla/JDK-8145099/webrev.03/</a></div><div><br></div><div><div>Thanks,</div><div>/Staffan</div></div><span class=""><br><blockquote type="cite"><div><div dir="ltr"><div><br></div><div>sorry for this piecemeal review.</div><div><br></div><div>Kind Regards, Thomas</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 10, 2015 at 3:22 PM, Staffan Larsen <span dir="ltr"><<a href="mailto:staffan.larsen@oracle.com" target="_blank">staffan.larsen@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi Thommas,<div><br><div><span><blockquote type="cite"><div>On 10 dec. 2015, at 14:49, Thomas StÃ¼fe <<a href="mailto:thomas.stuefe@gmail.com" target="_blank">thomas.stuefe@gmail.com</a>> wrote:</div><br><div><div dir="ltr">Hi Stefan,<div><br></div><div>Disclaimer: not a "R"eviewer.</div><div><br></div><div><a href="http://cr.openjdk.java.net/~sla/JDK-8145099/webrev.01/agent/src/os/linux/LinuxDebuggerLocal.c.udiff.html" target="_blank">http://cr.openjdk.java.net/~sla/JDK-8145099/webrev.01/agent/src/os/linux/LinuxDebuggerLocal.c.udiff.html</a><br></div><br>I am not sure why you pass sizeof(err_buf) - 1 instead of sizeof(err_buf) to Pgrab and sizeof(msg) - 1 instead of sizeof(msg) to snprintf?Â snprintf will always zero terminate in case of truncation, at least on posix platforms.</div></div></blockquote><div><br></div></span><div>Good point. I was just being overly cautious without thinking.</div><div><br></div><div>Updated webrev:Â <a href="http://cr.openjdk.java.net/~sla/JDK-8145099/webrev.02/" target="_blank">http://cr.openjdk.java.net/~sla/JDK-8145099/webrev.02/</a></div><div><br></div><div><div>Thanks,</div><div>/Staffan</div></div><span><br><blockquote type="cite"><div><div dir="ltr"><div><br></div><div>Otherwise this looks good.</div><div><br></div><div><div>Kind Regards, Thomas</div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 10, 2015 at 2:19 PM, Staffan Larsen <span dir="ltr"><<a href="mailto:staffan.larsen@oracle.com" target="_blank">staffan.larsen@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Please review this patch to add a better error message to SA when it fails to attach to a process on linux. Currently the error says "Can't attach to the processâ€. After this change the message will look something like: "Can't attach to the process: ptrace(PTRACE_ATTACH, ..) failed for 28417: Operation not permittedâ€<div><br></div><div><div>webrev:Â <a href="http://cr.openjdk.java.net/~sla/JDK-8145099/webrev.01/" target="_blank">http://cr.openjdk.java.net/~sla/JDK-8145099/webrev.01/</a></div><div>bug:Â <a href="https://bugs.openjdk.java.net/browse/JDK-8145099" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8145099</a></div><div><br></div><div>Thanks,</div><div>/Staffan</div><div><br></div><div><br><div><div><br></div></div></div></div></div></blockquote></div><br></div>

</div></blockquote></span></div><br></div></div></blockquote></div><br></div>

</div></blockquote></span></div><br></div></div></blockquote></div><br></div>