fix error handling for pthread_sigmask
authorRich Felker <dalias@aerifal.cx>
Thu, 10 Mar 2011 01:31:06 +0000 (20:31 -0500)
committerRich Felker <dalias@aerifal.cx>
Thu, 10 Mar 2011 01:31:06 +0000 (20:31 -0500)
it must return errno, not -1, and should reject invalud values for how.

src/signal/sigprocmask.c
src/thread/pthread_sigmask.c [new file with mode: 0644]

index 1bc5baa..96d0847 100644 (file)
@@ -1,4 +1,5 @@
 #include <signal.h>
 #include <signal.h>
+#include <errno.h>
 #include "syscall.h"
 #include "libc.h"
 #include "pthread_impl.h"
 #include "syscall.h"
 #include "libc.h"
 #include "pthread_impl.h"
@@ -11,6 +12,10 @@ int __libc_sigprocmask(int how, const sigset_t *set, sigset_t *old)
 int __sigprocmask(int how, const sigset_t *set, sigset_t *old)
 {
        sigset_t tmp;
 int __sigprocmask(int how, const sigset_t *set, sigset_t *old)
 {
        sigset_t tmp;
+       if (how > 2U) {
+               errno = EINVAL;
+               return -1;
+       }
        /* Disallow blocking thread control signals */
        if (set && how != SIG_UNBLOCK) {
                tmp = *set;
        /* Disallow blocking thread control signals */
        if (set && how != SIG_UNBLOCK) {
                tmp = *set;
@@ -22,4 +27,3 @@ int __sigprocmask(int how, const sigset_t *set, sigset_t *old)
 }
 
 weak_alias(__sigprocmask, sigprocmask);
 }
 
 weak_alias(__sigprocmask, sigprocmask);
-weak_alias(__sigprocmask, pthread_sigmask);
diff --git a/src/thread/pthread_sigmask.c b/src/thread/pthread_sigmask.c
new file mode 100644 (file)
index 0000000..6cc21d2
--- /dev/null
@@ -0,0 +1,10 @@
+#include <signal.h>
+#include <errno.h>
+#include <pthread.h>
+
+int pthread_sigmask(int how, const sigset_t *set, sigset_t *old)
+{
+       int ret = sigprocmask(how, set, old);
+       if (ret) return errno;
+       return 0;
+}