Make the conditional, which decides what siglist to put a signal on,

more concise and improve the comment.

Submitted by: bde

1 files changed, 5 insertions, 8 deletions

diff --git a/sys/kern/kern_sig.c b/sys/kern/kern_sig.c
index 8d22399..099a7d0 100644
--- a/sys/kern/kern_sig.c
+++ b/sys/kern/kern_sig.c
@@ -1655,15 +1655,12 @@ do_tdsignal(struct thread *td, int sig, sigtarget_t target)
 	prop = sigprop(sig);
 
 	/*
-	 * If this thread is blocking this signal then we'll leave it in the
-	 * proc so that we can find it in the first thread that unblocks
-	 * it--  unless the signal is meant for the thread and not the process.
+	 * If the signal is blocked and not destined for this thread, then
+	 * assign it to the process so that we can find it later in the first
+	 * thread that unblocks it.  Otherwise, assign it to this thread now.
 	 */
-	if (target == SIGTARGET_P)
-		siglist = SIGISMEMBER(td->td_sigmask, sig) ?
-			      &p->p_siglist : &td->td_siglist;
-	else
-		siglist = &td->td_siglist;
+	siglist = (target != SIGTARGET_TD && SIGISMEMBER(td->td_sigmask, sig)) ?
+	    &p->p_siglist : &td->td_siglist;
 
 	/*
 	 * If proc is traced, always give parent a chance;