Style(9) cleanup.
authorChris Pressey <cpressey@dragonflybsd.org>
Wed, 24 Mar 2004 18:23:47 +0000 (18:23 +0000)
committerChris Pressey <cpressey@dragonflybsd.org>
Wed, 24 Mar 2004 18:23:47 +0000 (18:23 +0000)
- Convert K&R-style function definitions to ANSI style.
- Remove ``register'' keywords.
- (fdformat only) Remove/add whitespace around operators, add parens
  after sizeof and return, use NULL instead of O for null pointer,
  and remove (void) casts on ignored return values,
- Move local variable out of redundant inner block.
- Remove redundant return from end of void functions.
- *argv[] -> **argv.
- NOT REACHED -> NOTREACHED.
- Correct wording of #endif hint-comment to __DragonFly__.

usr.sbin/fdformat/fdformat.c
usr.sbin/gifconfig/gifconfig.c
usr.sbin/moused/moused.c
usr.sbin/setkey/parse.y
usr.sbin/setkey/setkey.c
usr.sbin/setkey/test-pfkey.c
usr.sbin/setkey/test-policy.c

index 1771038..a1c4550 100644 (file)
@@ -24,7 +24,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.sbin/fdformat/fdformat.c,v 1.11.2.4 2001/07/19 13:20:42 joerg Exp $
- * $DragonFly: src/usr.sbin/fdformat/fdformat.c,v 1.3 2003/11/16 14:10:45 eirikn Exp $
+ * $DragonFly: src/usr.sbin/fdformat/fdformat.c,v 1.4 2004/03/24 18:23:46 cpressey Exp $
  */
 
 /*
@@ -58,14 +58,15 @@ format_track(int fd, int cyl, int secs, int head, int rate,
              int gaplen, int secsize, int fill,int interleave)
 {
        struct fd_formb f;
-       register int i,j;
+       int i, j;
        int il[FD_MAX_NSEC + 1];
 
-       memset(il,0,sizeof il);
-       for(j = 0, i = 1; i <= secs; i++) {
-           while(il[(j%secs)+1]) j++;
-           il[(j%secs)+1] = i;
-           j += interleave;
+       memset(il, 0, sizeof(il));
+       for (j = 0, i = 1; i <= secs; i++) {
+               while (il[(j % secs) + 1])
+                       j++;
+               il[(j % secs) + 1] = i;
+               j += interleave;
        }
 
        f.format_version = FD_FORMAT_VERSION;
@@ -77,20 +78,20 @@ format_track(int fd, int cyl, int secs, int head, int rate,
        f.fd_formb_nsecs = secs;
        f.fd_formb_gaplen = gaplen;
        f.fd_formb_fillbyte = fill;
-       for(i = 0; i < secs; i++) {
+       for (i = 0; i < secs; i++) {
                f.fd_formb_cylno(i) = cyl;
                f.fd_formb_headno(i) = head;
                f.fd_formb_secno(i) = il[i+1];
                f.fd_formb_secsize(i) = secsize;
        }
-       if(ioctl(fd, FD_FORM, (caddr_t)&f) < 0)
+       if (ioctl(fd, FD_FORM, (caddr_t)&f) < 0)
                err(1, "ioctl(FD_FORM)");
 }
 
 static int
 verify_track(int fd, int track, int tracksize)
 {
-       static char *buf = 0;
+       static char *buf = NULL;
        static int bufsz = 0;
        int fdopts = -1, ofdopts, rv = 0;
 
@@ -99,28 +100,28 @@ verify_track(int fd, int track, int tracksize)
        else {
                ofdopts = fdopts;
                fdopts |= FDOPT_NORETRY;
-               (void)ioctl(fd, FD_SOPTS, &fdopts);
+               ioctl(fd, FD_SOPTS, &fdopts);
        }
 
        if (bufsz < tracksize) {
-               if (buf)
-                       free (buf);
+               if (buf != NULL)        /* XXX why would this ever be? */
+                       free(buf);
                bufsz = tracksize;
-               buf = 0;
+               buf = NULL;
        }
-       if (! buf)
-               buf = malloc (bufsz);
-       if (! buf)
+       if (buf == NULL)
+               buf = malloc(bufsz);
+       if (buf == NULL)
                errx(2, "out of memory");
