diff options
author | Rich Felker <dalias@aerifal.cx> | 2014-04-15 20:42:39 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2014-04-16 02:46:06 -0400 |
commit | 6ca7c837aa798cc610550b199d4db638605f17f2 (patch) | |
tree | 634a7d8421909b9792a5b718eff89636ba942780 /src/thread/pthread_self.c | |
parent | 52d262fa536768ca97100877f6b023ed3ffe5ff0 (diff) | |
download | musl-6ca7c837aa798cc610550b199d4db638605f17f2.tar.gz musl-6ca7c837aa798cc610550b199d4db638605f17f2.tar.xz musl-6ca7c837aa798cc610550b199d4db638605f17f2.zip |
fix deadlock race in pthread_once
at the end of successful pthread_once, there was a race window during which another thread calling pthread_once would momentarily change the state back from 2 (finished) to 1 (in-progress). in this case, the status was immediately changed back, but with no wake call, meaning that waiters which arrived during this short window could block forever. there are two possible fixes. one would be adding the wake to the code path where it was missing. but it's better just to avoid reverting the status at all, by using compare-and-swap instead of swap. (cherry picked from commit 0d0c2f40344640a2a6942dda156509593f51db5d)
Diffstat (limited to 'src/thread/pthread_self.c')
0 files changed, 0 insertions, 0 deletions