<div dir="ltr">Based on in person discussion, this is not an option, because there are users of ptrace that aren't debuggers.  See lsan and heapchecker for example.</div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sat, Oct 26, 2013 at 3:50 AM, Alexander Potapenko <span dir="ltr"><<a href="mailto:glider@google.com" target="_blank">glider@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p dir="ltr">It should be easy to detect whether we're running under gdb (e.g. if we're being ptraced; perhaps something needs to be done in the presence of seccomp-bpf)</p>
<div class="gmail_quote"><div class="im">On Oct 25, 2013 10:55 PM, "Kostya Serebryany" <<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>> wrote:<br type="attribution"></div><blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div class="im">I am not opposed to this change, but I'd like to test it a bit more carefully to make sure none of our users get angry on us for behavior change. <div>I'd suggest to put this s<span style="font-family:arial,sans-serif;font-size:13px">igtrap under a flag (off by default) and then flip the default to true after a bit of testing. </span></div>


<div><span style="font-family:arial,sans-serif;font-size:13px">We already have </span><span style="font-size:13px;font-family:arial,sans-serif">abort_on_error, may add sigtrap_on_error or some such. </span></div>
<div><span style="font-size:13px;font-family:arial,sans-serif"><br></span></div></div><div class="im"><div><span style="font-size:13px;font-family:arial,sans-serif">(may not reply until Mon next time)</span></div>
</div></div><div><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Fri, Oct 25, 2013 at 7:26 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>


</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><p dir="ltr">Sigtrap, which seems less likely to cause deep surprise in the presence of signal handlers.</p>
</div><div>
<div>
<div class="gmail_quote"><div class="im">On Oct 25, 2013 3:56 AM, "Kostya Serebryany" <<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>> wrote:<br type="attribution"></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">


<div dir="ltr">So this will change the default behavior from doing exit(1) to doing abort(), right? </div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Fri, Oct 25, 2013 at 3:16 AM, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>></span> wrote:<br>




</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><div dir="ltr">Running sanitized code under gdb can be annoying because the program will print out its error then exit, instead of the usual behaviour when we get an abort() of stopping the program there and allowing the user to inspect its stack trace.<div>






<br></div><div>Add a trap to the exit path in sanitizer_common. Please review!</div><span><font color="#888888"><div><br></div><div>Nick</div></font></span></div>
<br></div><div class="im">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></div></blockquote></div><br></div><div class="im">
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></div></blockquote></div>
</div></div></blockquote></div><br></div>
</div><div class="im"><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></div></blockquote></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>