]> git.ozlabs.org Git - ppp.git/blobdiff - pppd/options.c
Rework options processing stuff to implement option priorities
[ppp.git] / pppd / options.c
index 838a3b8d78738f6716b96b585d9f8e3ebdd8e6f8..ec52cbbf71b0d0ae6db352f53dc9ff00c3292fdb 100644 (file)
  * WARRANTIES OF MERCHANTIBILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
-#ifndef lint
-static char rcsid[] = "$Id: options.c,v 1.51 1999/03/12 06:07:19 paulus Exp $";
-#endif
+#define RCSID  "$Id: options.c,v 1.78 2001/02/22 03:15:20 paulus Exp $"
 
 #include <ctype.h>
 #include <stdio.h>
 #include <errno.h>
 #include <unistd.h>
-#include <limits.h>
+#include <fcntl.h>
 #include <stdlib.h>
-#include <termios.h>
 #include <syslog.h>
 #include <string.h>
-#include <netdb.h>
 #include <pwd.h>
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
+#ifdef PLUGIN
+#include <dlfcn.h>
+#endif
 #ifdef PPP_FILTER
 #include <pcap.h>
 #include <pcap-int.h>  /* XXX: To get struct pcap */
@@ -44,22 +39,13 @@ static char rcsid[] = "$Id: options.c,v 1.51 1999/03/12 06:07:19 paulus Exp $";
 #include "pppd.h"
 #include "pathnames.h"
 #include "patchlevel.h"
-#include "fsm.h"
-#include "lcp.h"
-#include "ipcp.h"
-#include "upap.h"
-#include "chap.h"
-#include "ccp.h"
-
-#include <net/ppp-comp.h>
-
-#define FALSE  0
-#define TRUE   1
 
 #if defined(ultrix) || defined(NeXT)
 char *strdup __P((char *));
 #endif
 
+static const char rcsid[] = RCSID;
+
 /*
  * Option variables and default values.
  */
@@ -69,35 +55,30 @@ int dflag = 0;              /* Tell libpcap we want debugging */
 int    debug = 0;              /* Debug flag */
 int    kdebugflag = 0;         /* Tell kernel to print debug messages */
 int    default_device = 1;     /* Using /dev/tty or equivalent */
-char   devnam[MAXPATHLEN] = "/dev/tty";        /* Device name */
-int    crtscts = 0;            /* Use hardware flow control */
-bool   modem = 1;              /* Use modem control lines */
-int    inspeed = 0;            /* Input/Output speed requested */
-u_int32_t netmask = 0;         /* IP netmask to set on interface */
-bool   lockflag = 0;           /* Create lock file to lock the serial dev */
+char   devnam[MAXPATHLEN];     /* Device name */
 bool   nodetach = 0;           /* Don't detach from controlling tty */
 bool   updetach = 0;           /* Detach once link is up */
-char   *connector = NULL;      /* Script to establish physical link */
-char   *disconnector = NULL;   /* Script to disestablish physical link */
-char   *welcomer = NULL;       /* Script to run after phys link estab. */
 int    maxconnect = 0;         /* Maximum connect time */
 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 */
 int    holdoff = 30;           /* # seconds to pause before reconnecting */
+bool   holdoff_specified;      /* true if a holdoff value has been given */
+int    log_to_fd = 1;          /* send log messages to this fd too */
+int    maxfail = 10;           /* max # of unsuccessful connection attempts */
+char   linkname[MAXPATHLEN];   /* logical name for link */
+bool   tune_kernel;            /* may alter kernel settings */
+int    connect_delay = 1000;   /* wait this many ms after connect script */
+int    req_unit = -1;          /* requested interface unit */
+bool   multilink = 0;          /* Enable multilink operation */
+char   *bundle_name = NULL;    /* bundle name for multilink */
 
 extern option_t auth_options[];
-
-struct option_info connector_info;
-struct option_info disconnector_info;
-struct option_info welcomer_info;
-struct option_info devnam_info;
+extern struct stat devstat;
 
 #ifdef PPP_FILTER
 struct bpf_program pass_filter;/* Filter program for packets to pass */
@@ -105,30 +86,48 @@ struct     bpf_program active_filter; /* Filter program for link-active pkts */
 pcap_t  pc;                    /* Fake struct pcap so we can compile expr */
 #endif
 
