changeset:   100562:d6d64168ee8c
user:        Victor Stinner <victor.stinner@gmail.com>
date:        Wed Mar 16 23:19:15 2016 +0100
files:       Python/pylifecycle.c
description:
Py_FatalError: disable faulthandler earlier

Issue #26563: Py_FatalError: disable faulthandler before trying to flush
sys.stdout and sys.stderr.


diff -r 5b4f173f0690 -r d6d64168ee8c Python/pylifecycle.c
--- a/Python/pylifecycle.c	Wed Mar 16 22:45:24 2016 +0100
+++ b/Python/pylifecycle.c	Wed Mar 16 23:19:15 2016 +0100
@@ -1371,17 +1371,17 @@
     if (!_Py_FatalError_PrintExc(fd))
         _Py_FatalError_DumpTracebacks(fd);
 
+    /* The main purpose of faulthandler is to display the traceback. We already
+     * did our best to display it. So faulthandler can now be disabled.
+     * (Don't trigger it on abort().) */
+    _PyFaulthandler_Fini();
+
     /* Check if the current Python thread hold the GIL */
     if (PyThreadState_GET() != NULL) {
         /* Flush sys.stdout and sys.stderr */
         flush_std_files();
     }
 
-    /* The main purpose of faulthandler is to display the traceback. We already
-     * did our best to display it. So faulthandler can now be disabled.
-     * (Don't trigger it on abort().) */
-    _PyFaulthandler_Fini();
-
 #ifdef MS_WINDOWS
     len = strlen(msg);