]> git.cameronkatri.com Git - bsdgames-darwin.git/blobdiff - sail/sync.c
Don't cast the return value of calloc().
[bsdgames-darwin.git] / sail / sync.c
index b6e93e1df2bd88815d5b06ec0cc7f7ad1e02e89e..e5277ca6b6db2b4e5b895431b5edd2b0458b6e7a 100644 (file)
@@ -1,4 +1,4 @@
-/*     $NetBSD: sync.c,v 1.14 2000/02/09 22:27:56 jsm Exp $    */
+/*     $NetBSD: sync.c,v 1.28 2009/03/14 19:35:13 dholland Exp $       */
 
 /*
  * Copyright (c) 1983, 1993
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
- * 3. All advertising materials mentioning features or use of this software
- *    must display the following acknowledgement:
- *     This product includes software developed by the University of
- *     California, Berkeley and its contributors.
- * 4. Neither the name of the University nor the names of its contributors
+ * 3. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
  *
 #if 0
 static char sccsid[] = "@(#)sync.c     8.2 (Berkeley) 4/28/95";
 #else
-__RCSID("$NetBSD: sync.c,v 1.14 2000/02/09 22:27:56 jsm Exp $");
+__RCSID("$NetBSD: sync.c,v 1.28 2009/03/14 19:35:13 dholland Exp $");
 #endif
 #endif /* not lint */
 
+#include <sys/stat.h>
+
 #include <fcntl.h>
 #include <errno.h>
-#ifdef __STDC__
+#include <signal.h>
 #include <stdarg.h>
-#else
-#include <varargs.h>
-#endif
+#include <stdio.h>
 #include <stdlib.h>
-#include <unistd.h>
-#include <sys/types.h>
-#include <sys/stat.h>
+#include <string.h>
 #include <time.h>
+#include <unistd.h>
 #include "extern.h"
 #include "pathnames.h"
 
 #define BUFSIZE 4096
 
+static int sync_update(int, struct ship *, const char *,
+                      long, long, long, long);
+
 static const char SF[] = _PATH_SYNC;
 static const char LF[] = _PATH_LOCK;
 static char sync_buf[BUFSIZE];
@@ -69,11 +67,7 @@ static long sync_seek;
 static FILE *sync_fp;
 
 void
