about summary refs log tree commit diff
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.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;