+char *current_option;          /* the name of the option being parsed */
+int  privileged_option;                /* set iff the current option came from root */
+char *option_source;           /* string saying where the option came from */
+int  option_priority = OPRIO_CFGFILE; /* priority of the current options */
+bool devnam_fixed;             /* can no longer change device name */
+
+bool log_to_file;              /* log_to_fd is a file opened by us */
+bool log_to_specific_fd;       /* log_to_fd was specified by user option */
+
 /*
  * Prototypes
  */
-static int setdevname __P((char *, int));
-static int setipaddr __P((char *));
-static int setspeed __P((char *));
-static int noopt __P((char **));
 static int setdomain __P((char **));
-static int setnetmask __P((char **));
-static int setxonxoff __P((char **));
 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));
+static int setlogfile __P((char **));
+#ifdef PLUGIN
+static int loadplugin __P((char **));
+#endif
 
 #ifdef PPP_FILTER
 static int setpassfilter __P((char **));
 static int setactivefilter __P((char **));
 #endif
 
-
 static option_t *find_option __P((char *name));
-static int process_option __P((option_t *, char **));
+static int process_option __P((option_t *, char *, char **));
 static int n_arguments __P((option_t *));
+static int number_option __P((char *, u_int32_t *, int));
+
+/*
+ * Structure to store extra lists of options.
+ */
+struct option_list {
+    option_t *options;
+    struct option_list *next;
+};
+
+static struct option_list *extra_options = NULL;
 
 /*
  * Valid arguments.
@@ -150,59 +149,66 @@ option_t general_options[] = {
       "Set time in seconds before retrying connection" },
     { "idle", o_int, &idle_time_limit,
       "Set time in seconds before disconnecting idle link" },
-    { "lock", o_bool, &lockflag,
-      "Lock serial device with UUCP-style lock file", 1 },
-    { "-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 | OPT_PRIVFIX, &connector_info },
-    { "disconnect", o_string, &disconnector,
-      "Program to disconnect serial device",
-      OPT_A2INFO | OPT_PRIVFIX, &disconnector_info },
-    { "welcome", o_string, &welcomer,
-      "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,
-      "Set hardware (RTS/CTS) flow control", OPT_NOARG|OPT_VAL(1) },
-    { "nocrtscts", o_int, &crtscts,
-      "Disable hardware flow control", OPT_NOARG|OPT_VAL(-1) },
-    { "-crtscts", o_int, &crtscts,
-      "Disable hardware flow control", OPT_NOARG|OPT_VAL(-1) },
-    { "cdtrcts", o_int, &crtscts,
-      "Set alternate hardware (DTR/CTS) flow control", OPT_NOARG|OPT_VAL(2) },
-    { "nocdtrcts", o_int, &crtscts,
-      "Disable hardware flow control", OPT_NOARG|OPT_VAL(-1) },
-    { "xonxoff", o_special_noarg, setxonxoff,
-      "Set software (XON/XOFF) flow control" },
-    { "domain", o_special, setdomain,
-      "Add given domain name to hostname" },
-    { "mtu", o_int, &lcp_allowoptions[0].mru,
-      "Set our MTU", OPT_LIMITS, NULL, MAXMRU, MINMRU },
-    { "netmask", o_special, setnetmask,
-      "set netmask" },
-    { "modem", o_bool, &modem,
-      "Use modem control lines", 1 },
-    { "local", o_bool, &modem,
-      "Don't use modem control lines" },
-    { "file", o_special, readfile,
+    { "domain", o_special, (void *)setdomain,
+      "Add given domain name to hostname", OPT_PRIV },
+    { "file", o_special, (void *)readfile,
       "Take options from a file" },
-    { "call", o_special, callfile,
+    { "call", o_special, (void *)callfile,
       "Take options from a privileged file" },
     { "persist", o_bool, &persist,
       "Keep on reopening connection after close", 1 },
     { "nopersist", o_bool, &persist,
       "Turn off persist option" },
     { "demand", o_bool, &demand,
-      "Dial on demand", 1, &persist },
-    { "--version", o_special_noarg, showversion,
+      "Dial on demand", OPT_INITONLY | 1, &persist },
+    { "--version", o_special_noarg, (void *)showversion,
       "Show version number" },
-    { "--help", o_special_noarg, showhelp,
+    { "--help", o_special_noarg, (void *)showhelp,
       "Show brief listing of options" },
-    { "-h", o_special_noarg, showhelp,
+    { "-h", o_special_noarg, (void *)showhelp,
       "Show brief listing of options" },
+    { "logfd", o_int, &log_to_fd,
+      "Send log messages to this file descriptor",
+      0, &log_to_specific_fd },
+    { "logfile", o_special, (void *)setlogfile,
+      "Append log messages to this file" },
+    { "nolog", o_int, &log_to_fd,
+      "Don't send log messages to any file",
+      OPT_NOARG | OPT_VAL(-1) },
+    { "nologfd", o_int, &log_to_fd,
+      "Don't send log messages to any file descriptor",
+      OPT_NOARG | OPT_VAL(-1) },
+    { "linkname", o_string, linkname,
+      "Set logical name for link",
+      OPT_PRIV|OPT_STATIC, NULL, MAXPATHLEN },
+    { "maxfail", o_int, &maxfail,
+      "Maximum number of unsuccessful connection attempts to allow" },
+    { "ktune", o_bool, &tune_kernel,
+      "Alter kernel settings as necessary", 1 },
+    { "noktune", o_bool, &tune_kernel,
+      "Don't alter kernel settings", 0 },
+    { "connect-delay", o_int, &connect_delay,
+      "Maximum time (in ms) to wait after connect script finishes" },
+    { "unit", o_int, &req_unit,
+      "PPP interface unit number to use if possible", OPT_LLIMIT, 0, 0 },
+#ifdef HAVE_MULTILINK
+    { "multilink", o_bool, &multilink,
+      "Enable multilink operation", 1 },
+    { "nomultilink", o_bool, &multilink,
+      "Disable multilink operation", 0 },
+    { "mp", o_bool, &multilink,
+      "Enable multilink operation", 1 },
+    { "nomp", o_bool, &multilink,
+      "Disable multilink operation", 0 },
+    { "bundle", o_string, &bundle_name,
+      "Bundle name for multilink" },
+#endif /* HAVE_MULTILINK */
+#ifdef PLUGIN
+    { "plugin", o_special, (void *)loadplugin,
+      "Load a plug-in module into pppd", OPT_PRIV },
+#endif
 
 #ifdef PPP_FILTER
     { "pdebug", o_int, &dflag,
@@ -248,77 +254,31 @@ parse_args(argc, argv)
 {
     char *arg;
     option_t *opt;
-    int ret;
+    int n;
 
     privileged_option = privileged;
     option_source = "command line";
+    option_priority = OPRIO_CMDLINE;
     while (argc > 0) {
        arg = *argv++;
        --argc;
-
-       /*
-        * First see if it's an option in the new option list.
-        */
        opt = find_option(arg);
-       if (opt != NULL) {
-           int n = n_arguments(opt);
-           if (argc < n) {
-               option_error("too few parameters for option %s", arg);
-               return 0;
-           }
-           current_option = arg;
-           if (!process_option(opt, argv))
-               return 0;
-           argc -= n;
-           argv += n;
-           continue;
-       }
-
-       /*
-        * Maybe a tty name, speed or IP address?
-        */
-       if ((ret = setdevname(arg, 0)) == 0
-           && (ret = setspeed(arg)) == 0
-           && (ret = setipaddr(arg)) == 0) {
+       if (opt == NULL) {
            option_error("unrecognized option '%s'", arg);
            usage();
            return 0;
        }
-       if (ret < 0)    /* error */
+       n = n_arguments(opt);
+       if (argc < n) {
+           option_error("too few parameters for option %s", arg);
            return 0;
-    }
-    return 1;
-}
-
-/*
- * scan_args - scan the command line arguments to get the tty name,
- * if specified.
- */
-void
-scan_args(argc, argv)
-    int argc;
-    char **argv;
-{
-    char *arg;
-    option_t *opt;
-
-    privileged_option = privileged;
-    while (argc > 0) {
-       arg = *argv++;
-       --argc;
-
-       /* Skip options and their arguments */
-       opt = find_option(arg);
-       if (opt != NULL) {
-           int n = n_arguments(opt);
-           argc -= n;
-           argv += n;
-           continue;
        }
-
-       /* Check if it's a tty name and copy it if so */
-       (void) setdevname(arg, 1);
+       if (!process_option(opt, arg, argv))
+           return 0;
+       argc -= n;
+       argv += n;
     }
+    return 1;
 }
 
 /*
@@ -333,9 +293,9 @@ options_from_file(filename, must_exist, check_prot, priv)
     int priv;
 {
     FILE *f;
-    int i, newline, ret;
+    int i, newline, ret, err;
     option_t *opt;
-    int oldpriv;
+    int oldpriv, n;
     char *oldsource;
     char *argv[MAXARGS];
     char args[MAXARGS][MAXWORDLEN];
@@ -344,21 +304,19 @@ options_from_file(filename, must_exist, check_prot, priv)
     if (check_prot)
        seteuid(getuid());
     f = fopen(filename, "r");
+    err = errno;
     if (check_prot)
        seteuid(0);
     if (f == NULL) {
-       if (!must_exist && errno == ENOENT)
+       errno = err;
+       if (!must_exist) {
+           if (err != ENOENT && err != ENOTDIR)
+               warn("Warning: can't open options file %s: %m", filename);
            return 1;
+       }
        option_error("Can't open options file %s: %m", filename);
        return 0;
     }
-#if 0  /* check done by setting effective UID above */
-    if (check_prot && !readable(fileno(f))) {
-       option_error("Can't open options file %s: access denied", filename);
-       fclose(f);
-       return 0;
-    }
-#endif
 
     oldpriv = privileged_option;
     privileged_option = priv;
@@ -368,38 +326,23 @@ options_from_file(filename, must_exist, check_prot, priv)
        option_source = "file";
     ret = 0;
     while (getword(f, cmd, &newline, filename)) {
-       /*
-        * First see if it's a command.
-        */
        opt = find_option(cmd);
-       if (opt != NULL) {
-           int n = n_arguments(opt);
-           for (i = 0; i < n; ++i) {
-               if (!getword(f, args[i], &newline, filename)) {
-                   option_error(
-                       "In file %s: too few parameters for option '%s'",
-                       filename, cmd);
-                   goto err;
-               }
-               argv[i] = args[i];
-           }
-           current_option = cmd;
-           if (!process_option(opt, argv))
-               goto err;
-           continue;
-       }
-
-       /*
-        * Maybe a tty name, speed or IP address?
-        */
-       if ((i = setdevname(cmd, 0)) == 0
-           && (i = setspeed(cmd)) == 0
-           && (i = setipaddr(cmd)) == 0) {
+       if (opt == NULL) {
            option_error("In file %s: unrecognized option '%s'",
                         filename, cmd);
            goto err;
        }
-       if (i < 0)              /* error */
+       n = n_arguments(opt);
+       for (i = 0; i < n; ++i) {
+           if (!getword(f, args[i], &newline, filename)) {
+               option_error(
+                       "In file %s: too few parameters for option '%s'",
+                       filename, cmd);
+               goto err;
+           }
+           argv[i] = args[i];
+       }
+       if (!process_option(opt, cmd, argv))
            goto err;
     }
     ret = 1;
@@ -432,6 +375,7 @@ options_from_user()
     if (path == NULL)
        novm("init file name");
     slprintf(path, pl, "%s/%s", user, file);
+    option_priority = OPRIO_CFGFILE;
     ret = options_from_file(path, 0, 1, privileged);
     free(path);
     return ret;
@@ -440,6 +384,10 @@ options_from_user()
 /*
  * options_for_tty - See if an options file exists for the serial
  * device, and if so, interpret options from it.
+ * We only allow the per-tty options file to override anything from
+ * the command line if it is something that the user can't override
+ * once it has been set by root; this is done by giving configuration
+ * files a lower priority than the command line.
  */
 int
 options_for_tty()
@@ -451,7 +399,7 @@ options_for_tty()
     dev = devnam;
     if (strncmp(dev, "/dev/", 5) == 0)
        dev += 5;
-    if (strcmp(dev, "tty") == 0)
+    if (dev[0] == 0 || strcmp(dev, "tty") == 0)
        return 1;               /* don't look for /etc/ppp/options.tty */
     pl = strlen(_PATH_TTYOPT) + strlen(dev) + 1;
     path = malloc(pl);
@@ -462,11 +410,76 @@ options_for_tty()
     for (p = path + strlen(_PATH_TTYOPT); *p != 0; ++p)
        if (*p == '/')
            *p = '.';
+    option_priority = OPRIO_CFGFILE;
     ret = options_from_file(path, 0, 0, 1);
     free(path);
     return ret;
 }
 
+/*
+ * options_from_list - process a string of options in a wordlist.
+ */
+int
+options_from_list(w, priv)
+    struct wordlist *w;
+    int priv;
+{
+    char *argv[MAXARGS];
+    option_t *opt;
+    int i, n, ret = 0;
+    struct wordlist *w0;
+
+    privileged_option = priv;
+    option_source = "secrets file";
+    option_priority = OPRIO_SECFILE;
+
+    while (w != NULL) {
+       opt = find_option(w->word);
+       if (opt == NULL) {
+           option_error("In secrets file: unrecognized option '%s'",
+                        w->word);
+           goto err;
+       }
+       n = n_arguments(opt);
+       w0 = w;
+       for (i = 0; i < n; ++i) {
+           w = w->next;
+           if (w == NULL) {
+               option_error(
+                       "In secrets file: too few parameters for option '%s'",
+                       w0->word);
+               goto err;
+           }
+           argv[i] = w->word;
+       }
+       if (!process_option(opt, w0->word, argv))
+           goto err;
+    }
+    ret = 1;
+
+err:
+    return ret;
+}
+
+/*
+ * match_option - see if this option matches an option_t structure.
+ */
+static int
+match_option(name, opt, dowild)
+    char *name;
+    option_t *opt;
+    int dowild;
+{
+       int (*match) __P((char *, char **, int));
+
+       if (dowild != (opt->type == o_wild))
+               return 0;
+       if (!dowild)
+               return strcmp(name, opt->name) == 0;
+       match = (int (*) __P((char *, char **, int))) opt->addr;
+       return (*match)(name, NULL, 0);
+}
+
 /*
  * find_option - scan the option lists for the various protocols
  * looking for an entry with the given name.
@@ -476,50 +489,85 @@ static option_t *
 find_option(name)
     char *name;
 {
-    option_t *opt;
-    int i;
-
-    for (opt = general_options; opt->name != NULL; ++opt)
-       if (strcmp(name, opt->name) == 0)
-           return opt;
-    for (opt = auth_options; opt->name != NULL; ++opt)
-       if (strcmp(name, opt->name) == 0)
-           return opt;
-    for (i = 0; protocols[i] != NULL; ++i)
-       if ((opt = protocols[i]->options) != NULL)
-           for (; opt->name != NULL; ++opt)
-               if (strcmp(name, opt->name) == 0)
-                   return opt;
-    return NULL;
+       option_t *opt;
+       struct option_list *list;
+       int i, dowild;
+
+       for (dowild = 0; dowild <= 1; ++dowild) {
+               for (opt = general_options; opt->name != NULL; ++opt)
+                       if (match_option(name, opt, dowild))
+                               return opt;
+               for (opt = auth_options; opt->name != NULL; ++opt)
+                       if (match_option(name, opt, dowild))
+                               return opt;
+               for (list = extra_options; list != NULL; list = list->next)
+                       for (opt = list->options; opt->name != NULL; ++opt)
+                               if (match_option(name, opt, dowild))
+                                       return opt;
+               for (i = 0; protocols[i] != NULL; ++i)
+                       if ((opt = protocols[i]->options) != NULL)
+                               for (; opt->name != NULL; ++opt)
+                                       if (match_option(name, opt, dowild))
+                                               return opt;
+       }
+       return NULL;
 }
 
 /*
  * process_option - process one new-style option.
  */
 static int
-process_option(opt, argv)
+process_option(opt, cmd, argv)
     option_t *opt;
+    char *cmd;
     char **argv;
 {
     u_int32_t v;
     int iv, a;
     char *sv;
     int (*parser) __P((char **));
+    int (*wildp) __P((char *, char **, int));
+    char *optopt = (opt->type == o_wild)? "": " option";
+    int prio = option_priority;
+
+    if ((opt->flags & OPT_PRIVFIX) && privileged_option)
+       prio += OPRIO_ROOT;
+    /* Multipart options (like the local:remote IP address option)
+       need to keep a separate priority value for each part,
+       so we let the parser function handle that. */
+    if ((opt->flags & OPT_MULTIPART) == 0) {
+       if (prio < opt->priority) {
+           /* new value doesn't override old */
+           if (prio == OPRIO_CMDLINE && opt->priority > OPRIO_ROOT) {
+               option_error("%s%s set in %s cannot be overridden\n",
+                                opt->name, optopt, opt->source);
+               return 0;
+           }
+           return 1;
+       }
+       if (prio > OPRIO_ROOT && opt->priority == OPRIO_CMDLINE)
+           warn("%s%s from %s overrides command line",
+                opt->name, optopt, option_source);
+    }
 
+    if ((opt->flags & OPT_INITONLY) && phase != PHASE_INITIALIZE) {
+       option_error("%s%s cannot be changed after initialization",
+                    opt->name, optopt);
+       return 0;
+    }
     if ((opt->flags & OPT_PRIV) && !privileged_option) {
-       option_error("using the %s option requires root privilege", opt->name);
+       option_error("using the %s%s requires root privilege",
+                    opt->name, optopt);
        return 0;
     }
     if ((opt->flags & OPT_ENABLE) && *(bool *)(opt->addr2) == 0) {
-       option_error("%s option is disabled", opt->name);
+       option_error("%s%s is disabled", opt->name, optopt);
        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;
-       }
+    if ((opt->flags & OPT_DEVEQUIV) && devnam_fixed) {
+       option_error("the %s%s may not be changed in %s",
+                    opt->name, optopt, option_source);
+       return 0;
     }
 
     switch (opt->type) {
@@ -589,7 +637,7 @@ process_option(opt, argv)
 
     case o_string:
        if (opt->flags & OPT_STATIC) {
-           strlcpy((char *)(opt->addr), opt->upper_limit, *argv);
+           strlcpy((char *)(opt->addr), *argv, opt->upper_limit);
        } else {
            sv = strdup(*argv);
            if (sv == NULL)
@@ -604,17 +652,20 @@ process_option(opt, argv)
        if (!(*parser)(argv))
            return 0;
        break;
-    }
 
-    if (opt->addr2) {
-       if (opt->flags & OPT_A2INFO) {
-           struct option_info *ip = (struct option_info *) opt->addr2;
-           ip->priv = privileged_option;
-           ip->source = option_source;
-       } else if ((opt->flags & (OPT_A2COPY|OPT_ENABLE)) == 0)
-           *(bool *)(opt->addr2) = 1;
+    case o_wild:
+       wildp = (int (*) __P((char *, char **, int))) opt->addr;
+       if (!(*wildp)(cmd, argv, 1))
+           return 0;
+       break;
     }
 
+    if (opt->addr2 && (opt->flags & (OPT_A2COPY|OPT_ENABLE)) == 0)
+       *(bool *)(opt->addr2) = 1;
+
+    opt->source = option_source;
+    opt->priority = prio;
+
     return 1;
 }
 
@@ -625,14 +676,31 @@ static int
 n_arguments(opt)
     option_t *opt;
 {
-    return (opt->type == o_bool || opt->type == o_special_noarg
-           || (opt->flags & OPT_NOARG))? 0: 1;
+       return (opt->type == o_bool || opt->type == o_special_noarg
+               || (opt->flags & OPT_NOARG))? 0: 1;
 }
 
 /*
- * usage - print out a message telling how to use the program.
+ * add_options - add a list of options to the set we grok.
  */
 void
+add_options(opt)
+    option_t *opt;
+{
+    struct option_list *list;
+
+    list = malloc(sizeof(*list));
+    if (list == 0)
+       novm("option list entry");
+    list->options = opt;
+    list->next = extra_options;
+    extra_options = list;
+}
+
+/*
+ * usage - print out a message telling how to use the program.
+ */
+static void
 usage()
 {
     if (phase == PHASE_INITIALIZE)
@@ -678,9 +746,9 @@ void
 option_error __V((char *fmt, ...))
 {
     va_list args;
-    char buf[256];
+    char buf[1024];
 
-#if __STDC__
+#if defined(__STDC__)
     va_start(args, fmt);
 #else
     char *fmt;
@@ -694,6 +762,7 @@ option_error __V((char *fmt, ...))
     syslog(LOG_ERR, "%s", buf);
 }
 
+#if 0
 /*
  * readable - check if a file is readable by the real user.
  */
@@ -719,6 +788,7 @@ readable(fd)
            return sbuf.st_mode & S_IRGRP;
     return sbuf.st_mode & S_IROTH;
 }
+#endif
 
 /*
  * Read a word from a file.
@@ -969,7 +1039,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;
@@ -1064,16 +1134,6 @@ callfile(argv)
 }
 
 #ifdef PPP_FILTER
-/*
- * setpdebug - Set libpcap debugging level.
- */
-static int
-setpdebug(argv)
-    char **argv;
-{
-    return int_option(*argv, &dflag);
-}
-
 /*
  * setpassfilter - Set the pass filter for packets
  */
@@ -1107,21 +1167,6 @@ setactivefilter(argv)
 }
 #endif
 
-/*
- * noopt - Disable all options.
- */
-static int
-noopt(argv)
-    char **argv;
-{
-    BZERO((char *) &lcp_wantoptions[0], sizeof (struct lcp_options));
-    BZERO((char *) &lcp_allowoptions[0], sizeof (struct lcp_options));
-    BZERO((char *) &ipcp_wantoptions[0], sizeof (struct ipcp_options));
-    BZERO((char *) &ipcp_allowoptions[0], sizeof (struct ipcp_options));
-
-    return (1);
-}
-
 /*
  * setdomain - Set domain name to append to hostname 
  */
@@ -1129,10 +1174,6 @@ static int
 setdomain(argv)
     char **argv;
 {
-    if (!privileged_option) {
-       option_error("using the domain option requires root privilege");
-       return 0;
-    }
     gethostname(hostname, MAXNAMELEN);
     if (**argv != 0) {
        if (**argv != '.')
@@ -1144,190 +1185,58 @@ setdomain(argv)
 }
 
 
-/*
- * setspeed - Set the speed.
- */
-static int
-setspeed(arg)
-    char *arg;
-{
-    char *ptr;
-    int spd;
-
-    spd = strtol(arg, &ptr, 0);
-    if (ptr == arg || *ptr != 0 || spd == 0)
-       return 0;
-    inspeed = spd;
-    return 1;
-}
-
-
-/*
- * setdevname - Set the device name.
- */
 static int
-setdevname(cp, quiet)
-    char *cp;
-    int quiet;
+setlogfile(argv)
+    char **argv;
 {
-    struct stat statbuf;
-    char dev[MAXPATHLEN];
-
-    if (*cp == 0)
-       return 0;
-
-    if (strncmp("/dev/", cp, 5) != 0) {
-       strlcpy(dev, sizeof(dev), "/dev/");
-       strlcat(dev, sizeof(dev), cp);
-       cp = dev;
-    }
+    int fd, err;
 
-    /*
-     * Check if there is a device by this name.
-     */
-    if (stat(cp, &statbuf) < 0) {
-       if (errno == ENOENT || quiet)
-           return 0;
-       option_error("Couldn't stat %s: %m", cp);
-       return -1;
-    }
-
-    if (devnam_info.priv && !privileged_option) {
-       if (!quiet)
-           option_error("device name cannot be overridden");
-       return -1;
-    }
-
-    strlcpy(devnam, sizeof(devnam), cp);
-    default_device = FALSE;
-    devnam_info.priv = privileged_option;
-    devnam_info.source = option_source;
-  
-    return 1;
-}
-
-
-/*
- * setipaddr - Set the IP address
- */
-static int
-setipaddr(arg)
-    char *arg;
-{
-    struct hostent *hp;
-    char *colon;
-    u_int32_t local, remote;
-    ipcp_options *wo = &ipcp_wantoptions[0];
-  
-    /*
-     * IP address pair separated by ":".
-     */
-    if ((colon = strchr(arg, ':')) == NULL)
+    if (!privileged_option)
+       seteuid(getuid());
+    fd = open(*argv, O_WRONLY | O_APPEND | O_CREAT | O_EXCL, 0644);
+    if (fd < 0 && errno == EEXIST)
+       fd = open(*argv, O_WRONLY | O_APPEND);
+    err = errno;
+    if (!privileged_option)
+       seteuid(0);
+    if (fd < 0) {
+       errno = err;
+       option_error("Can't open log file %s: %m", *argv);
        return 0;
-  
-    /*
-     * If colon first character, then no local addr.
-     */
-    if (colon != arg) {
-       *colon = '\0';
-       if ((local = inet_addr(arg)) == -1) {
-           if ((hp = gethostbyname(arg)) == NULL) {
-               option_error("unknown host: %s", arg);
-               return -1;
-           } else {
-               local = *(u_int32_t *)hp->h_addr;
-           }
-       }
-       if (bad_ip_adrs(local)) {
-           option_error("bad local IP address %s", ip_ntoa(local));
-           return -1;
-       }
-       if (local != 0)
-           wo->ouraddr = local;
-       *colon = ':';
-    }
-  
-    /*
-     * If colon last character, then no remote addr.
-     */
-    if (*++colon != '\0') {
-       if ((remote = inet_addr(colon)) == -1) {
-           if ((hp = gethostbyname(colon)) == NULL) {
-               option_error("unknown host: %s", colon);
-               return -1;
-           } else {
-               remote = *(u_int32_t *)hp->h_addr;
-               if (remote_name[0] == 0)
-                   strlcpy(remote_name, sizeof(remote_name), colon);
-           }
-       }
-       if (bad_ip_adrs(remote)) {
-           option_error("bad remote IP address %s", ip_ntoa(remote));
-           return -1;
-       }
-       if (remote != 0)
-           wo->hisaddr = remote;
     }
-
+    if (log_to_file && log_to_fd >= 0)
+       close(log_to_fd);
+    log_to_fd = fd;
+    log_to_file = 1;
     return 1;
 }
 
-
-/*
- * setnetmask - set the netmask to be used on the interface.
- */
+#ifdef PLUGIN
 static int
-setnetmask(argv)
+loadplugin(argv)
     char **argv;
 {
-    u_int32_t mask, b;
-    int n, ok;
-    char *p, *endp;
-
-    /*
-     * Unfortunately, if we use inet_addr, we can't tell whether
-     * a result of all 1s is an error or a valid 255.255.255.255.
-     */
-    p = *argv;
-    ok = 0;
-    mask = 0;
-    for (n = 3;; --n) {
-       b = strtoul(p, &endp, 0);
-       if (endp == p)
-           break;
-       if (b < 0 || b > 255) {
-           if (n == 3) {
-               /* accept e.g. 0xffffff00 */
-               p = endp;
-               mask = b;
-           }
-           break;
-       }
-       mask |= b << (n * 8);
-       p = endp;
-       if (*p != '.' || n == 0)
-           break;
-       ++p;
+    char *arg = *argv;
+    void *handle;
+    const char *err;
+    void (*init) __P((void));
+
+    handle = dlopen(arg, RTLD_GLOBAL | RTLD_NOW);
+    if (handle == 0) {
+       err = dlerror();
+       if (err != 0)
+           option_error("%s", err);
+       option_error("Couldn't load plugin %s", arg);
+       return 0;
     }
-
-    mask = htonl(mask);
-
-    if (*p != 0 || (netmask & ~mask) != 0) {
-       option_error("invalid netmask value '%s'", *argv);
+    init = (void (*)(void))dlsym(handle, "plugin_init");
+    if (init == 0) {
+       option_error("%s has no initialization entry point", arg);
+       dlclose(handle);
        return 0;
     }
-
-    netmask = mask;
-    return (1);
-}
-
-static int
-setxonxoff(argv)
-    char **argv;
-{
-    lcp_wantoptions[0].asyncmap |= 0x000A0000; /* escape ^S and ^Q */
-    lcp_wantoptions[0].neg_asyncmap = 1;
-
-    crtscts = -2;
-    return (1);
+    info("Plugin %s loaded.", arg);
+    (*init)();
+    return 1;
 }
+#endif /* PLUGIN */