diff options
author | Florian Weimer <fweimer@redhat.com> | 2020-05-19 12:03:44 +0200 |
---|---|---|
committer | Florian Weimer <fweimer@redhat.com> | 2020-06-02 11:59:26 +0200 |
commit | ba9f6ee9bb8a894c9e2fb715edf693dd157b420a (patch) | |
tree | aeb011ca2517f47f8dbcb9ca0a99f2148c30ff50 /sysdeps/unix/sysv/linux | |
parent | ec41af45a6d25f70f9c7ea15cb831a2b2fea3855 (diff) | |
download | glibc-ba9f6ee9bb8a894c9e2fb715edf693dd157b420a.tar.gz glibc-ba9f6ee9bb8a894c9e2fb715edf693dd157b420a.tar.xz glibc-ba9f6ee9bb8a894c9e2fb715edf693dd157b420a.zip |
Linux: Use __pthread_attr_setsigmask_internal for timer helper thread
timer_create needs to create threads with all signals blocked, including SIGTIMER (which happens to equal SIGCANCEL). Fixes commit b3cae39dcbfa2432b3f3aa28854d8ac57f0de1b8 ("nptl: Start new threads with all signals blocked [BZ #25098]"). Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Diffstat (limited to 'sysdeps/unix/sysv/linux')
-rw-r--r-- | sysdeps/unix/sysv/linux/timer_routines.c | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/sysdeps/unix/sysv/linux/timer_routines.c b/sysdeps/unix/sysv/linux/timer_routines.c index 63083f6f91..86fad2a2c0 100644 --- a/sysdeps/unix/sysv/linux/timer_routines.c +++ b/sysdeps/unix/sysv/linux/timer_routines.c @@ -136,23 +136,24 @@ __start_helper_thread (void) (void) pthread_attr_init (&attr); (void) pthread_attr_setstacksize (&attr, __pthread_get_minstack (&attr)); - /* Block all signals in the helper thread but SIGSETXID. To do this - thoroughly we temporarily have to block all signals here. The - helper can lose wakeups if SIGTIMER is not blocked throughout. */ + /* Block all signals in the helper thread but SIGSETXID. */ sigset_t ss; - __libc_signal_block_app (&ss); - __libc_signal_block_sigtimer (NULL); + __sigfillset (&ss); + __sigdelset (&ss, SIGSETXID); + int res = __pthread_attr_setsigmask_internal (&attr, &ss); + if (res != 0) + { + pthread_attr_destroy (&attr); + return; + } /* Create the helper thread for this timer. */ pthread_t th; - int res = pthread_create (&th, &attr, timer_helper_thread, NULL); + res = pthread_create (&th, &attr, timer_helper_thread, NULL); if (res == 0) /* We managed to start the helper thread. */ __helper_tid = ((struct pthread *) th)->tid; - /* Restore the signal mask. */ - __libc_signal_restore_set (&ss); - /* No need for the attribute anymore. */ (void) pthread_attr_destroy (&attr); |