-       if (lseek (fd, (long) track*tracksize, 0) < 0)
+       if (lseek(fd, (long)track * tracksize, 0) < 0)
                rv = -1;
        /* try twice reading it, without using the normal retrier */
-       else if (read (fd, buf, tracksize) != tracksize
-                && read (fd, buf, tracksize) != tracksize)
+       else if (read(fd, buf, tracksize) != tracksize
+                && read(fd, buf, tracksize) != tracksize)
                rv = -1;
        if(fdopts != -1)
-               (void)ioctl(fd, FD_SOPTS, &ofdopts);
-       return (rv);
+               ioctl(fd, FD_SOPTS, &ofdopts);
+       return(rv);
 }
 
 static const char *
@@ -130,17 +131,17 @@ makename(const char *arg, const char *suffix)
 
        memset(namebuff, 0, 20);
        if(*arg == '\0') /* ??? */
-               return arg;
+               return(arg);
        if(*arg == '/')  /* do not convert absolute pathnames */
-               return arg;
+               return(arg);
        strcpy(namebuff, _PATH_DEV);
        strncat(namebuff, arg, 3);
        strcat(namebuff, suffix);
-       return namebuff;
+       return(namebuff);
 }
 
 static void
-usage (void)
+usage(void)
 {
        fprintf(stderr, "%s\n%s\n",
        "usage: fdformat [-y] [-q] [-n | -v] [-f #] [-c #] [-s #] [-h #]",
@@ -149,21 +150,21 @@ usage (void)
 }
 
 static int
-yes (void)
+yes(void)
 {
-       char reply [256], *p;
+       char reply[256], *p;
 
-       reply[sizeof(reply)-1] = 0;
+       reply[sizeof(reply) - 1] = 0;
        for (;;) {
                fflush(stdout);
-               if (! fgets (reply, sizeof(reply)-1, stdin))
-                       return (0);
-               for (p=reply; *p==' ' || *p=='\t'; ++p)
+               if (fgets(reply, sizeof(reply) - 1, stdin) == NULL)
+                       return(0);
+               for (p = reply; *p == ' ' || *p == '\t'; ++p)
                        continue;
-               if (*p=='y' || *p=='Y')
-                       return (1);
-               if (*p=='n' || *p=='N' || *p=='\n' || *p=='\r')
-                       return (0);
+               if (*p == 'y' || *p == 'Y')
+                       return(1);
+               if (*p == 'n' || *p == 'N' || *p == '\n' || *p == '\r')
+                       return(0);
                printf("Answer `yes' or `no': ");
        }
 }
@@ -236,8 +237,8 @@ main(int argc, char **argv)
 #define MAXPRINTERRS 10
        struct fdc_status fdcs[MAXPRINTERRS];
 
-       while((c = getopt(argc, argv, "f:c:s:h:r:g:S:F:t:i:qyvn")) != -1)
-               switch(c) {
+       while ((c = getopt(argc, argv, "f:c:s:h:r:g:S:F:t:i:qyvn")) != -1)
+               switch (c) {
                case 'f':       /* format in kilobytes */
                        format = atoi(optarg);
                        break;
@@ -299,10 +300,10 @@ main(int argc, char **argv)
                        usage();
                }
 
-       if(optind != argc - 1)
+       if (optind != argc - 1)
                usage();
 
-       switch(format) {
+       switch (format) {
        default:
                errx(2, "bad floppy size: %dK", format);
        case -1:   suffix = "";      break;
@@ -320,10 +321,10 @@ main(int argc, char **argv)
 
        devname = makename(argv[optind], suffix);
 
-       if((fd = open(devname, O_RDWR)) < 0)
+       if ((fd = open(devname, O_RDWR)) < 0)
                err(1, "%s", devname);
 
-       if(ioctl(fd, FD_GTYPE, &fdt) < 0)
+       if (ioctl(fd, FD_GTYPE, &fdt) < 0)
                errx(1, "not a floppy disk: %s", devname);
        fdopts = FDOPT_NOERRLOG;
        if (ioctl(fd, FD_SOPTS, &fdopts) == -1)
@@ -348,31 +349,31 @@ main(int argc, char **argv)
        if (steps >= 0)   fdt.steptrac = steps;
        if (intleave >= 0) fdt.f_inter = intleave;
 
-       bytes_per_track = fdt.sectrac * (1<<fdt.secsize) * 128;
+       bytes_per_track = fdt.sectrac * (1 << fdt.secsize) * 128;
 
        /* XXX  20/40 = 0.5 */
        tracks_per_dot = (fdt.tracks * fdt.heads + 20) / 40;
 
        if (verify_only) {
-               if(!quiet)
+               if (!quiet)
                        printf("Verify %dK floppy `%s'.\n",
                                fdt.tracks * fdt.heads * bytes_per_track / 1024,
                                devname);
        }
-       else if(!quiet && !confirm) {
+       else if (!quiet && !confirm) {
                printf("Format %dK floppy `%s'? (y/n): ",
                        fdt.tracks * fdt.heads * bytes_per_track / 1024,
                        devname);
-               if(! yes ()) {
+               if (!yes()) {
                        printf("Not confirmed.\n");
-                       return 3;
+                       return(3);
                }
        }
 
        /*
         * Formatting.
         */
-       if(!quiet) {
+       if (!quiet) {
                int i;
 
                printf("Processing ");
@@ -418,7 +419,7 @@ main(int argc, char **argv)
                        }
                }
        }
-       if(!quiet)
+       if (!quiet)
                printf(" done.\n");
 
        if (!quiet && errs) {
@@ -435,7 +436,7 @@ main(int argc, char **argv)
                        fprintf(stderr, "(Further errors not printed.)\n");
        }
 
-       return errs != 0;
+       return(errs != 0);
 }
 /*
  * Local Variables:
index 41c7d51..1eb2054 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/usr.sbin/gifconfig/gifconfig.c,v 1.2.2.4 2002/08/30 14:23:39 sobomax Exp $        */
-/*     $DragonFly: src/usr.sbin/gifconfig/gifconfig.c,v 1.5 2004/02/04 15:39:54 rob Exp $      */
+/*     $DragonFly: src/usr.sbin/gifconfig/gifconfig.c,v 1.6 2004/03/24 18:23:46 cpressey Exp $ */
 /*     $KAME: gifconfig.c,v 1.14 2001/01/01 04:04:56 jinmei Exp $      */
 
 /*
@@ -61,7 +61,7 @@
 #include <net/if.h>
 #if defined(__DragonFly__)
 #include <net/if_var.h>
-#endif /* __DragonFly >= 3 */
+#endif /* __DragonFly__ */
 #include <net/if_dl.h>
 #include <net/if_types.h>
 #include <net/route.h>
@@ -383,7 +383,7 @@ ifconfig(int argc, char **argv, int af, struct afswtch *rafp)
        }
 
        while (argc > 0) {
-               register struct cmd *p;
+               struct cmd *p;
 
                for (p = cmds; p->c_name; p++)
                        if (strcmp(*argv, p->c_name) == 0)
@@ -783,7 +783,7 @@ SIN(addreq.ifra_addr), SIN(addreq.ifra_dstaddr)};
 void
 in_getaddr(char *s, int which)
 {
-       register struct sockaddr_in *sin = sintab[which];
+       struct sockaddr_in *sin = sintab[which];
        struct hostent *hp;
        struct netent *np;
 
@@ -808,7 +808,7 @@ SIN6(in6_addreq.ifra_addr), SIN6(in6_addreq.ifra_dstaddr)};
 void
 in6_getaddr(char *s, int which)
 {
-       register struct sockaddr_in6 *sin = sin6tab[which];
+       struct sockaddr_in6 *sin = sin6tab[which];
 
        sin->sin6_len = sizeof(*sin);
        sin->sin6_family = AF_INET6;
@@ -820,8 +820,8 @@ in6_getaddr(char *s, int which)
 void
 in6_getprefix(char *plen, int which)
 {
-       register struct sockaddr_in6 *sin = sin6tab[which];
-       register u_char *cp;
+       struct sockaddr_in6 *sin = sin6tab[which];
+       u_char *cp;
        int len = atoi(plen);
 
        if ((len < 0) || (len > 128))
@@ -842,10 +842,10 @@ in6_getprefix(char *plen, int which)
  * Print a value a la the %b format of the kernel's printf
  */
 void
-printb(char *s, register unsigned int v, register char *bits)
+printb(char *s, unsigned int v, char *bits)
 {
-       register int i, any = 0;
-       register char c;
+       int i, any = 0;
+       char c;
 
        if (bits && *bits == 8)
                printf("%s=%o", s, v & 0xffff);
@@ -873,8 +873,8 @@ printb(char *s, register unsigned int v, register char *bits)
 int
 prefix(void *val, int size)
 {
-        register u_char *name = (u_char *)val;
-        register int byte, bit, plen = 0;
+        u_char *name = (u_char *)val;
+        int byte, bit, plen = 0;
 
         for (byte = 0; byte < size; byte++, plen += 8)
                 if (name[byte] != 0xff)
index 58ea2b2..880d9f8 100644 (file)
@@ -46,7 +46,7 @@
 
 /*
  * $FreeBSD: src/usr.sbin/moused/moused.c,v 1.37.2.11 2002/04/15 00:52:08 will Exp $
- * $DragonFly: src/usr.sbin/moused/moused.c,v 1.2 2003/06/17 04:29:57 dillon Exp $
+ * $DragonFly: src/usr.sbin/moused/moused.c,v 1.3 2004/03/24 18:23:46 cpressey Exp $
  */
 #include <ctype.h>
 #include <err.h>
@@ -506,7 +506,7 @@ static void mremote_clientchg(int add);
 static int kidspad(u_char rxc, mousestatus_t *act);
 
 int
-main(int argc, char *argv[])
+main(int argc, char **argv)
 {
     int c;
     int        i;
@@ -985,8 +985,8 @@ moused(void)
            }
        }
     }
-    /* NOT REACHED */
-}              
+    /* NOTREACHED */
+}
 
 static void
 hup(int sig)
@@ -2092,8 +2092,8 @@ r_installmap(char *arg)
 static void
 r_map(mousestatus_t *act1, mousestatus_t *act2)
 {
-    register int pb;
-    register int pbuttons;
+    int pb;
+    int pbuttons;
     int lbuttons;
 
     pbuttons = act1->button;
index 11f7a04..77c5ad6 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/usr.sbin/setkey/parse.y,v 1.1.2.2 2001/07/03 11:02:17 ume Exp $   */
-/*     $DragonFly: src/usr.sbin/setkey/parse.y,v 1.3 2003/11/03 19:31:43 eirikn Exp $  */
+/*     $DragonFly: src/usr.sbin/setkey/parse.y,v 1.4 2004/03/24 18:23:46 cpressey Exp $        */
 /*     $KAME: kame/kame/kame/setkey/parse.y,v 1.36 2001/06/07 15:53:12 sakane Exp $    */
 
 /*
@@ -600,7 +600,7 @@ policy_requests
 %%
 
 int
-setkeymsg()
+setkeymsg(void)
 {
        struct sadb_msg m_msg;
 
@@ -853,10 +853,7 @@ setkeymsg()
 }
 
 static struct addrinfo *
-parse_addr(host, port, flag)
-       char *host;
-       char *port;
-       int flag;
+parse_addr(char *host, char *port, int flag)
 {
        struct addrinfo hints, *res = NULL;
        int error;
@@ -877,10 +874,7 @@ parse_addr(host, port, flag)
 }
 
 static int
-setvarbuf(off, ebuf, elen, vbuf, vlen)
-       caddr_t vbuf;
-       struct sadb_ext *ebuf;
-       int *off, elen, vlen;
+setvarbuf(int *off, struct sadb_ext *ebuf, int elen, caddr_t vbuf, int vlen)
 {
        memset(m_buf + *off, 0, PFKEY_UNUNIT64(ebuf->sadb_ext_len));
        memcpy(m_buf + *off, (caddr_t)ebuf, elen);
@@ -891,7 +885,7 @@ setvarbuf(off, ebuf, elen, vbuf, vlen)
 }
 
 void
-parse_init()
+parse_init(void)
 {
        p_type = 0;
        p_spi = 0;
@@ -922,7 +916,7 @@ parse_init()
 }
 
 void
-free_buffer()
+free_buffer(void)
 {
        if (p_src) free(p_src);
        if (p_dst) free(p_dst);
index fcdccc5..fcae505 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/usr.sbin/setkey/setkey.c,v 1.1.2.3 2003/04/26 23:53:54 sumikawa Exp $     */
-/*     $DragonFly: src/usr.sbin/setkey/setkey.c,v 1.3 2003/11/03 19:31:43 eirikn Exp $ */
+/*     $DragonFly: src/usr.sbin/setkey/setkey.c,v 1.4 2004/03/24 18:23:47 cpressey Exp $       */
 /*     $KAME: setkey.c,v 1.18 2001/05/08 04:36:39 itojun Exp $ */
 
 /*
@@ -107,20 +107,18 @@ Usage()
 }
 
 int
-main(ac, av)
-       int ac;
-       char **av;
+main(int argc, char **argv)
 {
        FILE *fp = stdin;
        int c;
 
-       pname = *av;
+       pname = *argv;
 
-       if (ac == 1) Usage();
+       if (argc == 1) Usage();
 
        thiszone = gmt2local(0);
 
-       while ((c = getopt(ac, av, "acdf:hlvxDFP")) != -1) {
+       while ((c = getopt(argc, argv, "acdf:hlvxDFP")) != -1) {
                switch (c) {
                case 'c':
                        f_mode = MODE_SCRIPT;
@@ -195,7 +193,7 @@ main(ac, av)
 }
 
 int
-get_supported()
+get_supported(void)
 {
        int so;
 
@@ -218,8 +216,7 @@ get_supported()
 }
 
 void
-sendkeyshort(type)
-        u_int type;
+sendkeyshort(u_int type)
 {
        struct sadb_msg *m_msg = (struct sadb_msg *)m_buf;
 
@@ -235,12 +232,10 @@ sendkeyshort(type)
        m_msg->sadb_msg_pid = getpid();
 
        sendkeymsg();
-
-       return;
 }
 
 void
-promisc()
+promisc(void)
 {
        struct sadb_msg *m_msg = (struct sadb_msg *)m_buf;
        u_char rbuf[1024 * 32]; /* XXX: Enough ? Should I do MSG_PEEK ? */
@@ -313,28 +308,25 @@ promisc()
 }
 
 int
-sendkeymsg()
+sendkeymsg(void)
 {
        int so;
-
        u_char rbuf[1024 * 32]; /* XXX: Enough ? Should I do MSG_PEEK ? */
        int len;
        struct sadb_msg *msg;
+       struct timeval tv;
 
        if ((so = pfkey_open()) < 0) {
                perror("pfkey_open");
                return -1;
        }
 
-    {
-       struct timeval tv;
        tv.tv_sec = 1;
        tv.tv_usec = 0;
        if (setsockopt(so, SOL_SOCKET, SO_RCVTIMEO, &tv, sizeof(tv)) < 0) {
                perror("setsockopt");
                goto end;
        }
-    }
 
        if (f_forever)
                shortdump_hdr();
@@ -381,11 +373,8 @@ end:
 }
 
 int
-postproc(msg, len)
-       struct sadb_msg *msg;
-       int len;
+postproc(struct sadb_msg *msg, int len)
 {
-
        if (msg->sadb_msg_errno != 0) {
                char inf[80];
                char *errmsg = NULL;
@@ -489,8 +478,7 @@ static char *ipproto[] = {
        (((x) < sizeof(tab)/sizeof(tab[0]) && tab[(x)]) ? tab[(x)] : numstr(x))
 
 const char *
-numstr(x)
-       int x;
+numstr(int x)
 {
        static char buf[20];
        snprintf(buf, sizeof(buf), "#%d", x);
@@ -505,8 +493,7 @@ shortdump_hdr()
 }
 
 void
-shortdump(msg)
-       struct sadb_msg *msg;
+shortdump(struct sadb_msg *msg)
 {
        caddr_t mhp[SADB_EXT_MAX + 1];
        char buf[NI_MAXHOST], pbuf[NI_MAXSERV];
@@ -515,7 +502,9 @@ shortdump(msg)
        struct sadb_lifetime *lts, *lth, *ltc;
        struct sockaddr *s;
        u_int t;
-       time_t cur = time(0);
+       time_t cur;
+
+       cur = time(0);
 
        pfkey_align(msg, mhp);
        pfkey_check(mhp);
@@ -592,7 +581,7 @@ shortdump(msg)
  * Print the timestamp
  */
 static void
-printdate()
+printdate(void)
 {
        struct timeval tp;
        int s;
@@ -623,8 +612,8 @@ printdate()
 int32_t
 gmt2local(time_t t)
 {
-       register int dt, dir;
-       register struct tm *gmt, *loc;
+       int dt, dir;
+       struct tm *gmt, *loc;
        struct tm sgmt;
 
        if (t == 0)
index 20c3494..f1cb507 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/usr.sbin/setkey/test-pfkey.c,v 1.1.2.1 2000/07/15 07:37:04 kris Exp $     */
-/*     $DragonFly: src/usr.sbin/setkey/test-pfkey.c,v 1.3 2003/11/03 19:31:43 eirikn Exp $     */
+/*     $DragonFly: src/usr.sbin/setkey/test-pfkey.c,v 1.4 2004/03/24 18:23:47 cpressey Exp $   */
 /*     $KAME: test-pfkey.c,v 1.4 2000/06/07 00:29:14 itojun Exp $      */
 
 /*
@@ -68,22 +68,20 @@ void key_setsadbaddr(u_int, u_int, caddr_t);
 void key_setsadbextbuf(caddr_t, int, caddr_t, int, caddr_t, int);
 
 void
-Usage()
+Usage(void)
 {
        printf("Usage:\t%s number\n", pname);
        exit(0);
 }
 
 int
-main(ac, av)
-       int ac;
-       char **av;
+main(int argv, char **argv)
 {
-       pname = *av;
+       pname = *argv;
 
-       if (ac == 1) Usage();
+       if (argc == 1) Usage();
 
-       key_setsadbmsg(atoi(*(av+1)));
+       key_setsadbmsg(atoi(*(argv+1)));
        sendkeymsg();
 
        exit(0);
@@ -91,7 +89,7 @@ main(ac, av)
 
 /* %%% */
 int
-sendkeymsg()
+sendkeymsg(void)
 {
        u_char rbuf[1024 * 32]; /* XXX: Enough ? Should I do MSG_PEEK ? */
        int so, len;
@@ -133,8 +131,7 @@ end:
 }
 
 void
-key_setsadbmsg(type)
-       u_int type;
+key_setsadbmsg(u_int type)
 {
        struct sadb_msg m_msg;
 
@@ -265,12 +262,10 @@ key_setsadbmsg(type)
        }
 
        ((struct sadb_msg *)m_buf)->sadb_msg_len = PFKEY_UNIT64(m_len);
-
-       return;
 }
 
 void
-key_setsadbsens()
+key_setsadbsens(void)
 {
        struct sadb_sens m_sens;
        u_char buf[64];
@@ -298,12 +293,10 @@ key_setsadbsens()
                        (caddr_t)&m_sens, sizeof(struct sadb_sens),
                        buf, slen + ilen);
        m_len += len;
-
-       return;
 }
 
 void
-key_setsadbprop()
+key_setsadbprop(void)
 {
        struct sadb_prop m_prop;
        struct sadb_comb *m_comb;
@@ -360,14 +353,10 @@ key_setsadbprop()
                        (caddr_t)&m_prop, sizeof(struct sadb_prop),
                        buf, sizeof(*m_comb) * 2);
        m_len += len;
-
-       return;
 }
 
 void
-key_setsadbid(ext, str)
-       u_int ext;
-       caddr_t str;
+key_setsadbid(u_int ext, caddr_t str)
 {
        struct sadb_ident m_id;
        u_int idlen = strlen(str), len;
@@ -383,13 +372,10 @@ key_setsadbid(ext, str)
                        (caddr_t)&m_id, sizeof(struct sadb_ident),
                        str, idlen);
        m_len += len;
-
-       return;
 }
 
 void
-key_setsadblft(ext, time)
-       u_int ext, time;
+key_setsadblft(u_int ext, u_int time)
 {
        struct sadb_lifetime m_lft;
 
@@ -402,12 +388,10 @@ key_setsadblft(ext, time)
 
        memcpy(m_buf + m_len, &m_lft, sizeof(struct sadb_lifetime));
        m_len += sizeof(struct sadb_lifetime);
-
-       return;
 }
 
 void
-key_setspirange()
+key_setspirange(void)
 {
        struct sadb_spirange m_spi;
 
@@ -419,14 +403,10 @@ key_setspirange()
 
        memcpy(m_buf + m_len, &m_spi, sizeof(struct sadb_spirange));
        m_len += sizeof(struct sadb_spirange);
-
-       return;
 }
 
 void
-key_setsadbkey(ext, str)
-       u_int ext;
-       caddr_t str;
+key_setsadbkey(u_int ext, caddr_t str)
 {
        struct sadb_key m_key;
        u_int keylen = strlen(str);
@@ -442,12 +422,10 @@ key_setsadbkey(ext, str)
                        (caddr_t)&m_key, sizeof(struct sadb_key),
                        str, keylen);
        m_len += len;
-
-       return;
 }
 
 void
-key_setsadbsa()
+key_setsadbsa(void)
 {
        struct sadb_sa m_sa;
 
@@ -462,14 +440,10 @@ key_setsadbsa()
 
        memcpy(m_buf + m_len, &m_sa, sizeof(struct sadb_sa));
        m_len += sizeof(struct sadb_sa);
-
-       return;
 }
 
 void
-key_setsadbaddr(ext, af, str)
-       u_int ext, af;
-       caddr_t str;
+key_setsadbaddr(u_int ext, u_int af, caddr_t str)
 {
        struct sadb_address m_addr;
        u_int len;
@@ -514,19 +488,14 @@ key_setsadbaddr(ext, af, str)
        m_len += len;
 
        freeaddrinfo(res);
-
-       return;
 }
 
 void
-key_setsadbextbuf(dst, off, ebuf, elen, vbuf, vlen)
-       caddr_t dst, ebuf, vbuf;
-       int off, elen, vlen;
+key_setsadbextbuf(caddr_t dst, int off, caddr_t ebuf, int elen,
+                 caddr_t vbuf, int vlen)
 {
        memset(dst + off, 0, elen + vlen);
        memcpy(dst + off, (caddr_t)ebuf, elen);
        memcpy(dst + off + elen, vbuf, vlen);
-
-       return;
 }
 
index 90a0d3b..5014bd2 100644 (file)
@@ -27,7 +27,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.sbin/setkey/test-policy.c,v 1.1 2000/01/06 12:40:53 shin Exp $
- * $DragonFly: src/usr.sbin/setkey/test-policy.c,v 1.2 2003/06/17 04:30:03 dillon Exp $
+ * $DragonFly: src/usr.sbin/setkey/test-policy.c,v 1.3 2004/03/24 18:23:47 cpressey Exp $
  */
 
 #include <sys/types.h>
@@ -73,7 +73,8 @@ u_char        *p_secpolicy;
 int    test(char *buf, int family);
 char   *setpolicy(char *req);
 
-main()
+int
+main(int argc __unused, char **argv __unused)
 {
        int i;
        char *buf;
@@ -93,11 +94,13 @@ main()
        }
 }
 
-int test(char *policy, int family)
+int
+test(char *policy, int family)
 {
        int so, proto, optname;
        int len;
        char getbuf[1024];
+       char *buf = NULL;
 
        switch (family) {
        case PF_INET:
@@ -121,9 +124,6 @@ int test(char *policy, int family)
        if (getsockopt(so, proto, optname, getbuf, &len) < 0)
                perror("getsockopt");
 
-    {
-       char *buf = NULL;
-
        printf("\tgetlen:%d\n", len);
 
        if ((buf = ipsec_dump_policy(getbuf, NULL)) == NULL)
@@ -132,12 +132,12 @@ int test(char *policy, int family)
                printf("\t[%s]\n", buf);
 
        free(buf);
-    }
 
        close (so);
 }
 
-char *setpolicy(char *req)
+char *
+setpolicy(char *req)
 {
        int len;
        char *buf;