]> git.ozlabs.org Git - ppp.git/blobdiff - pppd/options.c
remove a couple of ansi-C-isms.
[ppp.git] / pppd / options.c
index b2c7b58574f0944cb090fd6e0123ee508d6476b0..53456cc8d13ead6754a11e37b39c12724777d465 100644 (file)
  */
 
 #ifndef lint
-static char rcsid[] = "$Id: options.c,v 1.46 1999/02/26 11:03:34 paulus Exp $";
+static char rcsid[] = "$Id: options.c,v 1.53 1999/03/19 04:23:44 paulus Exp $";
 #endif
 
 #include <ctype.h>
 #include <stdio.h>
 #include <errno.h>
 #include <unistd.h>
-#include <limits.h>
 #include <stdlib.h>
 #include <termios.h>
 #include <syslog.h>
@@ -53,17 +52,10 @@ static char rcsid[] = "$Id: options.c,v 1.46 1999/02/26 11:03:34 paulus Exp $";
 
 #include <net/ppp-comp.h>
 
-#define FALSE  0
-#define TRUE   1
-
 #if defined(ultrix) || defined(NeXT)
 char *strdup __P((char *));
 #endif
 
-#ifndef GIDSET_TYPE
-#define GIDSET_TYPE    gid_t
-#endif
-
 /*
  * Option variables and default values.
  */
@@ -89,8 +81,6 @@ char  user[MAXNAMELEN];       /* Username for PAP */
 char   passwd[MAXSECRETLEN];   /* Password for PAP */
 bool   persist = 0;            /* Reopen link after it goes down */
 char   our_name[MAXNAMELEN];   /* Our name for authentication purposes */
-char   remote_name[MAXNAMELEN]; /* Peer's name for authentication */
-int    explicit_remote = 0;    /* User specified explicit remote name */
 bool   demand = 0;             /* do dial-on-demand */
 char   *ipparam = NULL;        /* Extra parameter for ip up/down scripts */
 int    idle_time_limit = 0;    /* Disconnect if idle for this many seconds */
@@ -123,6 +113,7 @@ static int readfile __P((char **));
 static int callfile __P((char **));
 static int showversion __P((char **));
 static int showhelp __P((char **));
+static void usage __P((void));
 
 #ifdef PPP_FILTER
 static int setpassfilter __P((char **));
@@ -133,6 +124,7 @@ static int setactivefilter __P((char **));
 static option_t *find_option __P((char *name));
 static int process_option __P((option_t *, char **));
 static int n_arguments __P((option_t *));
+static int number_option __P((char *, u_int32_t *, int));
 
 /*
  * Valid arguments.
@@ -159,11 +151,14 @@ option_t general_options[] = {
     { "-all", o_special_noarg, noopt,
       "Don't request/allow any LCP or IPCP options (useless)" },
     { "connect", o_string, &connector,
-      "A program to set up a connection", OPT_A2INFO, &connector_info },
+      "A program to set up a connection",
+      OPT_A2INFO | OPT_PRIVFIX, &connector_info },
     { "disconnect", o_string, &disconnector,
-      "Program to disconnect serial device", OPT_A2INFO, &disconnector_info },
+      "Program to disconnect serial device",
+      OPT_A2INFO | OPT_PRIVFIX, &disconnector_info },
     { "welcome", o_string, &welcomer,
-      "Script to welcome client", OPT_A2INFO, &welcomer_info },
+      "Script to welcome client",
+      OPT_A2INFO | OPT_PRIVFIX, &welcomer_info },
     { "maxconnect", o_int, &maxconnect,
       "Set connection time limit", OPT_LLIMIT|OPT_NOINCR|OPT_ZEROINF },
     { "crtscts", o_int, &crtscts,
@@ -222,7 +217,7 @@ option_t general_options[] = {
 #endif
 
 static char *usage_string = "\
-pppd version %s patch level %d%s\n\
+pppd version %s.%d%s\n\
 Usage: %s [ options ], where options are:\n\
        <device>        Communicate over the named device\n\
        <speed>         Set the baud rate to <speed>\n\
@@ -342,17 +337,17 @@ options_from_file(filename, must_exist, check_prot, priv)
     char args[MAXARGS][MAXWORDLEN];
     char cmd[MAXWORDLEN];
 
-    if ((f = fopen(filename, "r")) == NULL) {
+    if (check_prot)
+       seteuid(getuid());
+    f = fopen(filename, "r");
+    if (check_prot)
+       seteuid(0);
+    if (f == NULL) {
        if (!must_exist && errno == ENOENT)
            return 1;
        option_error("Can't open options file %s: %m", filename);
        return 0;
     }
-    if (check_prot && !readable(fileno(f))) {
-       option_error("Can't open options file %s: access denied", filename);
-       fclose(f);
-       return 0;
-    }
 
     oldpriv = privileged_option;
     privileged_option = priv;
@@ -415,17 +410,17 @@ options_from_user()
     char *user, *path, *file;
     int ret;
     struct passwd *pw;
+    size_t pl;
 
     pw = getpwuid(getuid());
     if (pw == NULL || (user = pw->pw_dir) == NULL || user[0] == 0)
        return 1;
     file = _PATH_USEROPT;
-    path = malloc(strlen(user) + strlen(file) + 2);
+    pl = strlen(user) + strlen(file) + 2;
+    path = malloc(pl);
     if (path == NULL)
        novm("init file name");
-    strcpy(path, user);
-    strcat(path, "/");
-    strcat(path, file);
+    slprintf(path, pl, "%s/%s", user, file);
     ret = options_from_file(path, 0, 1, privileged);
     free(path);
     return ret;
@@ -440,20 +435,22 @@ options_for_tty()
 {
     char *dev, *path, *p;
     int ret;
+    size_t pl;
 
     dev = devnam;
     if (strncmp(dev, "/dev/", 5) == 0)
        dev += 5;
     if (strcmp(dev, "tty") == 0)
        return 1;               /* don't look for /etc/ppp/options.tty */
