about summary refs log tree commit diff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog10
-rw-r--r--sysdeps/mach/hurd/sigwait.c15
2 files changed, 18 insertions, 7 deletions
diff --git a/ChangeLog b/ChangeLog
index 968b321936..67cb599d71 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,15 @@
+2001-12-26  Roland McGrath  <roland@frob.com>
+
+	* sysdeps/mach/hurd/sigwait.c (__sigwait): When returning immediately,
+	check only (SS->pending & MASK) for the signal to return.
+	From Jeroen Dekkers <jeroen@dekkers.cx>.
+
 2001-12-22  Roland McGrath  <roland@frob.com>
 
+	* iconv/skeleton.c [!RESET_INPUT_BUFFER && !SAVE_RESET_STATE]:
+	Use preprocessor #if conditionals instead of `if' to avoid
+	warnings about divide by zero in dead code.
+
 	* hurd/Versions (libc: GLIBC_2.2.5): Add _hurd_port_set,
 	_hurd_port_init, and __hurd_self_sigstate.
 
diff --git a/sysdeps/mach/hurd/sigwait.c b/sysdeps/mach/hurd/sigwait.c
index d44a8a30a7..9ddb89980c 100644
--- a/sysdeps/mach/hurd/sigwait.c
+++ b/sysdeps/mach/hurd/sigwait.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1996, 1997 Free Software Foundation, Inc.
+/* Copyright (C) 1996,97,2001 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
@@ -28,7 +28,7 @@ int
 __sigwait (const sigset_t *set, int *sig)
 {
   struct hurd_sigstate *ss;
-  sigset_t mask;
+  sigset_t mask, ready;
   int signo = 0;
   struct hurd_signal_preemptor preemptor;
   jmp_buf buf;
@@ -72,20 +72,21 @@ __sigwait (const sigset_t *set, int *sig)
   ss = _hurd_self_sigstate ();
   __spin_lock (&ss->lock);
 
-  /* See if one of these signals is currently pending */
-  if (ss->pending & mask)
+  /* See if one of these signals is currently pending.  */
+  ready = ss->pending & mask;
+  if (ready)
     {
       for (signo = 1; signo < NSIG; signo++)
-	if (__sigismember (&ss->pending, signo))
+	if (__sigismember (&ready, signo))
 	  {
-	    __sigdelset (&ss->pending, signo);
+	    __sigdelset (&ready, signo);
 	    goto all_done;
 	  }
       /* Huh?  Where'd it go? */
       abort ();
     }
 
-  /* Wait for one of them to show up */
+  /* Wait for one of them to show up.  */
 
   if (!setjmp (buf))
     {