m68knommu: Don't lose state if sigframe setup fails
authorAl Viro <viro@zeniv.linux.org.uk>
Tue, 12 Oct 2010 03:35:08 +0000 (23:35 -0400)
committerGeert Uytterhoeven <geert@linux-m68k.org>
Fri, 7 Jan 2011 13:06:59 +0000 (14:06 +0100)
exact parallel to m68k analog

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Acked-by: Greg Ungerer <gerg@uclinux.org>
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
arch/m68knommu/kernel/signal.c

index c070f3f..e00f1bb 100644 (file)
@@ -528,7 +528,7 @@ get_sigframe(struct k_sigaction *ka, struct pt_regs *regs, size_t frame_size)
        return (void *)((usp - frame_size) & -8UL);
 }
 
-static void setup_frame (int sig, struct k_sigaction *ka,
+static int setup_frame (int sig, struct k_sigaction *ka,
                         sigset_t *set, struct pt_regs *regs)
 {
        struct sigframe *frame;
@@ -582,14 +582,14 @@ adjust_stack:
                tregs->pc = regs->pc;
                tregs->sr = regs->sr;
        }
-       return;
+       return err;
 
 give_sigsegv:
        force_sigsegv(sig, current);
        goto adjust_stack;
 }
 
-static void setup_rt_frame (int sig, struct k_sigaction *ka, siginfo_t *info,
+static int setup_rt_frame (int sig, struct k_sigaction *ka, siginfo_t *info,
                            sigset_t *set, struct pt_regs *regs)
 {
        struct rt_sigframe *frame;
@@ -646,7 +646,7 @@ adjust_stack:
                tregs->pc = regs->pc;
                tregs->sr = regs->sr;
        }
-       return;
+       return err;
 
 give_sigsegv:
        force_sigsegv(sig, current);
@@ -693,6 +693,7 @@ static void
 handle_signal(int sig, struct k_sigaction *ka, siginfo_t *info,
              sigset_t *oldset, struct pt_regs *regs)
 {
+       int err;
        /* are we from a system call? */
        if (regs->orig_d0 >= 0)
                /* If so, check system call restarting.. */
@@ -700,9 +701,12 @@ handle_signal(int sig, struct k_sigaction *ka, siginfo_t *info,
 
        /* set up the stack frame */
        if (ka->sa.sa_flags & SA_SIGINFO)
-               setup_rt_frame(sig, ka, info, oldset, regs);
+               err = setup_rt_frame(sig, ka, info, oldset, regs);
        else
-               setup_frame(sig, ka, oldset, regs);
+               err = setup_frame(sig, ka, oldset, regs);
+
+       if (err)
+               return;
 
        spin_lock_irq(&current->sighand->siglock);
        sigorsets(&current->blocked,&current->blocked,&ka->sa.sa_mask);
@@ -710,6 +714,8 @@ handle_signal(int sig, struct k_sigaction *ka, siginfo_t *info,
                sigaddset(&current->blocked,sig);
        recalc_sigpending();
        spin_unlock_irq(&current->sighand->siglock);
+
+       clear_thread_flag(TIF_RESTORE_SIGMASK);
 }
 
 /*
@@ -742,7 +748,6 @@ asmlinkage void do_signal(struct pt_regs *regs)
        if (signr > 0) {
                /* Whee!  Actually deliver the signal.  */
                handle_signal(signr, &ka, &info, oldset, regs);
-               clear_thread_flag(TIF_RESTORE_SIGMASK);
                return;
        }