From 513720b73b403c3c9aca3f25c085422f4132c292 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Mon, 1 Dec 2008 18:41:55 +0000 Subject: remove checks for "impossible" errors, clarify uses of unoptimized sigXXX ops (they check signo), use faster sigops where approproate. text data bss dec hex filename - 68 0 0 68 44 libc/signal/sighold.o + 63 0 0 63 3f libc/signal/sighold.o - 114 0 0 114 72 libc/signal/sigintr.o + 110 0 0 110 6e libc/signal/sigintr.o - 113 0 0 113 71 libc/signal/sigpause.o + 108 0 0 108 6c libc/signal/sigpause.o - 68 0 0 68 44 libc/signal/sigrelse.o + 63 0 0 63 3f libc/signal/sigrelse.o --- libc/signal/sighold.c | 5 ++--- libc/signal/sigintr.c | 7 ++----- libc/signal/sigpause.c | 8 +++----- libc/signal/sigrelse.c | 5 ++--- 4 files changed, 9 insertions(+), 16 deletions(-) (limited to 'libc/signal') diff --git a/libc/signal/sighold.c b/libc/signal/sighold.c index f8003d60f..fde0a0c2a 100644 --- a/libc/signal/sighold.c +++ b/libc/signal/sighold.c @@ -30,10 +30,9 @@ int sighold (int sig) sigset_t set; /* Retrieve current signal set. */ - if (sigprocmask (SIG_SETMASK, NULL, &set) < 0) - return -1; + sigprocmask (SIG_SETMASK, NULL, &set); /* can't fail */ - /* Add the specified signal. */ + /* Bound-check sig, add it to the set. */ if (sigaddset (&set, sig) < 0) return -1; diff --git a/libc/signal/sigintr.c b/libc/signal/sigintr.c index 351c82b75..23f87e199 100644 --- a/libc/signal/sigintr.c +++ b/libc/signal/sigintr.c @@ -34,7 +34,7 @@ int siginterrupt (int sig, int interrupt) #ifdef SA_RESTART struct sigaction action; - if (sigaction (sig, (struct sigaction *) NULL, &action) < 0) + if (sigaction (sig, NULL, &action) < 0) return -1; if (interrupt) @@ -48,10 +48,7 @@ int siginterrupt (int sig, int interrupt) action.sa_flags |= SA_RESTART; } - if (sigaction (sig, &action, (struct sigaction *) NULL) < 0) - return -1; - - return 0; + return sigaction (sig, &action, NULL); #else __set_errno (ENOSYS); return -1; diff --git a/libc/signal/sigpause.c b/libc/signal/sigpause.c index 96dd93368..152f9ca32 100644 --- a/libc/signal/sigpause.c +++ b/libc/signal/sigpause.c @@ -40,12 +40,10 @@ int __sigpause (int sig_or_mask, int is_sig) if (is_sig) { -//TODO: error check for sig_or_mask = BIGNUM? - /* The modern X/Open implementation is requested. */ - if (sigprocmask (0, NULL, &set) < 0 - /* Yes, we call `sigdelset' and not `__sigdelset'. */ - || sigdelset (&set, sig_or_mask) < 0) + sigprocmask (SIG_BLOCK, NULL, &set); + /* Bound-check sig_or_mask, remove it from the set. */ + if (sigdelset (&set, sig_or_mask) < 0) return -1; } else diff --git a/libc/signal/sigrelse.c b/libc/signal/sigrelse.c index f5ee6fedc..1a7141ac5 100644 --- a/libc/signal/sigrelse.c +++ b/libc/signal/sigrelse.c @@ -30,10 +30,9 @@ int sigrelse (int sig) sigset_t set; /* Retrieve current signal set. */ - if (sigprocmask (SIG_SETMASK, NULL, &set) < 0) - return -1; + sigprocmask (SIG_SETMASK, NULL, &set); /* can't fail */ - /* Remove the specified signal. */ + /* Bound-check sig, remove it from the set. */ if (sigdelset (&set, sig) < 0) return -1; -- cgit v1.2.3