diff options
| author | Roland McGrath <roland@redhat.com> | 2007-11-12 18:41:55 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-11-12 19:05:23 -0500 |
| commit | 325d22df7b19e0116aff3391d3a03f73d0634ded (patch) | |
| tree | 25a46ecf33feb35f243bfaeb36510892789ff8a8 /kernel | |
| parent | 6fa02839bf9412e18e773d04e96182b4cd0b5d57 (diff) | |
sigwait eats blocked default-ignore signals
While a signal is blocked, it must be posted even if its action is
SIG_IGN or is SIG_DFL with the default action to ignore. This works
right most of the time, but is broken when a sigwait (rt_sigtimedwait)
is in progress. This changes the early-discard check to respect
real_blocked. ~blocked is the set to check for "should wake up now",
but ~(blocked|real_blocked) is the set for "blocked" semantics as
defined by POSIX.
This fixes bugzilla entry 9347, see
http://bugzilla.kernel.org/show_bug.cgi?id=9347
Signed-off-by: Roland McGrath <roland@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/signal.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/signal.c b/kernel/signal.c index 909a0cc6bc70..afa4f781f924 100644 --- a/kernel/signal.c +++ b/kernel/signal.c | |||
| @@ -55,7 +55,7 @@ static int sig_ignored(struct task_struct *t, int sig) | |||
| 55 | * signal handler may change by the time it is | 55 | * signal handler may change by the time it is |
| 56 | * unblocked. | 56 | * unblocked. |
| 57 | */ | 57 | */ |
| 58 | if (sigismember(&t->blocked, sig)) | 58 | if (sigismember(&t->blocked, sig) || sigismember(&t->real_blocked, sig)) |
| 59 | return 0; | 59 | return 0; |
| 60 | 60 | ||
| 61 | /* Is it explicitly or implicitly ignored? */ | 61 | /* Is it explicitly or implicitly ignored? */ |