-fmtship(buf, len, fmt, ship)
-       char *buf;
-       size_t len;
-       const char *fmt;
-       struct ship *ship;
+fmtship(char *buf, size_t len, const char *fmt, struct ship *ship)
 {
        while (*fmt) {
                if (len-- == 0) {
@@ -98,78 +92,50 @@ fmtship(buf, len, fmt, ship)
 
 /*VARARGS3*/
 void
-#ifdef __STDC__
 makesignal(struct ship *from, const char *fmt, struct ship *ship, ...)
-#else
-makesignal(va_alias)
-       va_dcl
-#endif
 {
        char message[BUFSIZ];
        char format[BUFSIZ];
        va_list ap;
-#ifndef __STDC__
-       struct ship *from;
-       const char *fmt;
-       struct ship *ship;
 
-       va_start(ap);
-       from = va_arg(ap, struct ship *);
-       fmt = va_arg(ap, const char *);
-       ship = va_arg(ap, struct ship *);
-#else
        va_start(ap, ship);
-#endif
        fmtship(format, sizeof(format), fmt, ship);
-       (void) vsprintf(message, format, ap);
+       vsnprintf(message, sizeof(message), format, ap);
        va_end(ap);
        Writestr(W_SIGNAL, from, message);
 }
 
+/*VARARGS2*/
 void
-#ifdef __STDC__
 makemsg(struct ship *from, const char *fmt, ...)
-#else
-makemsg(va_alias)
-       va_dcl
-#endif
 {
        char message[BUFSIZ];
        va_list ap;
-#ifndef __STDC__
-       struct ship *from;
-       const char *fmt;
 
-       va_start(ap);
-       from = va_arg(ap, struct ship *);
-       fmt = va_arg(ap, const char *);
-#else
        va_start(ap, fmt);
-#endif
-       (void) vsprintf(message, fmt, ap);
+       vsnprintf(message, sizeof(message), fmt, ap);
        va_end(ap);
        Writestr(W_SIGNAL, from, message);
 }
 
 int
-sync_exists(game)
-       int game;
+sync_exists(int gamenum)
 {
        char buf[sizeof sync_file];
        struct stat s;
        time_t t;
 
-       (void) sprintf(buf, SF, game);
-       (void) time(&t);
+       snprintf(buf, sizeof(buf), SF, gamenum);
+       time(&t);
        setegid(egid);
        if (stat(buf, &s) < 0) {
                setegid(gid);
                return 0;
        }
        if (s.st_mtime < t - 60*60*2) {         /* 2 hours */
-               (void) unlink(buf);
-               (void) sprintf(buf, LF, game);
-               (void) unlink(buf);
+               unlink(buf);
+               snprintf(buf, sizeof(buf), LF, gamenum);
+               unlink(buf);
                setegid(gid);
                return 0;
        } else {
@@ -179,18 +145,18 @@ sync_exists(game)
 }
 
 int
-sync_open()
+sync_open(void)
 {
        struct stat tmp;
        if (sync_fp != NULL)
-               (void) fclose(sync_fp);
-       (void) sprintf(sync_lock, LF, game);
-       (void) sprintf(sync_file, SF, game);
+               fclose(sync_fp);
+       snprintf(sync_lock, sizeof(sync_lock), LF, game);
+       snprintf(sync_file, sizeof(sync_file), SF, game);
        setegid(egid);
        if (stat(sync_file, &tmp) < 0) {
                mode_t omask = umask(002);
                sync_fp = fopen(sync_file, "w+");
-               (void) umask(omask);
+               umask(omask);
        } else
                sync_fp = fopen(sync_file, "r+");
        setegid(gid);
@@ -201,54 +167,48 @@ sync_open()
 }
 
 void
-sync_close(remove)
-       char remove;
+sync_close(int doremove)
 {
        if (sync_fp != 0)
-               (void) fclose(sync_fp);
-       if (remove) {
+               fclose(sync_fp);
+       if (doremove) {
                setegid(egid);
-               (void) unlink(sync_file);
+               unlink(sync_file);
                setegid(gid);
        }
 }
 
 void
-Write(type, ship, a, b, c, d)
-       int type;
-       struct ship *ship;
-       long a, b, c, d;
+Write(int type, struct ship *ship, long a, long b, long c, long d)
 {
+       size_t max = sizeof(sync_buf) - (sync_bp - sync_buf);
 
-       (void) sprintf(sync_bp, "%d %d 0 %ld %ld %ld %ld\n",
+       snprintf(sync_bp, max, "%d %d 0 %ld %ld %ld %ld\n",
                       type, ship->file->index, a, b, c, d);
        while (*sync_bp++)
                ;
        sync_bp--;
        if (sync_bp >= &sync_buf[sizeof sync_buf])
                abort();
-       (void) sync_update(type, ship, NULL, a, b, c, d);
+       sync_update(type, ship, NULL, a, b, c, d);
 }
 
 void
-Writestr(type, ship, a)
-       int type;
-       struct ship *ship;
-       const char *a;
+Writestr(int type, struct ship *ship, const char *a)
 {
+       size_t max = sizeof(sync_buf) - (sync_bp - sync_buf);
 
-       (void) sprintf(sync_bp, "%d %d 1 %s\n",
-                      type, ship->file->index, a);
+       snprintf(sync_bp, max, "%d %d 1 %s\n", type, ship->file->index, a);
        while (*sync_bp++)
                ;
        sync_bp--;
        if (sync_bp >= &sync_buf[sizeof sync_buf])
                abort();
-       (void) sync_update(type, ship, a, 0, 0, 0, 0);
+       sync_update(type, ship, a, 0, 0, 0, 0);
 }
 
 int
-Sync()
+Sync(void)
 {
        sig_t sighup, sigint;
        int n;
@@ -280,7 +240,7 @@ Sync()
        }
        if (n <= 0)
                return -1;
-       (void) fseek(sync_fp, sync_seek, SEEK_SET);
+       fseek(sync_fp, sync_seek, SEEK_SET);
        for (;;) {
                switch (fscanf(sync_fp, "%d%d%d", &type, &shipnum, &isstr)) {
                case 3:
@@ -295,9 +255,13 @@ Sync()
                if (isstr != 0 && isstr != 1)
                        goto bad;
                if (isstr) {
+                       int ch;
                        char *p;
+
                        for (p = buf;;) {
-                               switch (*p++ = getc(sync_fp)) {
+                               ch = getc(sync_fp);
+                               *p++ = ch;
+                               switch (ch) {
                                case '\n':
                                        p--;
                                case EOF:
@@ -315,7 +279,8 @@ Sync()
                        astr = p;
                        a = b = c = d = 0;
                } else {
-                       if (fscanf(sync_fp, "%ld%ld%ld%ld", &a, &b, &c, &d) != 4)
+                       if (fscanf(sync_fp, "%ld%ld%ld%ld", &a, &b, &c, &d)
+                           != 4)
                                goto bad;
                        astr = NULL;
                }
@@ -326,31 +291,28 @@ bad:
        erred++;
 out:
        if (!erred && sync_bp != sync_buf) {
-               (void) fseek(sync_fp, 0L, SEEK_END);
-               (void) fwrite(sync_buf, sizeof *sync_buf, sync_bp - sync_buf,
+               fseek(sync_fp, 0L, SEEK_END);
+               fwrite(sync_buf, sizeof *sync_buf, sync_bp - sync_buf,
                        sync_fp);
-               (void) fflush(sync_fp);
+               fflush(sync_fp);
                sync_bp = sync_buf;
        }
        sync_seek = ftell(sync_fp);
 #ifdef LOCK_EX
-       (void) flock(fileno(sync_fp), LOCK_UN);
+       flock(fileno(sync_fp), LOCK_UN);
 #else
        setegid(egid);
-       (void) unlink(sync_lock);
+       unlink(sync_lock);
        setegid(gid);
 #endif
-       (void) signal(SIGHUP, sighup);
-       (void) signal(SIGINT, sigint);
+       signal(SIGHUP, sighup);
+       signal(SIGINT, sigint);
        return erred ? -1 : 0;
 }
 
-int
-sync_update(type, ship, astr, a, b, c, d)
-       int type;
-       struct ship *ship;
-       const char *astr;
-       long a, b, c, d;
+static int
+sync_update(int type, struct ship *ship, const char *astr,
+           long a, long b, long c, long d)
 {
        switch (type) {
        case W_DBP: {
@@ -427,9 +389,8 @@ sync_update(type, ship, astr, a, b, c, d)
                break;
                }
        case W_CAPTAIN:
-               (void) strncpy(ship->file->captain, astr,
-                       sizeof ship->file->captain - 1);
-               ship->file->captain[sizeof ship->file->captain - 1] = 0;
+               strlcpy(ship->file->captain, astr,
+                       sizeof ship->file->captain);
                break;
        case W_CAPTURED:
                if (a < 0)
@@ -466,9 +427,8 @@ sync_update(type, ship, astr, a, b, c, d)
                ship->specs->hull = a;
                break;
        case W_MOVE:
-               (void) strncpy(ship->file->movebuf, astr,
-                       sizeof ship->file->movebuf - 1);
-               ship->file->movebuf[sizeof ship->file->movebuf - 1] = 0;
+               strlcpy(ship->file->movebuf, astr,
+                       sizeof ship->file->movebuf);
                break;
        case W_PCREW:
                ship->file->pcrew = a;
@@ -529,7 +489,7 @@ sync_update(type, ship, astr, a, b, c, d)
                windspeed = b;
                break;
        case W_BEGIN:
-               (void) strcpy(ship->file->captain, "begin");
+               strcpy(ship->file->captain, "begin");
                people++;
                break;
        case W_END: