diff options
| author |   guenther <guenther@openbsd.org> | 2012-03-03 09:36:26 +0000 |
|---|---|---|
| committer | guenther <guenther@openbsd.org> | 2012-03-03 09:36:26 +0000 |
| commit | ce8a7339e20e3e13b3dadd581f8977d7576826a1 (patch) | |
| tree | 9bec128e5ce1e7b7c15a78ce644e1bd697903fac /regress/lib/libpthread | |
| parent | The wlmouse offset should be part of the client, not the server. From (diff) | |
| download | wireguard-openbsd-ce8a7339e20e3e13b3dadd581f8977d7576826a1.tar.xz wireguard-openbsd-ce8a7339e20e3e13b3dadd581f8977d7576826a1.zip | |
While we guarantee that a zero-filled sem_t variable will trigger
EINVAL errors, a sem_t on the stack may have garbage, so make the
tests portable by moving the sem_t from function to file scope.
problem found by miod@
Diffstat (limited to 'regress/lib/libpthread')
3 files changed, 9 insertions, 6 deletions
diff --git a/regress/lib/libpthread/semaphore/sem_getvalue/sem_getvalue.c b/regress/lib/libpthread/semaphore/sem_getvalue/sem_getvalue.c index 354f6ac6c44..19e596f90f7 100644 --- a/regress/lib/libpthread/semaphore/sem_getvalue/sem_getvalue.c +++ b/regress/lib/libpthread/semaphore/sem_getvalue/sem_getvalue.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sem_getvalue.c,v 1.1.1.1 2012/01/04 17:36:40 mpi Exp $ */ +/* $OpenBSD: sem_getvalue.c,v 1.2 2012/03/03 09:36:26 guenther Exp $ */ /* * Martin Pieuchot <mpi@openbsd.org>, 2011. Public Domain. */ @@ -7,10 +7,11 @@ #include <semaphore.h> #include "test.h" +sem_t sem; + int main(int argc, char **argv) { - sem_t sem; int val; CHECKr(sem_init(&sem, 0, 0)); diff --git a/regress/lib/libpthread/semaphore/sem_trywait/sem_trywait.c b/regress/lib/libpthread/semaphore/sem_trywait/sem_trywait.c index 020c19e283c..b4a4fe64ff6 100644 --- a/regress/lib/libpthread/semaphore/sem_trywait/sem_trywait.c +++ b/regress/lib/libpthread/semaphore/sem_trywait/sem_trywait.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sem_trywait.c,v 1.1.1.1 2012/01/04 17:36:40 mpi Exp $ */ +/* $OpenBSD: sem_trywait.c,v 1.2 2012/03/03 09:36:26 guenther Exp $ */ /* * Martin Pieuchot <mpi@openbsd.org>, 2011. Public Domain. */ @@ -8,10 +8,11 @@ #include <semaphore.h> #include "test.h" +sem_t sem; + int main(int argc, char **argv) { - sem_t sem; int val; CHECKn(sem_trywait(&sem)); diff --git a/regress/lib/libpthread/semaphore/sem_wait/sem_wait.c b/regress/lib/libpthread/semaphore/sem_wait/sem_wait.c index b9b3dd5b8e2..efc3e932ffe 100644 --- a/regress/lib/libpthread/semaphore/sem_wait/sem_wait.c +++ b/regress/lib/libpthread/semaphore/sem_wait/sem_wait.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sem_wait.c,v 1.1.1.1 2012/01/04 17:36:40 mpi Exp $ */ +/* $OpenBSD: sem_wait.c,v 1.2 2012/03/03 09:36:26 guenther Exp $ */ /* * Martin Pieuchot <mpi@openbsd.org>, 2011. Public Domain. */ @@ -12,11 +12,12 @@ void *waiter(void *arg); +sem_t sem; + int main(int argc, char **argv) { pthread_t th; - sem_t sem; CHECKn(sem_wait(&sem)); ASSERT(errno == EINVAL); |