Module Name:    src
Committed By:   jruoho
Date:           Tue Apr 17 09:23:21 UTC 2012

Modified Files:
        src/tests/lib/librumphijack: h_client.c h_cwd.c h_netget.c

Log Message:
Return from main() and use EXIT_FAILURE/EXIT_SUCCESS from stdlib(3).


To generate a diff of this commit:
cvs rdiff -u -r1.6 -r1.7 src/tests/lib/librumphijack/h_client.c
cvs rdiff -u -r1.2 -r1.3 src/tests/lib/librumphijack/h_cwd.c
cvs rdiff -u -r1.1 -r1.2 src/tests/lib/librumphijack/h_netget.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/tests/lib/librumphijack/h_client.c
diff -u src/tests/lib/librumphijack/h_client.c:1.6 src/tests/lib/librumphijack/h_client.c:1.7
--- src/tests/lib/librumphijack/h_client.c:1.6	Mon Mar 14 15:56:40 2011
+++ src/tests/lib/librumphijack/h_client.c	Tue Apr 17 09:23:21 2012
@@ -1,4 +1,4 @@
-/*	$NetBSD: h_client.c,v 1.6 2011/03/14 15:56:40 pooka Exp $	*/
+/*	$NetBSD: h_client.c,v 1.7 2012/04/17 09:23:21 jruoho Exp $	*/
 
 /*
  * Copyright (c) 2011 The NetBSD Foundation, Inc.
@@ -57,19 +57,19 @@ main(int argc, char *argv[])
 		tv.tv_usec = 1;
 
 		if (pipe(pipefd) == -1)
-			err(1, "pipe");
+			err(EXIT_FAILURE, "pipe");
 		FD_ZERO(&rfds);
 		FD_SET(pipefd[0], &rfds);
 
 		rv = select(pipefd[0]+1, &rfds, NULL, NULL, &tv);
 		if (rv == -1)
-			err(1, "select");
+			err(EXIT_FAILURE, "select");
 		if (rv != 0)
-			errx(1, "select succesful");
+			errx(EXIT_FAILURE, "select succesful");
 
 		if (FD_ISSET(pipefd[0], &rfds))
-			errx(1, "stdin fileno is still set");
-		exit(0);
+			errx(EXIT_FAILURE, "stdin fileno is still set");
+		return EXIT_SUCCESS;
 	} else if (strcmp(argv[1], "select_allunset") == 0) {
 		fd_set fds;
 		struct timeval tv;
@@ -82,24 +82,24 @@ main(int argc, char *argv[])
 
 		rv = select(100, &fds, &fds, &fds, &tv);
 		if (rv == -1)
-			err(1, "select");
+			err(EXIT_FAILURE, "select");
 		if (rv != 0)
-			errx(1, "select succesful");
+			errx(EXIT_FAILURE, "select succesful");
 
 		rv = select(0, NULL, NULL, NULL, &tv);
 		if (rv == -1)
-			err(1, "select2");
+			err(EXIT_FAILURE, "select2");
 		if (rv != 0)
-			errx(1, "select2 succesful");
+			errx(EXIT_FAILURE, "select2 succesful");
 
-		exit(0);
+		return EXIT_SUCCESS;
 	} else if (strcmp(argv[1], "invafd") == 0) {
 		struct pollfd pfd[2];
 		int fd, rv;
 
 		fd = open("/rump/dev/null", O_RDWR);
 		if (fd == -1)
-			err(1, "open");
+			err(EXIT_FAILURE, "open");
 		close(fd);
 
 		pfd[0].fd = STDIN_FILENO;
@@ -108,26 +108,27 @@ main(int argc, char *argv[])
 		pfd[1].events = POLLIN;
 
 		if ((rv = poll(pfd, 2, INFTIM)) != 1)
-			errx(1, "poll unexpected rv %d (%d)", rv, errno);
+			errx(EXIT_FAILURE, "poll unexpected rv %d (%d)",
+			    rv, errno);
 		if (pfd[1].revents != POLLNVAL || pfd[0].revents != 0)
-			errx(1, "poll unexpected revents");
+			errx(EXIT_FAILURE, "poll unexpected revents");
 
-		exit(0);
+		return EXIT_SUCCESS;
 	} else if (strcmp(argv[1], "fdoff8") == 0) {
 		int fd;
 
 		do
 			if ((fd = open("/dev/null", O_RDWR)) == -1)
-				err(1, "open1");
+				err(EXIT_FAILURE, "open1");
 		while (fd < 7);
 		fd = open("/dev/null", O_RDWR);
 		if (fd != -1 || errno != ENFILE)
-			errx(1, "unexpected fd8 %d %d", fd, errno);
+			errx(EXIT_FAILURE, "unexpected fd8 %d %d", fd, errno);
 		if (fcntl(0, F_MAXFD) != 7)
-			errx(1, "fd leak?");
+			errx(EXIT_FAILURE, "fd leak?");
 		if ((fd = open("/rump/dev/null", O_RDWR)) != 8)
-			errx(1, "rump open %d %d", fd, errno);
-		exit(0);
+			errx(EXIT_FAILURE, "rump open %d %d", fd, errno);
+		return EXIT_SUCCESS;
 	} else {
 		return ENOTSUP;
 	}

Index: src/tests/lib/librumphijack/h_cwd.c
diff -u src/tests/lib/librumphijack/h_cwd.c:1.2 src/tests/lib/librumphijack/h_cwd.c:1.3
--- src/tests/lib/librumphijack/h_cwd.c:1.2	Sat Feb 19 19:57:28 2011
+++ src/tests/lib/librumphijack/h_cwd.c	Tue Apr 17 09:23:21 2012
@@ -1,4 +1,4 @@
-/*      $NetBSD: h_cwd.c,v 1.2 2011/02/19 19:57:28 pooka Exp $	*/
+/*      $NetBSD: h_cwd.c,v 1.3 2012/04/17 09:23:21 jruoho Exp $	*/
 
 /*-
  * Copyright (c) 2011 The NetBSD Foundation, Inc.
@@ -33,6 +33,7 @@
 #include <err.h>
 #include <errno.h>
 #include <fcntl.h>
+#include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 
@@ -58,7 +59,7 @@ dochdir(const char *path, const char *er
 {
 
 	if (chdir(path) == -1)
-		err(1, "%s", errmsg);
+		err(EXIT_FAILURE, "%s", errmsg);
 }
 
 static void
@@ -68,9 +69,9 @@ dofchdir(const char *path, const char *e
 
 	fd = open(path, O_RDONLY);
 	if (fd == -1)
-		err(1, "open %s", errmsg);
+		err(EXIT_FAILURE, "open %s", errmsg);
 	if (fchdir(fd) == -1)
-		err(1, "fchdir %s", errmsg);
+		err(EXIT_FAILURE, "fchdir %s", errmsg);
 	close(fd);
 }
 static void (*thechdir)(const char *, const char *);
@@ -81,44 +82,44 @@ simple(void)
 
 	thechdir(prefix, "chdir1");
 	if (getcwd(pwd, sizeof(pwd)) == NULL)
-		err(1, "getcwd1");
+		err(EXIT_FAILURE, "getcwd1");
 	if (strcmp(pwd, prefix) != 0)
-		errx(1, "strcmp1");
+		errx(EXIT_FAILURE, "strcmp1");
 
 	if (mkdir("dir", 0777) == -1)
-		err(1, "mkdir2");
+		err(EXIT_FAILURE, "mkdir2");
 	thechdir("dir", "chdir2");
 	if (getcwd(pwd, sizeof(pwd)) == NULL)
-		err(1, "getcwd2");
+		err(EXIT_FAILURE, "getcwd2");
 	if (strcmp(pwd, makepath("dir")) != 0)
-		errx(1, "strcmp2");
+		errx(EXIT_FAILURE, "strcmp2");
 
 	if (mkdir("dir", 0777) == -1)
-		err(1, "mkdir3");
+		err(EXIT_FAILURE, "mkdir3");
 	thechdir("dir", "chdir3");
 	if (getcwd(pwd, sizeof(pwd)) == NULL)
-		err(1, "getcwd3");
+		err(EXIT_FAILURE, "getcwd3");
 	if (strcmp(pwd, makepath("dir/dir")) != 0)
-		errx(1, "strcmp3");
+		errx(EXIT_FAILURE, "strcmp3");
 
 	thechdir("..", "chdir4");
 	if (getcwd(pwd, sizeof(pwd)) == NULL)
-		err(1, "getcwd4");
+		err(EXIT_FAILURE, "getcwd4");
 	if (strcmp(pwd, makepath("dir")) != 0)
-		errx(1, "strcmp4");
+		errx(EXIT_FAILURE, "strcmp4");
 
 
 	thechdir("../../../../../../..", "chdir5");
 	if (getcwd(pwd, sizeof(pwd)) == NULL)
-		err(1, "getcwd5");
+		err(EXIT_FAILURE, "getcwd5");
 	if (strcmp(pwd, prefix) != 0)
-		errx(1, "strcmp5");
+		errx(EXIT_FAILURE, "strcmp5");
 
 	thechdir("/", "chdir6");
 	if (getcwd(pwd, sizeof(pwd)) == NULL)
-		err(1, "getcwd6");
+		err(EXIT_FAILURE, "getcwd6");
 	if (strcmp(pwd, "/") != 0)
-		errx(1, "strcmp6");
+		errx(EXIT_FAILURE, "strcmp6");
 }
 
 static void
@@ -127,18 +128,18 @@ symlinktest(void)
 
 	thechdir(prefix, "chdir1");
 	if (mkdir("adir", 0777) == -1)
-		err(1, "mkdir1");
+		err(EXIT_FAILURE, "mkdir1");
 	if (mkdir("anotherdir", 0777) == -1)
-		err(1, "mkdir2");
+		err(EXIT_FAILURE, "mkdir2");
 
 	if (symlink("/adir", "anotherdir/lincthesink") == -1)
-		err(1, "symlink");
+		err(EXIT_FAILURE, "symlink");
 
 	thechdir("anotherdir/lincthesink", "chdir2");
 	if (getcwd(pwd, sizeof(pwd)) == NULL)
-		err(1, "getcwd");
+		err(EXIT_FAILURE, "getcwd");
 	if (strcmp(pwd, makepath("adir")) != 0)
-		errx(1, "strcmp");
+		errx(EXIT_FAILURE, "strcmp");
 }
 
 int
@@ -156,12 +157,12 @@ main(int argc, char *argv[])
 	else if (strcmp(argv[3], "fchdir") == 0)
 		thechdir = dofchdir;
 	else
-		errx(1, "invalid chdir type");
+		errx(EXIT_FAILURE, "invalid chdir type");
 
 	if (strcmp(argv[2], "simple") == 0)
 		simple();
 	if (strcmp(argv[2], "symlink") == 0)
 		symlinktest();
 
-	return 0;
+	return EXIT_SUCCESS;
 }

Index: src/tests/lib/librumphijack/h_netget.c
diff -u src/tests/lib/librumphijack/h_netget.c:1.1 src/tests/lib/librumphijack/h_netget.c:1.2
--- src/tests/lib/librumphijack/h_netget.c:1.1	Sun Feb  6 18:44:29 2011
+++ src/tests/lib/librumphijack/h_netget.c	Tue Apr 17 09:23:21 2012
@@ -1,4 +1,4 @@
-/*      $NetBSD: h_netget.c,v 1.1 2011/02/06 18:44:29 pooka Exp $	*/
+/*      $NetBSD: h_netget.c,v 1.2 2012/04/17 09:23:21 jruoho Exp $	*/
 
 /*-
  * Copyright (c) 2011 Antti Kantee.  All Rights Reserved.
@@ -31,7 +31,7 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: h_netget.c,v 1.1 2011/02/06 18:44:29 pooka Exp $");
+__RCSID("$NetBSD: h_netget.c,v 1.2 2012/04/17 09:23:21 jruoho Exp $");
 
 #include <sys/types.h>
 #include <sys/socket.h>
@@ -61,12 +61,12 @@ main(int argc, char *argv[])
 	if (argc != 4) {
 		fprintf(stderr, "usage: %s address port savefile\n",
 		    getprogname());
-		exit(1);
+		return EXIT_FAILURE;
 	}
 
 	s = socket(PF_INET, SOCK_STREAM, 0);
 	if (s == -1)
-		err(1, "socket");
+		err(EXIT_FAILURE, "socket");
 
 	memset(&sin, 0, sizeof(sin));
 	sin.sin_len = sizeof(sin);
@@ -76,26 +76,26 @@ main(int argc, char *argv[])
 
 	fd = open(argv[3], O_CREAT | O_RDWR, 0644);
 	if (fd == -1)
-		err(1, "open");
+		err(EXIT_FAILURE, "open");
 	if (ftruncate(fd, 0) == -1)
-		err(1, "ftruncate savefile");
+		err(EXIT_FAILURE, "ftruncate savefile");
 
 	if (connect(s, (struct sockaddr *)&sin, sizeof(sin)) == -1)
-		err(1, "connect");
+		err(EXIT_FAILURE, "connect");
 
 	if (write(s, GETSTR, strlen(GETSTR)) != strlen(GETSTR))
-		err(1, "socket write");
+		err(EXIT_FAILURE, "socket write");
 
 	for (;;) {
 		n = read(s, buf, sizeof(buf));
 		if (n == 0)
 			break;
 		if (n == -1)
-			err(1, "socket read");
+			err(EXIT_FAILURE, "socket read");
 
 		if (write(fd, buf, n) != n)
-			err(1, "write file");
+			err(EXIT_FAILURE, "write file");
 	}
 
-	exit(0);
+	return EXIT_SUCCESS;
 }

Reply via email to