]> sourceware.org Git - glibc.git/commit
nptl: Fix extraneous testing run by tst-rseq-nptl in the test driver
authorMaciej W. Rozycki <macro@redhat.com>
Fri, 16 Aug 2024 13:38:33 +0000 (14:38 +0100)
committerMaciej W. Rozycki <macro@redhat.com>
Fri, 16 Aug 2024 13:38:33 +0000 (14:38 +0100)
commit9fb237a1c861f3b6a014f7add3f564452ea23e61
tree30d14231deaa6735b6a722c8882166de0b480068
parentb22923abb046311ac9097a36b97b9b97342bac44
nptl: Fix extraneous testing run by tst-rseq-nptl in the test driver

Fix an issue with commit 8f4632deb354 ("Linux: rseq registration tests")
and prevent testing from being run in the process of the test driver
itself rather than just the test child where one has been forked.  The
problem here is the unguarded use of a destructor to call a part of the
testing.  The destructor function, 'do_rseq_destructor_test' is called
implicitly at program completion, however because it is associated with
the executable itself rather than an individual process, it is called
both in the test child *and* in the test driver itself.

Prevent this from happening by providing a guard variable that only
enables test invocation from 'do_rseq_destructor_test' in the process
that has first run 'do_test'.  Consequently extra testing is invoked
from 'do_rseq_destructor_test' only once and in the correct process,
regardless of the use or the lack of of the '--direct' option.  Where
called in the controlling test driver process that has neved called
'do_test' the destructor function silently returns right away without
taking any further actions, letting the test driver fail gracefully
where applicable.

This arrangement prevents 'tst-rseq-nptl' from ever causing testing to
hang forever and never complete, such as currently happening with the
'mips-linux-gnu' (o32 ABI) target.

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
sysdeps/unix/sysv/linux/tst-rseq-nptl.c
This page took 0.038585 seconds and 5 git commands to generate.