diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2012-04-26 23:25:06 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-05-21 14:20:27 -0400 |
| commit | 5a30a798fb501429522a30b6027a603244d659b2 (patch) | |
| tree | 17a0e45af5c3b32f0a7005a094ad613b20ec6d61 /arch/um/kernel/signal.c | |
| parent | d50349b0c397407458ea8c57aee765d158e6f9ee (diff) | |
| download | blackbird-op-linux-5a30a798fb501429522a30b6027a603244d659b2.tar.gz blackbird-op-linux-5a30a798fb501429522a30b6027a603244d659b2.zip | |
um: wrong sigmask saved in case of multiple sigframes
we can't just find oldmask once; if there are multiple signals
and we loop building sigframes for those, ->saved_mask will be
definitely wrong for all but the first one.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'arch/um/kernel/signal.c')
| -rw-r--r-- | arch/um/kernel/signal.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/arch/um/kernel/signal.c b/arch/um/kernel/signal.c index 0dfcef92ec91..187118fbe1bc 100644 --- a/arch/um/kernel/signal.c +++ b/arch/um/kernel/signal.c @@ -74,15 +74,14 @@ static int kern_do_signal(struct pt_regs *regs) { struct k_sigaction ka_copy; siginfo_t info; - sigset_t *oldset; int sig, handled_sig = 0; - if (test_thread_flag(TIF_RESTORE_SIGMASK)) - oldset = ¤t->saved_sigmask; - else - oldset = ¤t->blocked; - while ((sig = get_signal_to_deliver(&info, &ka_copy, regs, NULL)) > 0) { + sigset_t *oldset; + if (test_thread_flag(TIF_RESTORE_SIGMASK)) + oldset = ¤t->saved_sigmask; + else + oldset = ¤t->blocked; handled_sig = 1; /* Whee! Actually deliver the signal. */ if (!handle_signal(regs, sig, &ka_copy, &info, oldset)) { |
