diff options
| author | Andreas Schwab <schwab@redhat.com> | 2010-06-25 10:41:17 +0200 |
|---|---|---|
| committer | Andreas Schwab <schwab@redhat.com> | 2011-09-07 14:43:35 +0200 |
| commit | 69c1dfc1a7964cc15d429350905aea821803a40c (patch) | |
| tree | 7b9de4af1e620f924362bc1e277a95c7a20cd109 | |
| parent | 184d66c5b942ea9991f7c68cfbeb1e773fd7d092 (diff) | |
Fix setxid race handling exiting threads
(cherry picked from commit 523df511514331a7fa0668b37a917b52f71684af)
| -rw-r--r-- | nptl/ChangeLog | 5 | ||||
| -rw-r--r-- | nptl/allocatestack.c | 11 |
2 files changed, 15 insertions, 1 deletions
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 <schwab@redhat.com> + + * allocatestack.c (setxid_mark_thread): Ensure that the exiting + thread is woken up. + 2011-08-08 Andreas Schwab <schwab@redhat.com> * 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)); |