diff options
author | Laurent MEYER <meyerlau@fr.ibm.com> | 2006-06-23 05:05:36 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-23 10:43:05 -0400 |
commit | d09042da7284a86ffbdd18695f517a71514ed598 (patch) | |
tree | 09e9a3de71a4a4cf6c8f8cb0c40ec0a916e01cae /arch/h8300 | |
parent | 785d55708c24c28d7646f3d1fe6c9f82fb714311 (diff) |
[PATCH] fix incorrect SA_ONSTACK behaviour for 64-bit processes
- When setting a sighandler using sigaction() call, if the flag
SA_ONSTACK is set and no alternate stack is provided via sigaltstack(),
the kernel still try to install the alternate stack. This behavior is
the opposite of the one which is documented in Single Unix Specifications
V3.
- Also when setting an alternate stack using sigaltstack() with the flag
SS_DISABLE, the kernel try to install the alternate stack on signal
delivery.
These two use cases makes the process crash at signal delivery.
Signed-off-by: Laurent Meyer <meyerlau@fr.ibm.com>
Cc: Richard Henderson <rth@twiddle.net>
Cc: Ivan Kokshaysky <ink@jurassic.park.msu.ru>
Cc: David Howells <dhowells@redhat.com>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Roman Zippel <zippel@linux-m68k.org>
Cc: Kyle McMartin <kyle@mcmartin.ca>
Cc: Paul Mundt <lethal@linux-sh.org>
Cc: Kazumoto Kojima <kkojima@rr.iij4u.or.jp>
Cc: Chris Zankel <chris@zankel.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/h8300')
-rw-r--r-- | arch/h8300/kernel/signal.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/h8300/kernel/signal.c b/arch/h8300/kernel/signal.c index f13d5e82d4b9..7787f70a05bb 100644 --- a/arch/h8300/kernel/signal.c +++ b/arch/h8300/kernel/signal.c | |||
@@ -307,7 +307,7 @@ get_sigframe(struct k_sigaction *ka, struct pt_regs *regs, size_t frame_size) | |||
307 | 307 | ||
308 | /* This is the X/Open sanctioned signal stack switching. */ | 308 | /* This is the X/Open sanctioned signal stack switching. */ |
309 | if (ka->sa.sa_flags & SA_ONSTACK) { | 309 | if (ka->sa.sa_flags & SA_ONSTACK) { |
310 | if (!on_sig_stack(usp)) | 310 | if (!sas_ss_flags(usp)) |
311 | usp = current->sas_ss_sp + current->sas_ss_size; | 311 | usp = current->sas_ss_sp + current->sas_ss_size; |
312 | } | 312 | } |
313 | return (void *)((usp - frame_size) & -8UL); | 313 | return (void *)((usp - frame_size) & -8UL); |