From fb6cf945f91f2b39edbe838eacb4fad85d153376 Mon Sep 17 00:00:00 2001 From: Christopher Faylor Date: Fri, 19 Jul 2013 23:13:58 +0000 Subject: [PATCH] * exceptions.cc (signal_exit): Add a FIXME comment. --- winsup/cygwin/ChangeLog | 4 ++++ winsup/cygwin/exceptions.cc | 2 ++ winsup/cygwin/release/1.7.22 | 4 ++++ 3 files changed, 10 insertions(+) diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index c36862ce8..aa970d2f0 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,7 @@ +2013-07-19 Christopher Faylor + + * exceptions.cc (signal_exit): Add a FIXME comment. + 2013-07-19 Christopher Faylor * exceptions.cc (signal_exit): Only dump core when it's a "kernel" diff --git a/winsup/cygwin/exceptions.cc b/winsup/cygwin/exceptions.cc index 3f1cdbfdb..0b7053cb2 100644 --- a/winsup/cygwin/exceptions.cc +++ b/winsup/cygwin/exceptions.cc @@ -1219,6 +1219,8 @@ signal_exit (int sig, siginfo_t *si) sig |= 0x80; /* Flag that we've "dumped core" */ if (try_to_debug ()) break; + /* FIXME: We're still dumping core even if !(sig & 0x80). Need to + investigate if the else clause can just be nuked. */ if (si->si_code != SI_USER && si->si_cyg) ((cygwin_exception *) si->si_cyg)->dumpstack (); else diff --git a/winsup/cygwin/release/1.7.22 b/winsup/cygwin/release/1.7.22 index 16fbe18ff..d5b7e8355 100644 --- a/winsup/cygwin/release/1.7.22 +++ b/winsup/cygwin/release/1.7.22 @@ -31,3 +31,7 @@ Bug fixes: - Fix a timer handle leak in sleep family of functions. Fixes: http://cygwin.com/ml/cygwin/2013-07/msg00379.html + +- Fix issue with raise() not causing the process to exit with a signal + status. Also only set "dumped core" flag when it's a "kernel" signal. + See: http://cygwin.com/ml/cygwin-apps/2013-07/msg00251.html