diff options
| author | Corinna Vinschen <corinna@vinschen.de> | 2015-06-27 12:08:07 +0200 |
|---|---|---|
| committer | Corinna Vinschen <corinna@vinschen.de> | 2015-06-27 12:08:07 +0200 |
| commit | c2c1b843092768bd922f587bba9107c97f2a7ade (patch) | |
| tree | 2c73294a2c042a9a8d071eae457245857c4083f8 /winsup/cygwin/signal.cc | |
| parent | ba170254e6c2a81e847779f719ff61912437444e (diff) | |
| download | cygnal-c2c1b843092768bd922f587bba9107c97f2a7ade.tar.gz cygnal-c2c1b843092768bd922f587bba9107c97f2a7ade.tar.bz2 cygnal-c2c1b843092768bd922f587bba9107c97f2a7ade.zip | |
Handle ss_flags value longjmp safe.
* exceptions.cc (_cygtls::call_signal_handler): Drop manipulating
thread's ss_flags here. It's not safe against longjmp.
* signal.cc (sigaltstack): Check if we're running on the alternate
stack and set ss_flags returned in oss to SS_ONSTACK.
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
Diffstat (limited to 'winsup/cygwin/signal.cc')
| -rw-r--r-- | winsup/cygwin/signal.cc | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/winsup/cygwin/signal.cc b/winsup/cygwin/signal.cc index 94faaf308..467c919c3 100644 --- a/winsup/cygwin/signal.cc +++ b/winsup/cygwin/signal.cc @@ -667,7 +667,17 @@ sigaltstack (const stack_t *ss, stack_t *oss) } } if (oss) - memcpy (oss, &me.altstack, sizeof *oss); + { + char stack_marker; + memcpy (oss, &me.altstack, sizeof *oss); + if (!me.altstack.ss_flags && me.altstack.ss_sp) + { + if (&stack_marker >= (char *) me.altstack.ss_sp + && &stack_marker < (char *) me.altstack.ss_sp + + me.altstack.ss_size) + oss->ss_flags = SS_ONSTACK; + } + } } __except (EFAULT) { |