diff options
| author | Florian Weimer <fweimer@redhat.com> | 2022-08-03 11:41:53 +0200 |
|---|---|---|
| committer | Florian Weimer <fweimer@redhat.com> | 2022-08-03 11:42:49 +0200 |
| commit | 68e036f27f31c3378201702e182246504fb00f87 (patch) | |
| tree | bd071e5243db8656f95624c262d271471d45562b | |
| parent | cca9684f2d7a74fc0b28bfb1859955e0e28d7b4b (diff) | |
nptl: Remove uses of assert_perror
__pthread_sigmask cannot actually fail with valid pointer arguments
(it would need a really broken seccomp filter), and we do not check
for errors elsewhere.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
| -rw-r--r-- | sysdeps/nptl/gai_misc.h | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/sysdeps/nptl/gai_misc.h b/sysdeps/nptl/gai_misc.h index c09350c2ed..4196eac6e6 100644 --- a/sysdeps/nptl/gai_misc.h +++ b/sysdeps/nptl/gai_misc.h @@ -81,9 +81,7 @@ __gai_start_notify_thread (void) { sigset_t ss; sigemptyset (&ss); - int sigerr __attribute__ ((unused)); - sigerr = __pthread_sigmask (SIG_SETMASK, &ss, NULL); - assert_perror (sigerr); + (void) __pthread_sigmask (SIG_SETMASK, &ss, NULL); } extern inline int @@ -106,15 +104,12 @@ __gai_create_helper_thread (pthread_t *threadp, void *(*tf) (void *), sigset_t ss; sigset_t oss; sigfillset (&ss); - int sigerr __attribute__ ((unused)); - sigerr = __pthread_sigmask (SIG_SETMASK, &ss, &oss); - assert_perror (sigerr); + (void) __pthread_sigmask (SIG_SETMASK, &ss, &oss); int ret = __pthread_create (threadp, &attr, tf, arg); /* Restore the signal mask. */ - sigerr = __pthread_sigmask (SIG_SETMASK, &oss, NULL); - assert_perror (sigerr); + (void) __pthread_sigmask (SIG_SETMASK, &oss, NULL); (void) __pthread_attr_destroy (&attr); return ret; |