summaryrefslogtreecommitdiff
path: root/src/thread
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-10-26 00:28:47 -0400
committerRich Felker <dalias@aerifal.cx>2011-10-26 00:28:47 -0400
commit26120950e26478740a29fd0b1fd78a0bd6b880c8 (patch)
tree0dde8c4e2b3fdd7d7b48718deca7d27e7cd6a1ba /src/thread
parent42976cee2d31bbaade6d2169a9f234cd82af0715 (diff)
downloadmusl-26120950e26478740a29fd0b1fd78a0bd6b880c8.tar.gz
musl-26120950e26478740a29fd0b1fd78a0bd6b880c8.tar.bz2
musl-26120950e26478740a29fd0b1fd78a0bd6b880c8.tar.xz
musl-26120950e26478740a29fd0b1fd78a0bd6b880c8.zip
report sem value overflows in sem_post
this is not required by the standard, but it's nicer than corrupting the state and rather inexpensive.
Diffstat (limited to 'src/thread')
-rw-r--r--src/thread/sem_post.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/thread/sem_post.c b/src/thread/sem_post.c
index 148ab780..14a2dfe2 100644
--- a/src/thread/sem_post.c
+++ b/src/thread/sem_post.c
@@ -7,6 +7,10 @@ int sem_post(sem_t *sem)
do {
val = sem->__val[0];
waiters = sem->__val[1];
+ if (val == SEM_VALUE_MAX) {
+ errno = EOVERFLOW;
+ return -1;
+ }
} while (a_cas(sem->__val, val, val+1+(val<0)) != val);
if (val<0 || waiters) __wake(sem->__val, 1, 0);
return 0;