-    path = malloc(strlen(_PATH_TTYOPT) + strlen(dev) + 1);
+    pl = strlen(_PATH_TTYOPT) + strlen(dev) + 1;
+    path = malloc(pl);
     if (path == NULL)
        novm("tty init file name");
-    strcpy(path, _PATH_TTYOPT);
+    slprintf(path, pl, "%s%s", _PATH_TTYOPT, dev);
     /* Turn slashes into dots, for Solaris case (e.g. /dev/term/a) */
-    for (p = path + strlen(path); *dev != 0; ++dev)
-       *p++ = (*dev == '/'? '.': *dev);
-    *p = 0;
+    for (p = path + strlen(_PATH_TTYOPT); *p != 0; ++p)
+       if (*p == '/')
+           *p = '.';
     ret = options_from_file(path, 0, 0, 1);
     free(path);
     return ret;
@@ -465,7 +462,8 @@ options_for_tty()
  * This could be optimized by using a hash table.
  */
 static option_t *
-find_option(char *name)
+find_option(name)
+    char *name;
 {
     option_t *opt;
     int i;
@@ -505,6 +503,13 @@ process_option(opt, argv)
        option_error("%s option is disabled", opt->name);
        return 0;
     }
+    if ((opt->flags & OPT_PRIVFIX) && !privileged_option) {
+       struct option_info *ip = (struct option_info *) opt->addr2;
+       if (ip && ip->priv) {
+           option_error("%s option cannot be overridden", opt->name);
+           return 0;
+       }
+    }
 
     switch (opt->type) {
     case o_bool:
@@ -573,11 +578,7 @@ process_option(opt, argv)
 
     case o_string:
        if (opt->flags & OPT_STATIC) {
-           if (opt->upper_limit) {
-               strncpy((char *)(opt->addr), *argv, opt->upper_limit);
-               ((char *)(opt->addr))[opt->upper_limit-1] = 0;
-           } else
-               strcpy((char *)(opt->addr), *argv);
+           strlcpy((char *)(opt->addr), *argv, opt->upper_limit);
        } else {
            sv = strdup(*argv);
            if (sv == NULL)
@@ -610,7 +611,8 @@ process_option(opt, argv)
  * n_arguments - tell how many arguments an option takes
  */
 static int
-n_arguments(option_t *opt)
+n_arguments(opt)
+    option_t *opt;
 {
     return (opt->type == o_bool || opt->type == o_special_noarg
            || (opt->flags & OPT_NOARG))? 0: 1;
@@ -619,7 +621,7 @@ n_arguments(option_t *opt)
 /*
  * usage - print out a message telling how to use the program.
  */
-void
+static void
 usage()
 {
     if (phase == PHASE_INITIALIZE)
@@ -649,7 +651,7 @@ showversion(argv)
     char **argv;
 {
     if (phase == PHASE_INITIALIZE) {
-       fprintf(stderr, "pppd version %s patch level %d%s\n",
+       fprintf(stderr, "pppd version %s.%d%s\n",
                VERSION, PATCHLEVEL, IMPLEMENTATION);
        exit(0);
     }
@@ -674,13 +676,14 @@ option_error __V((char *fmt, ...))
     va_start(args);
     fmt = va_arg(args, char *);
 #endif
-    vfmtmsg(buf, sizeof(buf), fmt, args);
+    vslprintf(buf, sizeof(buf), fmt, args);
     va_end(args);
     if (phase == PHASE_INITIALIZE)
        fprintf(stderr, "%s: %s\n", progname, buf);
     syslog(LOG_ERR, "%s", buf);
 }
 
+#if 0
 /*
  * readable - check if a file is readable by the real user.
  */
@@ -689,9 +692,8 @@ readable(fd)
     int fd;
 {
     uid_t uid;
-    int ngroups, i;
+    int i;
     struct stat sbuf;
-    GIDSET_TYPE groups[NGROUPS_MAX];
 
     uid = getuid();
     if (uid == 0)
@@ -702,12 +704,12 @@ readable(fd)
        return sbuf.st_mode & S_IRUSR;
     if (sbuf.st_gid == getgid())
        return sbuf.st_mode & S_IRGRP;
-    ngroups = getgroups(NGROUPS_MAX, groups);
     for (i = 0; i < ngroups; ++i)
        if (sbuf.st_gid == groups[i])
            return sbuf.st_mode & S_IRGRP;
     return sbuf.st_mode & S_IROTH;
 }
+#endif
 
 /*
  * Read a word from a file.
@@ -958,7 +960,7 @@ getword(f, word, newlinep, filename)
 /*
  * number_option - parse an unsigned numeric parameter for an option.
  */
-int
+static int
 number_option(str, valp, base)
     char *str;
     u_int32_t *valp;
@@ -1044,8 +1046,7 @@ callfile(argv)
     l = strlen(arg) + strlen(_PATH_PEERFILES) + 1;
     if ((fname = (char *) malloc(l)) == NULL)
        novm("call file name");
-    strcpy(fname, _PATH_PEERFILES);
-    strcat(fname, arg);
+    slprintf(fname, l, "%s%s", _PATH_PEERFILES, arg);
 
     ok = options_from_file(fname, 1, 1, 1);
 
@@ -1167,9 +1168,8 @@ setdevname(cp, quiet)
        return 0;
 
     if (strncmp("/dev/", cp, 5) != 0) {
-       strcpy(dev, "/dev/");
-       strncat(dev, cp, MAXPATHLEN - 5);
-       dev[MAXPATHLEN-1] = 0;
+       strlcpy(dev, "/dev/", sizeof(dev));
+       strlcat(dev, cp, sizeof(dev));
        cp = dev;
     }
 
@@ -1183,15 +1183,16 @@ setdevname(cp, quiet)
        return -1;
     }
 
-    if (!privileged_option) {
+    if (devnam_info.priv && !privileged_option) {
        if (!quiet)
-           option_error("setting the device name requires root privilege");
+           option_error("device name cannot be overridden");
        return -1;
     }
 
-    (void) strncpy(devnam, cp, MAXPATHLEN);
-    devnam[MAXPATHLEN-1] = 0;
-    default_device = FALSE;
+    strlcpy(devnam, cp, sizeof(devnam));
+    default_device = 0;
+    devnam_info.priv = privileged_option;
+    devnam_info.source = option_source;
   
     return 1;
 }
@@ -1247,10 +1248,8 @@ setipaddr(arg)
                return -1;
            } else {
                remote = *(u_int32_t *)hp->h_addr;
-               if (remote_name[0] == 0) {
-                   strncpy(remote_name, colon, MAXNAMELEN);
-                   remote_name[MAXNAMELEN-1] = 0;
-               }
+               if (remote_name[0] == 0)
+                   strlcpy(remote_name, colon, sizeof(remote_name));
            }
        }
        if (bad_ip_adrs(remote)) {