about summary refs log tree commit diff
path: root/sysdeps/generic
diff options
context:
space:
mode:
authorAdhemerval Zanella <adhemerval.zanella@linaro.org>2019-12-05 14:56:06 -0300
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>2019-12-27 11:18:23 -0300
commit58bd592536cbf491ff6f89b73b5c95beb4859e42 (patch)
tree31c1da8a5bd076f72124e2ac56f66a245f420264 /sysdeps/generic
parent11519fd0c908df2dad50f36a48eb4d0d12ea83a8 (diff)
downloadglibc-58bd592536cbf491ff6f89b73b5c95beb4859e42.tar.gz
glibc-58bd592536cbf491ff6f89b73b5c95beb4859e42.tar.xz
glibc-58bd592536cbf491ff6f89b73b5c95beb4859e42.zip
Fix return code for __libc_signal_* functions
The functions do not fail regardless of the argument value.  Also, for
Linux the return value is not correct on some platforms due the missing
usage of INTERNAL_SYSCALL_ERROR_P / INTERNAL_SYSCALL_ERRNO macros.

Checked on x86_64-linux-gnu, i686-linux-gnu, and sparc64-linux-gnu.
Diffstat (limited to 'sysdeps/generic')
-rw-r--r--sysdeps/generic/internal-signals.h12
1 files changed, 6 insertions, 6 deletions
diff --git a/sysdeps/generic/internal-signals.h b/sysdeps/generic/internal-signals.h
index a515e3e649..41c24dc4b3 100644
--- a/sysdeps/generic/internal-signals.h
+++ b/sysdeps/generic/internal-signals.h
@@ -34,28 +34,28 @@ __clear_internal_signals (sigset_t *set)
 {
 }
 
-static inline int
+static inline void
 __libc_signal_block_all (sigset_t *set)
 {
   sigset_t allset;
   __sigfillset (&allset);
-  return __sigprocmask (SIG_BLOCK, &allset, set);
+  __sigprocmask (SIG_BLOCK, &allset, set);
 }
 
-static inline int
+static inline void
 __libc_signal_block_app (sigset_t *set)
 {
   sigset_t allset;
   __sigfillset (&allset);
   __clear_internal_signals (&allset);
-  return __sigprocmask (SIG_BLOCK, &allset, set);
+  __sigprocmask (SIG_BLOCK, &allset, set);
 }
 
 /* Restore current process signal mask.  */
-static inline int
+static inline void
 __libc_signal_restore_set (const sigset_t *set)
 {
-  return __sigprocmask (SIG_SETMASK, set, NULL);
+  __sigprocmask (SIG_SETMASK, set, NULL);
 }