mirror of
https://sourceware.org/git/glibc.git
synced 2024-11-12 16:20:06 +00:00
27d83441a2
The behavior of the signal mask on threads created by timer_create for SIGEV_THREAD timers are implementation-defined and glibc explicit unblocks all signals before calling the user-defined function. This behavior, although not incorrect standard-wise, opens a race if a program using a blocked rt-signal plus sigwaitinfo (and without an installed signal handler for the rt-signal) receives a signal while executing the used-defined function for SIGEV_THREAD. A better alternative discussed in bug report is to rather block all signals (besides the internal ones not available to application usage). This patch fixes this issue by only unblocking SIGSETXID (used on set*uid function) and SIGCANCEL (used for thread cancellation). Checked on x86_64-linux-gnu and i686-linux-gnu.
79 lines
2.2 KiB
C
79 lines
2.2 KiB
C
/* Check resulting signal mask from POSIX timer using SIGEV_THREAD.
|
|
Copyright (C) 2020 Free Software Foundation, Inc.
|
|
This file is part of the GNU C Library.
|
|
|
|
The GNU C Library is free software; you can redistribute it and/or
|
|
modify it under the terms of the GNU Lesser General Public
|
|
License as published by the Free Software Foundation; either
|
|
version 2.1 of the License, or (at your option) any later version.
|
|
|
|
The GNU C Library is distributed in the hope that it will be useful,
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
Lesser General Public License for more details.
|
|
|
|
You should have received a copy of the GNU Lesser General Public
|
|
License along with the GNU C Library; if not, see
|
|
<https://www.gnu.org/licenses/>. */
|
|
|
|
#include <stdio.h>
|
|
#include <time.h>
|
|
#include <signal.h>
|
|
#include <stdbool.h>
|
|
|
|
#include <support/check.h>
|
|
#include <support/test-driver.h>
|
|
#include <support/xthread.h>
|
|
|
|
#include <internal-signals.h>
|
|
|
|
static pthread_barrier_t barrier;
|
|
|
|
static void
|
|
thread_handler (union sigval sv)
|
|
{
|
|
sigset_t ss;
|
|
sigprocmask (SIG_BLOCK, NULL, &ss);
|
|
if (test_verbose > 0)
|
|
printf ("%s: blocked signal mask = { ", __func__);
|
|
for (int sig = 1; sig < NSIG; sig++)
|
|
{
|
|
/* POSIX timers threads created to handle SIGEV_THREAD block all
|
|
signals except SIGKILL, SIGSTOP and glibc internals ones. */
|
|
if (sigismember (&ss, sig))
|
|
{
|
|
TEST_VERIFY (sig != SIGKILL && sig != SIGSTOP);
|
|
TEST_VERIFY (!__is_internal_signal (sig));
|
|
}
|
|
if (test_verbose && sigismember (&ss, sig))
|
|
printf ("%d, ", sig);
|
|
}
|
|
if (test_verbose > 0)
|
|
printf ("}\n");
|
|
|
|
xpthread_barrier_wait (&barrier);
|
|
}
|
|
|
|
static int
|
|
do_test (void)
|
|
{
|
|
struct sigevent sev = { 0 };
|
|
sev.sigev_notify = SIGEV_THREAD;
|
|
sev.sigev_notify_function = &thread_handler;
|
|
|
|
timer_t timerid;
|
|
TEST_COMPARE (timer_create (CLOCK_REALTIME, &sev, &timerid), 0);
|
|
|
|
xpthread_barrier_init (&barrier, NULL, 2);
|
|
|
|
struct itimerspec trigger = { 0 };
|
|
trigger.it_value.tv_nsec = 1000000;
|
|
TEST_COMPARE (timer_settime (timerid, 0, &trigger, NULL), 0);
|
|
|
|
xpthread_barrier_wait (&barrier);
|
|
|
|
return 0;
|
|
}
|
|
|
|
#include <support/test-driver.c>
|