diff options
| author | Richard Weinberger <richard@nod.at> | 2016-01-25 23:33:30 +0100 |
|---|---|---|
| committer | Richard Weinberger <richard@nod.at> | 2016-03-05 22:16:40 +0100 |
| commit | 322740efbb7ca26df845161e61cc41484b7e328e (patch) | |
| tree | b8caf74ddac8ab94661a6fed1aad6f65e3194fac /arch/um | |
| parent | fc77dbd34c5c99bce46d40a2491937c3bcbd10af (diff) | |
| download | linux-322740efbb7ca26df845161e61cc41484b7e328e.tar.gz linux-322740efbb7ca26df845161e61cc41484b7e328e.tar.bz2 linux-322740efbb7ca26df845161e61cc41484b7e328e.zip | |
Revert "um: Fix get_signal() usage"
Commit db2f24dc240856fb1d78005307f1523b7b3c121b
was plain wrong. I did not realize the we are
allowed to loop here.
In fact we have to loop and must not return to userspace
before all SIGSEGVs have been delivered.
Other archs do this directly in their entry code, UML
does it here.
Reported-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'arch/um')
| -rw-r--r-- | arch/um/kernel/signal.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/um/kernel/signal.c b/arch/um/kernel/signal.c index fc8be0e3a4ff..57acbd67d85d 100644 --- a/arch/um/kernel/signal.c +++ b/arch/um/kernel/signal.c @@ -69,7 +69,7 @@ void do_signal(struct pt_regs *regs) struct ksignal ksig; int handled_sig = 0; - if (get_signal(&ksig)) { + while (get_signal(&ksig)) { handled_sig = 1; /* Whee! Actually deliver the signal. */ handle_signal(&ksig, regs); |