<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 20, 2017 at 9:43 PM, Adam Jackson <span dir="ltr"><<a href="mailto:ajax@redhat.com" target="_blank">ajax@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">SIGQUIT is a normal termination request, but any other signal we handle<br>
here wants a core. This has the effect of making FatalError's call to<br>
AbortServer trigger the<br>
<br>
    if (CoreDump)<br>
        OsAbort();<br>
<br>
path. This will allow us to remove some DDX code that has the same net<br>
effect.<br>
<br>
Signed-off-by: Adam Jackson <<a href="mailto:ajax@redhat.com">ajax@redhat.com</a>><br>
---<br>
 os/osinit.c | 6 ++++--<br>
 1 file changed, 4 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/os/osinit.c b/os/osinit.c<br>
index cd769d181c..8575319fff 100644<br>
--- a/os/osinit.c<br>
+++ b/os/osinit.c<br>
@@ -52,6 +52,7 @@ SOFTWARE.<br>
 #include <X11/X.h><br>
 #include "os.h"<br>
 #include "osdep.h"<br>
+#include "opaque.h"<br>
 #include <X11/Xos.h><br>
 #include <signal.h><br>
 #include <errno.h><br>
@@ -74,8 +75,6 @@ SOFTWARE.<br>
 #define ADMPATH "/usr/adm/X%smsgs"<br>
 #endif<br>
<br>
-extern char *display;<br>
-<br>
 #ifdef RLIMIT_DATA<br>
 int limitDataSpace = -1;<br>
 #endif<br>
@@ -151,6 +150,9 @@ OsSigHandler(int signo)<br>
     }<br>
 #endif<br>
<br>
+    if (signo != SIGQUIT)<br>
+        CoreDump = TRUE;<br>
+<br>
     FatalError("Caught signal %d (%s). Server aborting\n",<br>
                signo, strsignal(signo));<br>
 }<br>
<span class="gmail-HOEnZb"><font color="#888888">--<br>
2.14.3<br>
<br>
______________________________<wbr>_________________<br>
<a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
Archives: <a href="http://lists.x.org/archives/xorg-devel" rel="noreferrer" target="_blank">http://lists.x.org/archives/<wbr>xorg-devel</a><br>
Info: <a href="https://lists.x.org/mailman/listinfo/xorg-devel" rel="noreferrer" target="_blank">https://lists.x.org/mailman/<wbr>listinfo/xorg-devel</a></font></span></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_extra">If we set Coredump to TRUE here for all signals but SIGQUIT, do we still need the “-core” command line option?</div><div class="gmail_extra"><br></div><div class="gmail_extra">Anyway, LGTM.</div><div class="gmail_extra"><br></div>Reviewed-by: Olivier Fourdan <<a href="mailto:ofourdan@redhat.com" target="_blank">ofourdan@redhat.com</a>></div></div>