[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 1/6] linux-user: Recheck for pending synchronous sig
From: |
Peter Maydell |
Subject: |
[Qemu-devel] [PATCH 1/6] linux-user: Recheck for pending synchronous signals too |
Date: |
Thu, 28 Jul 2016 16:44:45 +0100 |
In process_pending_signals() we restart the scan of possible
pending signals after calling handle_pending_signal() in
case some other signal has been generated. This rescan
should also include a check for a new synchronous signal
since those are in fact the only kind of new signal that
the signal frame setup process might produce.
Signed-off-by: Peter Maydell <address@hidden>
---
linux-user/signal.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/linux-user/signal.c b/linux-user/signal.c
index 85976da..87871ce 100644
--- a/linux-user/signal.c
+++ b/linux-user/signal.c
@@ -5925,6 +5925,7 @@ void process_pending_signals(CPUArchState *cpu_env)
sigfillset(&set);
sigprocmask(SIG_SETMASK, &set, 0);
+ restart_scan:
sig = ts->sync_signal.pending;
if (sig) {
/* Synchronous signals are forced,
@@ -5952,8 +5953,10 @@ void process_pending_signals(CPUArchState *cpu_env)
(!sigismember(blocked_set,
target_to_host_signal_table[sig]))) {
handle_pending_signal(cpu_env, sig, &ts->sigtab[sig - 1]);
- /* Restart scan from the beginning */
- sig = 1;
+ /* Restart scan from the beginning, as handle_pending_signal
+ * might have resulted in a new synchronous signal (eg
SIGSEGV).
+ */
+ goto restart_scan;
}
}
--
1.9.1
- [Qemu-devel] [PATCH 0/6] linux-user: Handle SEGV on signal entry/exit correctly, Peter Maydell, 2016/07/28
- [Qemu-devel] [PATCH 4/6] linux-user: ARM: Give SIGSEGV if signal frame setup fails, Peter Maydell, 2016/07/28
- [Qemu-devel] [PATCH 3/6] linux-user: SIGSEGV on signal entry need not be fatal, Peter Maydell, 2016/07/28
- [Qemu-devel] [PATCH 6/6] linux-user: Implement force_sigsegv() via force_sig(), Peter Maydell, 2016/07/28
- [Qemu-devel] [PATCH 2/6] linux-user: Pass si_type information to queue_signal() explicitly, Peter Maydell, 2016/07/28
- [Qemu-devel] [PATCH 1/6] linux-user: Recheck for pending synchronous signals too,
Peter Maydell <=
- [Qemu-devel] [PATCH 5/6] linux-user: SIGSEGV from sigreturn need not be fatal, Peter Maydell, 2016/07/28