Fix warnings and style(9) issues.

Set WARNS to 6.
This commit is contained in:
Rebecca Cran 2011-03-12 14:47:54 +00:00
parent 273fb3dc7c
commit d7d3cbdadf
2 changed files with 33 additions and 27 deletions

View File

@ -3,5 +3,6 @@
PROG= doat PROG= doat
NO_MAN= NO_MAN=
WARNS?=6
.include <bsd.prog.mk> .include <bsd.prog.mk>

View File

@ -26,25 +26,31 @@
* $FreeBSD$ * $FreeBSD$
*/ */
#include <sys/types.h>
#include <sys/syscall.h>
#include <sys/stat.h>
#include <errno.h> #include <errno.h>
#include <fcntl.h> #include <fcntl.h>
#include <stdbool.h> #include <stdbool.h>
#include <stdio.h> #include <stdio.h>
#include <stdlib.h> #include <stdlib.h>
#include <string.h> #include <string.h>
#include <sys/types.h> #include <unistd.h>
#include <sys/syscall.h>
#include <sys/stat.h> void cleanup(void);
void setup(void);
void setup_once(void);
union param { union param {
int i; int i;
char *cp; const char *cp;
mode_t m; mode_t m;
dev_t d; dev_t d;
void *vp; void *vp;
uid_t u; uid_t u;
gid_t g; gid_t g;
char **cpp; const char **cpp;
}; };
struct testcase { struct testcase {
@ -55,7 +61,7 @@ struct testcase {
struct test { struct test {
int syscall; int syscall;
int num_of_cases; int num_of_cases;
char *name; const char *name;
struct testcase tests[10]; /* no more than 10 tests */ struct testcase tests[10]; /* no more than 10 tests */
}; };
@ -64,21 +70,21 @@ struct test *tests;
#define NUM_OF_TESTS 14 /* we dont want the fexecve test to run */ #define NUM_OF_TESTS 14 /* we dont want the fexecve test to run */
char *absolute_path = NULL; char *absolute_path = NULL;
char *relative_path = "tmp/"; const char *relative_path = "tmp/";
char *not_dir_path = "/bin/date"; const char *not_dir_path = "/bin/date";
char *file = "foo"; const char *file = "foo";
char *absolute_file = NULL; char *absolute_file = NULL;
char *relative_file = NULL; char *relative_file = NULL;
char *symlinkf = "link"; const char *symlinkf = "link";
char *newlink = "nlink1"; const char *newlink = "nlink1";
char *newlink2 = "nlink2"; const char *newlink2 = "nlink2";
char *newlink3 = "nlink3"; const char *newlink3 = "nlink3";
char *newdir = "newdir"; const char *newdir = "newdir";
char *fifo = "fifo"; const char *fifo = "fifo";
char *nod = "nod"; const char *nod = "nod";
char *newfile = "newfile"; const char *newfile = "newfile";
char *newslink = "nslink1"; const char *newslink = "nslink1";
bool dir_exist = false; bool dir_exist = false;
bool file_exist = false; bool file_exist = false;
@ -88,15 +94,14 @@ int rel_fd, abs_fd, notd_fd, exec_fd;
struct timeval times[2]; struct timeval times[2];
struct stat buf; struct stat buf;
char *pargv[2] = { "/bin/date", NULL }; const char *pargv[2] = { "/bin/date", NULL };
#define PATH_MAX 1024 #define PATH_MAX 1024
char cbuf[PATH_MAX]; char cbuf[PATH_MAX];
void void
setup() setup(void)
{ {
int i, error; int i, error;
size_t siz;
struct stat sb; struct stat sb;
tests = calloc(NUM_OF_TESTS, sizeof(struct test)); tests = calloc(NUM_OF_TESTS, sizeof(struct test));
@ -607,13 +612,13 @@ setup()
} }
void void
cleanup() cleanup(void)
{ {
system("/bin/sh -c 'rm -rf tmp'"); system("/bin/sh -c 'rm -rf tmp'");
} }
void void
setup_once() setup_once(void)
{ {
} }
@ -623,10 +628,13 @@ main(int argc, char *argv[])
int i,j; int i,j;
int error; int error;
(void)argc;
(void)argv;
setup(); setup();
for (i = 0; i < NUM_OF_TESTS; i++) { for (i = 0; i < NUM_OF_TESTS; i++) {
printf("\nTest: %s\n", tests[i].name); printf("\nTest: %s\n", tests[i].name);
for (j = 0; j < tests[i].num_of_cases; j++) { for (j = 0; j < tests[i].num_of_cases; j++) {
error = syscall(tests[i].syscall, error = syscall(tests[i].syscall,
tests[i].tests[j].params[0], tests[i].tests[j].params[0],
@ -652,13 +660,10 @@ main(int argc, char *argv[])
printf("#%i ... OK\n", j); printf("#%i ... OK\n", j);
} }
} }
} }
} }
cleanup(); cleanup();
return (0); return (0);
} }