* sysdeps/mach/hurd/sigwait.c (__sigwait): Use __sigandset,
authorRoland McGrath <roland@gnu.org>
Wed, 2 Jan 2002 10:52:56 +0000 (10:52 +0000)
committerRoland McGrath <roland@gnu.org>
Wed, 2 Jan 2002 10:52:56 +0000 (10:52 +0000)
__sigisemptyset.  Don't use MASK uninitialized.

* sysdeps/mach/hurd/sigprocmask.c (__sigprocmask): Use __sigorset.
* hurd/hurdinit.c (_hurd_new_proc_init): Use __sigisemptyset.
* hurd/hurdsig.c (_hurd_internal_post_signal): Use __sigismember,
__sigdelset, __sigaddset, __sigorset.

ChangeLog
hurd/hurdinit.c
hurd/hurdsig.c
sysdeps/mach/hurd/sigprocmask.c
sysdeps/mach/hurd/sigwait.c

index c47eb3b..71380f7 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,13 @@
 2002-01-02  Roland McGrath  <roland@frob.com>
 
+       * sysdeps/mach/hurd/sigwait.c (__sigwait): Use __sigandset,
+       __sigisemptyset.  Don't use MASK uninitialized.
+
+       * sysdeps/mach/hurd/sigprocmask.c (__sigprocmask): Use __sigorset.
+       * hurd/hurdinit.c (_hurd_new_proc_init): Use __sigisemptyset.
+       * hurd/hurdsig.c (_hurd_internal_post_signal): Use __sigismember,
+       __sigdelset, __sigaddset, __sigorset.
+
        * sysdeps/generic/htonl.c: Use uint32_t instead of u_int32_t.
        * sysdeps/generic/htons.c: Use uint16_t instead of u_int16_t.
 
index 7a57031..c7f9d42 100644 (file)
@@ -166,7 +166,7 @@ _hurd_new_proc_init (char **argv,
      while.  Eventually it probably makes most sense for the exec server to
      mask out EXEC_SIGTRAP so the debugged program is closer to not being
      able to tell it's being debugged.  */
-  if (_hurdsig_traced
+  if (!__sigisemptyset (&_hurdsig_traced)
 #ifdef EXEC_SIGTRAP
       && !(_hurd_exec_flags & EXEC_SIGTRAP)
 #endif
index 34b28d9..40b570b 100644 (file)
@@ -588,7 +588,7 @@ _hurd_internal_post_signal (struct hurd_sigstate *ss,
     handler = ss->preemptors ? try_preemptor (ss->preemptors) : SIG_ERR;
 
     /* If no thread-specific preemptor, check for a global one.  */
-    if (handler == SIG_ERR && (__sigmask (signo) & _hurdsig_preempted_set))
+    if (handler == SIG_ERR && __sigismember (signo, _hurdsig_preempted_set))
       {
        __mutex_lock (&_hurd_siglock);
        handler = try_preemptor (_hurdsig_preemptors);
@@ -689,7 +689,7 @@ _hurd_internal_post_signal (struct hurd_sigstate *ss,
       if (__sigmask (signo) & STOPSIGS)
        /* Stop signals clear a pending SIGCONT even if they
           are handled or ignored (but not if preempted).  */
-       ss->pending &= ~sigmask (SIGCONT);
+       __sigdelset (&ss->pending, SIGCONT);
       else
        {
          if (signo == SIGCONT)
@@ -928,11 +928,11 @@ _hurd_internal_post_signal (struct hurd_sigstate *ss,
 
        /* Block requested signals while running the handler.  */
        scp->sc_mask = ss->blocked;
-       ss->blocked |= ss->actions[signo].sa_mask;
+       __sigorset (&ss->blocked, &ss->blocked, &ss->actions[signo].sa_mask);
 
        /* Also block SIGNO unless we're asked not to.  */
        if (! (ss->actions[signo].sa_flags & (SA_RESETHAND | SA_NODEFER)))
-         ss->blocked |= __sigmask (signo);
+         __sigaddset (&ss->blocked, signo);
 
        /* Reset to SIG_DFL if requested.  SIGILL and SIGTRAP cannot
            be automatically reset when delivered; the system silently
index f65dbd5..cbb5ecc 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1991, 92, 93, 94, 95, 96, 97 Free Software Foundation, Inc.
+/* Copyright (C) 1991,92,93,94,95,96,97,2002 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
@@ -49,7 +49,7 @@ __sigprocmask (how, set, oset)
       switch (how)
        {
        case SIG_BLOCK:
-         ss->blocked |= new;
+         __sigorset (&ss->blocked, &ss->blocked, &new);
          break;
 
        case SIG_UNBLOCK:
index 9ddb899..9794076 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1996,97,2001 Free Software Foundation, Inc.
+/* Copyright (C) 1996,97,2001,02 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
@@ -68,13 +68,15 @@ __sigwait (const sigset_t *set, int *sig)
   if (set != NULL)
     /* Crash before locking */
     mask = *set;
+  else
+    __sigemptyset (&mask);
 
   ss = _hurd_self_sigstate ();
   __spin_lock (&ss->lock);
 
   /* See if one of these signals is currently pending.  */
-  ready = ss->pending & mask;
-  if (ready)
+  __sigandset (&ready, &ss->pending, &mask);
+  if (! __sigisemptyset (&ready))
     {
       for (signo = 1; signo < NSIG; signo++)
        if (__sigismember (&ready, signo))