in clock_getres, check for null pointer before storing result

[musl] / src / signal / sigaction.c

diff --git a/src/signal/sigaction.c b/src/signal/sigaction.c
index d5f4774..0544508 100644 (file)
--- a/src/signal/sigaction.c
+++ b/src/signal/sigaction.c
@@ -4,9 +4,14 @@
 #include "syscall.h"
 #include "pthread_impl.h"
 #include "libc.h"
+#include "lock.h"
 #include "ksigaction.h"
 
-void __restore(), __restore_rt();
+volatile int dummy_lock[1] = { 0 };
+
+extern hidden volatile int __abort_lock[1];
+
+weak_alias(dummy_lock, __abort_lock);
 
 static int unmask_done;
 static unsigned long handler_set[_NSIG/(8*sizeof(long))];
@@ -16,13 +21,12 @@ void __get_handler_set(sigset_t *set)
        memcpy(set, handler_set, sizeof handler_set);
 }
 
+volatile int __eintr_valid_flag;
+
 int __libc_sigaction(int sig, const struct sigaction *restrict sa, struct sigaction *restrict old)
 {
        struct k_sigaction ksa, ksa_old;
-       if (sig >= (unsigned)_NSIG) {
-               errno = EINVAL;
-               return -1;
-       }
+       unsigned long set[_NSIG/(8*sizeof(long))];
        if (sa) {
                if ((uintptr_t)sa->sa_handler > 1UL) {
                        a_or_l(handler_set+(sig-1)/(8*sizeof(long)),
@@ -41,25 +45,40 @@ int __libc_sigaction(int sig, const struct sigaction *restrict sa, struct sigact
                                        SIGPT_SET, 0, _NSIG/8);
                                unmask_done = 1;
                        }
+
+                       if (!(sa->sa_flags & SA_RESTART)) {
+                               a_store(&__eintr_valid_flag, 1);
+                       }
+               }
+               /* Changing the disposition of SIGABRT to anything but
+                * SIG_DFL requires a lock, so that it cannot be changed
+                * while abort is terminating the process after simply
+                * calling raise(SIGABRT) failed to do so. */
+               if (sa->sa_handler != SIG_DFL && sig == SIGABRT) {
+                       __block_all_sigs(&set);
+                       LOCK(__abort_lock);
                }
                ksa.handler = sa->sa_handler;
                ksa.flags = sa->sa_flags | SA_RESTORER;
                ksa.restorer = (sa->sa_flags & SA_SIGINFO) ? __restore_rt : __restore;
-               memcpy(&ksa.mask, &sa->sa_mask, sizeof ksa.mask);
+               memcpy(&ksa.mask, &sa->sa_mask, _NSIG/8);
        }
-       if (syscall(SYS_rt_sigaction, sig, sa?&ksa:0, old?&ksa_old:0, sizeof ksa.mask))
-               return -1;
-       if (old) {
+       int r = __syscall(SYS_rt_sigaction, sig, sa?&ksa:0, old?&ksa_old:0, _NSIG/8);
+       if (sig == SIGABRT && sa && sa->sa_handler != SIG_DFL) {
+               UNLOCK(__abort_lock);
+               __restore_sigs(&set);
+       }
+       if (old && !r) {
                old->sa_handler = ksa_old.handler;
                old->sa_flags = ksa_old.flags;
-               memcpy(&old->sa_mask, &ksa_old.mask, sizeof ksa_old.mask);
+               memcpy(&old->sa_mask, &ksa_old.mask, _NSIG/8);
        }
-       return 0;
+       return __syscall_ret(r);
 }
 
 int __sigaction(int sig, const struct sigaction *restrict sa, struct sigaction *restrict old)
 {
-       if (sig-32U < 3) {
+       if (sig-32U < 3 || sig-1U >= _NSIG-1) {
                errno = EINVAL;
                return -1;
        }