[PATCH] Fix lll_unlock twice in pthread_cond_broadcast
Yang Yingliang
yangyingliang@huawei.com
Wed Apr 30 09:40:00 GMT 2014
lll_unlock() will be called again if it goes to "wake_all"
in pthread_cond_broadcast(). This may make another thread
which is waiting for lock in pthread_cond_timedwait() unlock.
So there are more than one threads get the lock, it will break
the shared data.
It's introduced by commit 8313cb997d2d("FUTEX_*_REQUEUE_PI support for non-x86 code")
---
nptl/pthread_cond_broadcast.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/nptl/pthread_cond_broadcast.c b/nptl/pthread_cond_broadcast.c
index ed30e7c60bc7..7c6c9ea9a2d5 100644
--- a/nptl/pthread_cond_broadcast.c
+++ b/nptl/pthread_cond_broadcast.c
@@ -81,6 +81,7 @@ __pthread_cond_broadcast (cond)
wake_all:
lll_futex_wake (&cond->__data.__futex, INT_MAX, pshared);
+ return 0;
}
/* We are done. */
--
1.8.0
More information about the Libc-alpha
mailing list