[PATCH 2/3] nptl: Remove uses of assert_perror
Adhemerval Zanella Netto
adhemerval.zanella@linaro.org
Mon Aug 1 20:01:15 GMT 2022
On 01/08/22 07:45, Florian Weimer via Libc-alpha wrote:
> __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.
LGTM, thanks.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
> ---
> sysdeps/nptl/gai_misc.h | 11 +++--------
> 1 file 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;
More information about the Libc-alpha
mailing list