diff options
| author | Rich Felker <dalias@aerifal.cx> | 2011-03-10 21:52:18 -0500 |
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2011-03-10 21:52:18 -0500 |
| commit | cfe581b6bc795e3f00ac30791314ec0f9be4b4ad (patch) | |
| tree | a9167b25647df5508a64e3b0b79b7582714c8e2e /src/thread/sem_timedwait.c | |
| parent | 81af503610761a69476a3adbe8341fa8b6d078aa (diff) | |
| download | musl-cfe581b6bc795e3f00ac30791314ec0f9be4b4ad.tar.gz musl-cfe581b6bc795e3f00ac30791314ec0f9be4b4ad.tar.xz musl-cfe581b6bc795e3f00ac30791314ec0f9be4b4ad.zip | |
fix some semaphore wait semantics (race condition deadlock and error checking)
Diffstat (limited to 'src/thread/sem_timedwait.c')
| -rw-r--r-- | src/thread/sem_timedwait.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/thread/sem_timedwait.c b/src/thread/sem_timedwait.c index e6c2495a..ad3bf075 100644 --- a/src/thread/sem_timedwait.c +++ b/src/thread/sem_timedwait.c @@ -8,7 +8,12 @@ int sem_timedwait(sem_t *sem, const struct timespec *at) for (;;) { if (a_fetch_add(sem->__val, -1) > 0) return 0; val = a_fetch_add(sem->__val, 1)+1; + if (val==1) __wake(sem->__val, 1, 0); CANCELPT_BEGIN; + if (at && at->tv_nsec >= 1000000000UL) { + errno = EINVAL; + return -1; + } if (val <= 0 && __timedwait(sem->__val, val, CLOCK_REALTIME, at, 0) == ETIMEDOUT) { errno = ETIMEDOUT; return -1; |