]> git.cameronkatri.com Git - bsdgames-darwin.git/blobdiff - hack/hack.options.c
sprintf -> snprintf, plus some use of strlcpy/strlcat where appropriate
[bsdgames-darwin.git] / hack / hack.options.c
index e9f304739f52834b6d7261cf51e633fa67c5e1dc..e01a0d04451f62150c2e15bd3d110189d7b58306 100644 (file)
@@ -1,20 +1,78 @@
-/*     $NetBSD: hack.options.c,v 1.4 1997/10/19 16:58:42 christos Exp $        */
+/*     $NetBSD: hack.options.c,v 1.9 2009/06/07 20:13:18 dholland Exp $        */
 
 /*
- * Copyright (c) Stichting Mathematisch Centrum, Amsterdam, 1985.
+ * Copyright (c) 1985, Stichting Centrum voor Wiskunde en Informatica,
+ * Amsterdam
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met:
+ *
+ * - Redistributions of source code must retain the above copyright notice,
+ * this list of conditions and the following disclaimer.
+ *
+ * - 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.
+ *
+ * - Neither the name of the Stichting Centrum voor Wiskunde en
+ * Informatica, nor the names of its contributors may be used to endorse or
+ * promote products derived from this software without specific prior
+ * written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS
+ * IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A
+ * PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER
+ * OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+ * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+/*
+ * Copyright (c) 1982 Jay Fenlason <hack@gnu.org>
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 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. The name of the author may not be used to endorse or promote products
+ *    derived from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES,
+ * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY
+ * AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL
+ * THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS;
+ * OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
+ * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR
+ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include <sys/cdefs.h>
 #ifndef lint
-__RCSID("$NetBSD: hack.options.c,v 1.4 1997/10/19 16:58:42 christos Exp $");
+__RCSID("$NetBSD: hack.options.c,v 1.9 2009/06/07 20:13:18 dholland Exp $");
 #endif                         /* not lint */
 
 #include <stdlib.h>
+#include <unistd.h>
 #include "hack.h"
 #include "extern.h"
 
 void
-initoptions()
+initoptions(void)
 {
        char           *opts;
 
@@ -31,9 +89,7 @@ initoptions()
 }
 
 void
-parseoptions(opts, from_env)
-       char           *opts;
-       boolean         from_env;
+parseoptions(char *opts, boolean from_env)
 {
        char           *op, *op2;
        unsigned        num;
@@ -121,7 +177,6 @@ parseoptions(opts, from_env)
                while (*op) {
                        num = 1;
                        if (digit(*op)) {
-/*###124 [cc] warning: implicit declaration of function `atoi'%%%*/
                                num = atoi(op);
                                while (digit(*op))
                                        op++;
@@ -153,7 +208,7 @@ bad:
                if (!strncmp(opts, "help", 4)) {
                        pline("%s%s%s",
                              "To set options use `HACKOPTIONS=\"<options>\"' in your environment, or ",
-                             "give the command 'o' followed by the line `<options>' while playing. ",
+                             "give the command 'O' followed by the line `<options>' while playing. ",
                              "Here <options> is a list of <option>s separated by commas.");
                        pline("%s%s%s",
                              "Simple (boolean) options are rest_on_space, news, time, ",
@@ -168,7 +223,7 @@ bad:
                        return;
                }
                pline("Bad option: %s.", opts);
-               pline("Type `o help<cr>' for help.");
+               pline("Type `O help<cr>' for help.");
                return;
        }
        puts("Bad syntax in HACKOPTIONS.");
@@ -180,9 +235,10 @@ bad:
 }
 
 int
-doset()
+doset(void)
 {
-       char            buf[BUFSZ];
+       char buf[BUFSZ];
+       size_t pos;
 
        pline("What options do you want to set? ");
        getlin(buf);
@@ -190,22 +246,24 @@ doset()
                (void) strcpy(buf, "HACKOPTIONS=");
                (void) strcat(buf, flags.female ? "female," : "male,");
                if (flags.standout)
-                       (void) strcat(buf, "standout,");
+                       (void) strlcat(buf, "standout,", sizeof(buf));
                if (flags.nonull)
-                       (void) strcat(buf, "nonull,");
+                       (void) strlcat(buf, "nonull,", sizeof(buf));
                if (flags.nonews)
-                       (void) strcat(buf, "nonews,");
+                       (void) strlcat(buf, "nonews,", sizeof(buf));
                if (flags.time)
-                       (void) strcat(buf, "time,");
+                       (void) strlcat(buf, "time,", sizeof(buf));
                if (flags.notombstone)
-                       (void) strcat(buf, "notombstone,");
+                       (void) strlcat(buf, "notombstone,", sizeof(buf));
                if (flags.no_rest_on_space)
-                       (void) strcat(buf, "!rest_on_space,");
+                       (void) strlcat(buf, "!rest_on_space,", sizeof(buf));
                if (flags.end_top != 5 || flags.end_around != 4 || flags.end_own) {
-                       (void) sprintf(eos(buf), "endgame: %u topscores/%u around me",
+                       pos = strlen(buf);
+                       (void) snprintf(buf+pos, sizeof(buf)-pos,
+                                      "endgame: %u topscores/%u around me",
                                       flags.end_top, flags.end_around);
                        if (flags.end_own)
-                               (void) strcat(buf, "/own scores");
+                               (void) strlcat(buf, "/own scores", sizeof(buf));
                } else {
                        char           *eop = eos(buf);
                        if (*--eop == ',')