summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFederico G. Schwindt <fgsch@cvs.openbsd.org>2005-10-30 23:59:44 +0000
committerFederico G. Schwindt <fgsch@cvs.openbsd.org>2005-10-30 23:59:44 +0000
commitcb67519b77a103b61c3bbb7ce5c75ed47f806594 (patch)
tree63f0965918ab8c1a74127050d38ff6a9cf0389d3
parent3db6420a0dd659f095653164cbe7548e15cba677 (diff)
-Wall cleanup.
-rw-r--r--regress/lib/libpthread/group/group.c14
-rw-r--r--regress/lib/libpthread/pthread_mutex/pthread_mutex.c5
-rw-r--r--regress/lib/libpthread/socket/2/socket2.c4
-rw-r--r--regress/lib/libpthread/socket/3/socket3.c6
4 files changed, 19 insertions, 10 deletions
diff --git a/regress/lib/libpthread/group/group.c b/regress/lib/libpthread/group/group.c
index 5f6a8793f64..6fade5de422 100644
--- a/regress/lib/libpthread/group/group.c
+++ b/regress/lib/libpthread/group/group.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: group.c,v 1.5 2003/07/31 21:48:04 deraadt Exp $ */
+/* $OpenBSD: group.c,v 1.6 2005/10/30 23:59:43 fgsch Exp $ */
/* David Leonard <d@openbsd.org>, 2001. Public Domain. */
@@ -40,6 +40,7 @@ test(void *arg)
char *s;
char *oname;
char *opasswd;
+ size_t len;
/* Acquire lock for running first part. */
CHECKr(pthread_mutex_lock(&display));
@@ -58,13 +59,16 @@ test(void *arg)
ASSERT(grp->gr_gid == gid);
s = buf; /* Keep our private buffer on the stack */
+ len = sizeof(buf);
/* copy gr_name */
- strcpy(oname = s, grp->gr_name);
+ strlcpy(oname = s, grp->gr_name, len);
+ len -= 1 + strlen(s);
s += 1 + strlen(s);
/* copy gr_passwd */
- strcpy(opasswd = s, grp->gr_passwd);
+ strlcpy(opasswd = s, grp->gr_passwd, len);
+ len -= 1 + strlen(s);
s += 1 + strlen(s);
/* copy gr_gid */
@@ -72,7 +76,9 @@ test(void *arg)
/* copy gr_mem */
for (i = 0, p = grp->gr_mem; *p; p++) {
- strcpy(cpy[i] = s, *p); i++;
+ strlcpy(cpy[i] = s, *p, len);
+ i++;
+ len -= 1 + strlen(s);
s += 1 + strlen(s);
}
cpy[i] = NULL;
diff --git a/regress/lib/libpthread/pthread_mutex/pthread_mutex.c b/regress/lib/libpthread/pthread_mutex/pthread_mutex.c
index c534479b7fc..98121236acc 100644
--- a/regress/lib/libpthread/pthread_mutex/pthread_mutex.c
+++ b/regress/lib/libpthread/pthread_mutex/pthread_mutex.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: pthread_mutex.c,v 1.4 2003/07/31 21:48:05 deraadt Exp $ */
+/* $OpenBSD: pthread_mutex.c,v 1.5 2005/10/30 23:59:43 fgsch Exp $ */
/*
* Copyright (c) 1993, 1994, 1995, 1996 by Chris Provenzano and contributors,
* proven@mit.edu All rights reserved.
@@ -122,6 +122,7 @@ test_mutex_static(void)
printf("test_mutex_static()\n");
test_nocontention_lock(&mutex_static);
+ test_nocontention_trylock(&mutex_static);
test_contention_lock(&mutex_static);
}
@@ -133,6 +134,7 @@ test_mutex_fast(void)
printf("test_mutex_fast()\n");
CHECKr(pthread_mutex_init(&mutex_fast, NULL));
test_nocontention_lock(&mutex_fast);
+ test_nocontention_trylock(&mutex_fast);
test_contention_lock(&mutex_fast);
CHECKr(pthread_mutex_destroy(&mutex_fast));
}
@@ -149,6 +151,7 @@ test_mutex_debug(void)
PTHREAD_MUTEX_ERRORCHECK));
CHECKr(pthread_mutex_init(&mutex_debug, &mutex_debug_attr));
test_nocontention_lock(&mutex_debug);
+ test_nocontention_trylock(&mutex_debug);
test_contention_lock(&mutex_debug);
test_debug_double_lock(&mutex_debug);
test_debug_double_unlock(&mutex_debug);
diff --git a/regress/lib/libpthread/socket/2/socket2.c b/regress/lib/libpthread/socket/2/socket2.c
index 5b53937662b..78ca83da110 100644
--- a/regress/lib/libpthread/socket/2/socket2.c
+++ b/regress/lib/libpthread/socket/2/socket2.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: socket2.c,v 1.5 2004/02/28 08:06:47 deraadt Exp $ */
+/* $OpenBSD: socket2.c,v 1.6 2005/10/30 23:59:43 fgsch Exp $ */
/*
* Copyright (c) 1993, 1994, 1995, 1996 by Chris Provenzano and contributors,
* proven@mit.edu All rights reserved.
@@ -72,7 +72,7 @@ sock_write(void *arg)
static pthread_mutex_t waiter_mutex = PTHREAD_MUTEX_INITIALIZER;
static void *
-waiter(int sig)
+waiter(void *arg)
{
int status;
pid_t pid;
diff --git a/regress/lib/libpthread/socket/3/socket3.c b/regress/lib/libpthread/socket/3/socket3.c
index 6df2e25f2a6..241489b263f 100644
--- a/regress/lib/libpthread/socket/3/socket3.c
+++ b/regress/lib/libpthread/socket/3/socket3.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: socket3.c,v 1.3 2003/07/31 21:48:06 deraadt Exp $ */
+/* $OpenBSD: socket3.c,v 1.4 2005/10/30 23:59:43 fgsch Exp $ */
/* PUBLIC DOMAIN Oct 2002 <marc@snafu.org> */
/* Test blocking/non-blocking mode inheritance on accept */
@@ -28,7 +28,7 @@ sock_connect(void *arg)
int sock;
SET_NAME("connect");
- port = (int)arg;
+ port = *(int *)arg;
CHECKe(sock = socket(AF_INET, SOCK_STREAM, 0));
sin.sin_family = AF_INET;
sin.sin_port = htons(port);
@@ -80,7 +80,7 @@ sock_accept(void *arg)
/* Create another thread to connect to the listening socket. */
CHECKr(pthread_create(&connect_thread, NULL, sock_connect,
- (void*)port));
+ (void *)&port));
/*
* Use poll to check for a pending connection as the socket