From 69c1dfc1a7964cc15d429350905aea821803a40c Mon Sep 17 00:00:00 2001 From: Andreas Schwab Date: Fri, 25 Jun 2010 10:41:17 +0200 Subject: [PATCH] Fix setxid race handling exiting threads (cherry picked from commit 523df511514331a7fa0668b37a917b52f71684af) --- nptl/ChangeLog | 5 +++++ nptl/allocatestack.c | 11 ++++++++++- 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/nptl/ChangeLog b/nptl/ChangeLog index 7411ae5e6f..c4ae3cb38b 100644 --- a/nptl/ChangeLog +++ b/nptl/ChangeLog @@ -1,3 +1,8 @@ +2011-08-31 Andreas Schwab + + * allocatestack.c (setxid_mark_thread): Ensure that the exiting + thread is woken up. + 2011-08-08 Andreas Schwab * sysdeps/unix/sysv/linux/x86_64/cancellation.S: Maintain aligned diff --git a/nptl/allocatestack.c b/nptl/allocatestack.c index 82408f5178..d4364bf218 100644 --- a/nptl/allocatestack.c +++ b/nptl/allocatestack.c @@ -999,7 +999,16 @@ setxid_mark_thread (struct xid_command *cmdp, struct pthread *t) /* If the thread is exiting right now, ignore it. */ if ((ch & EXITING_BITMASK) != 0) - return; + { + /* Release the futex if there is no other setxid in + progress. */ + if ((ch & SETXID_BITMASK) == 0) + { + t->setxid_futex = 1; + lll_futex_wake (&t->setxid_futex, 1, LLL_PRIVATE); + } + return; + } } while (atomic_compare_and_exchange_bool_acq (&t->cancelhandling, ch | SETXID_BITMASK, ch));