Changeset 860 for vendor/current/lib/tevent

Timestamp:

May 12, 2014, 8:58:38 PM (11 years ago)

Author:

Silvan Scherrer

Message:

Samba 3.6: updated vendor to latest version

File:

• vendor/current/lib/tevent/tevent_signal.c (modified) (1 diff)

vendor/current/lib/tevent/tevent_signal.c

@@ -123 +123 @@
                 /* we've filled the info array - block this signal until
                    these ones are delivered */
+#ifdef HAVE_UCONTEXT_T
+                /*
+                 * This is the only way for this to work.
+                 * By default signum is blocked inside this
+                 * signal handler using a temporary mask,
+                 * but what we really need to do now is
+                 * block it in the callers mask, so it
+                 * stays blocked when the temporary signal
+                 * handler mask is replaced when we return
+                 * from here. The callers mask can be found
+                 * in the ucontext_t passed in as the
+                 * void *uctx argument.
+                 */
+                ucontext_t *ucp = (ucontext_t *)uctx;
+                sigaddset(&ucp->uc_sigmask, signum);
+#else
+                /*
+                 * WARNING !!! WARNING !!!!
+                 *
+                 * This code doesn't work.
+                 * By default signum is blocked inside this
+                 * signal handler, but calling sigprocmask
+                 * modifies the temporary signal mask being
+                 * used *inside* this handler, which will be
+                 * replaced by the callers signal mask once
+                 * we return from here. See Samba
+                 * bug #9550 for details.
+                 */
                 sigset_t set;
                 sigemptyset(&set);
                 sigaddset(&set, signum);
                 sigprocmask(SIG_BLOCK, &set, NULL);
+#endif
                 TEVENT_SIG_INCREMENT(sig_state->sig_blocked[signum]);
         }