If device name given, complain if not privileged
[ppp.git] / pppd / options.c
index 87a902fc0bb87524b878f14656afbb886aba4be0..346430d454f0037d2450e444ba7865ad03c77147 100644 (file)
  */
 
 #ifndef lint
-static char rcsid[] = "$Id: options.c,v 1.23 1995/07/04 12:33:01 paulus Exp $";
+static char rcsid[] = "$Id: options.c,v 1.45 1999/01/20 00:00:35 paulus Exp $";
 #endif
 
+#include <ctype.h>
 #include <stdio.h>
 #include <errno.h>
 #include <unistd.h>
@@ -34,6 +35,11 @@ static char rcsid[] = "$Id: options.c,v 1.23 1995/07/04 12:33:01 paulus Exp $";
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <netinet/in.h>
+#include <arpa/inet.h>
+#ifdef PPP_FILTER
+#include <pcap.h>
+#include <pcap-int.h>  /* XXX: To get struct pcap */
+#endif
 
 #include "pppd.h"
 #include "pathnames.h"
@@ -61,200 +67,156 @@ char *strdup __P((char *));
 /*
  * Option variables and default values.
  */
+#ifdef PPP_FILTER
+int    dflag = 0;              /* Tell libpcap we want debugging */
+#endif
 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 */
-int    modem = 1;              /* Use modem control lines */
+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 */
-int    lockflag = 0;           /* Create lock file to lock the serial dev */
-int    nodetach = 0;           /* Don't detach from controlling tty */
+bool   lockflag = 0;           /* Create lock file to lock the serial dev */
+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 */
-int    auth_required = 0;      /* Peer is required to authenticate */
-int    defaultroute = 0;       /* assign default route through interface */
-int    proxyarp = 0;           /* Set up proxy ARP entry for peer */
-int    persist = 0;            /* Reopen link after it goes down */
-int    uselogin = 0;           /* Use /etc/passwd for checking PAP */
-int    lcp_echo_interval = 0;  /* Interval between LCP echo-requests */
-int    lcp_echo_fails = 0;     /* Tolerance to unanswered echo-requests */
+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    usehostname = 0;        /* Use hostname for our_name */
-int    disable_defaultip = 0;  /* Don't use hostname for default IP adrs */
+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    cryptpap;               /* Passwords in pap-secrets are encrypted */
+int    idle_time_limit = 0;    /* Disconnect if idle for this many seconds */
+int    holdoff = 30;           /* # seconds to pause before reconnecting */
+
+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;
 
-#ifdef _linux_
-int idle_time_limit = 0;
-static int setidle __P((char **));
+#ifdef PPP_FILTER
+struct bpf_program pass_filter;/* Filter program for packets to pass */
+struct bpf_program active_filter; /* Filter program for link-active pkts */
+pcap_t  pc;                    /* Fake struct pcap so we can compile expr */
 #endif
 
 /*
  * Prototypes
  */
-static int setdebug __P((void));
-static int setkdebug __P((char **));
-static int setpassive __P((void));
-static int setsilent __P((void));
-static int noopt __P((void));
-static int setnovj __P((void));
-static int setnovjccomp __P((void));
-static int setvjslots __P((char **));
-static int reqpap __P((void));
-static int nopap __P((void));
-static int setupapfile __P((char **));
-static int nochap __P((void));
-static int reqchap __P((void));
+static int setdevname __P((char *, int));
+static int setipaddr __P((char *));
 static int setspeed __P((char *));
-static int noaccomp __P((void));
-static int noasyncmap __P((void));
-static int noipaddr __P((void));
-static int nomagicnumber __P((void));
-static int setasyncmap __P((char **));
-static int setescape __P((char **));
-static int setmru __P((char **));
-static int setmtu __P((char **));
-static int nomru __P((void));
-static int nopcomp __P((void));
-static int setconnector __P((char **));
-static int setdisconnector __P((char **));
+static int noopt __P((char **));
 static int setdomain __P((char **));
 static int setnetmask __P((char **));
-static int setcrtscts __P((void));
-static int setnocrtscts __P((void));
-static int setxonxoff __P((void));
-static int setnodetach __P((void));
-static int setmodem __P((void));
-static int setlocal __P((void));
-static int setlock __P((void));
-static int setname __P((char **));
-static int setuser __P((char **));
-static int setremote __P((char **));
-static int setauth __P((void));
+static int setxonxoff __P((char **));
 static int readfile __P((char **));
-static int setdefaultroute __P((void));
-static int setproxyarp __P((void));
-static int setpersist __P((void));
-static int setdologin __P((void));
-static int setusehostname __P((void));
-static int setnoipdflt __P((void));
-static int setlcptimeout __P((char **));
-static int setlcpterm __P((char **));
-static int setlcpconf __P((char **));
-static int setlcpfails __P((char **));
-static int setipcptimeout __P((char **));
-static int setipcpterm __P((char **));
-static int setipcpconf __P((char **));
-static int setipcpfails __P((char **));
-static int setpaptimeout __P((char **));
-static int setpapreqs __P((char **));
-static int setpapreqtime __P((char **));
-static int setchaptimeout __P((char **));
-static int setchapchal __P((char **));
-static int setchapintv __P((char **));
-static int setipcpaccl __P((void));
-static int setipcpaccr __P((void));
-static int setlcpechointv __P((char **));
-static int setlcpechofails __P((char **));
-static int setbsdcomp __P((char **));
-static int setnobsdcomp __P((void));
-static int setipparam __P((char **));
-static int setpapcrypt __P((void));
-
-static int number_option __P((char *, u_int32_t *, int));
-static int readable __P((int fd));
+static int callfile __P((char **));
+static int showversion __P((char **));
+static int showhelp __P((char **));
+
+#ifdef PPP_FILTER
+static int setpassfilter __P((char **));
+static int setactivefilter __P((char **));
+#endif
+
 
-void usage();
+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 readable __P((int fd));
 
 /*
  * Valid arguments.
  */
-static struct cmd {
-    char *cmd_name;
-    int num_args;
-    int (*cmd_func)();
-} cmds[] = {
-    {"-all", 0, noopt},                /* Don't request/allow any options */
-    {"-ac", 0, noaccomp},      /* Disable Address/Control compress */
-    {"-am", 0, noasyncmap},    /* Disable asyncmap negotiation */
-    {"-as", 1, setasyncmap},   /* set the desired async map */
-    {"-d", 0, setdebug},       /* Increase debugging level */
-    {"-detach", 0, setnodetach}, /* don't fork */
-    {"-ip", 0, noipaddr},      /* Disable IP address negotiation */
-    {"-mn", 0, nomagicnumber}, /* Disable magic number negotiation */
-    {"-mru", 0, nomru},                /* Disable mru negotiation */
-    {"-p", 0, setpassive},     /* Set passive mode */
-    {"-pc", 0, nopcomp},       /* Disable protocol field compress */
-    {"+ua", 1, setupapfile},   /* Get PAP user and password from file */
-    {"+pap", 0, reqpap},       /* Require PAP auth from peer */
-    {"-pap", 0, nopap},                /* Don't allow UPAP authentication with peer */
-    {"+chap", 0, reqchap},     /* Require CHAP authentication from peer */
-    {"-chap", 0, nochap},      /* Don't allow CHAP authentication with peer */
-    {"-vj", 0, setnovj},       /* disable VJ compression */
-    {"-vjccomp", 0, setnovjccomp}, /* disable VJ connection-ID compression */
-    {"vj-max-slots", 1, setvjslots}, /* Set maximum VJ header slots */
-    {"asyncmap", 1, setasyncmap}, /* set the desired async map */
-    {"escape", 1, setescape},  /* set chars to escape on transmission */
-    {"connect", 1, setconnector}, /* A program to set up a connection */
-    {"disconnect", 1, setdisconnector},        /* program to disconnect serial dev. */
-    {"crtscts", 0, setcrtscts},        /* set h/w flow control */
-    {"-crtscts", 0, setnocrtscts}, /* clear h/w flow control */
-    {"xonxoff", 0, setxonxoff},        /* set s/w flow control */
-    {"debug", 0, setdebug},    /* Increase debugging level */
-    {"kdebug", 1, setkdebug},  /* Enable kernel-level debugging */
-    {"domain", 1, setdomain},  /* Add given domain name to hostname*/
-    {"mru", 1, setmru},                /* Set MRU value for negotiation */
-    {"mtu", 1, setmtu},                /* Set our MTU */
-    {"netmask", 1, setnetmask},        /* set netmask */
-    {"passive", 0, setpassive},        /* Set passive mode */
-    {"silent", 0, setsilent},  /* Set silent mode */
-    {"modem", 0, setmodem},    /* Use modem control lines */
-    {"local", 0, setlocal},    /* Don't use modem control lines */
-    {"lock", 0, setlock},      /* Lock serial device (with lock file) */
-    {"name", 1, setname},      /* Set local name for authentication */
-    {"user", 1, setuser},      /* Set username for PAP auth with peer */
-    {"usehostname", 0, setusehostname},        /* Must use hostname for auth. */
-    {"remotename", 1, setremote}, /* Set remote name for authentication */
-    {"auth", 0, setauth},      /* Require authentication from peer */
-    {"file", 1, readfile},     /* Take options from a file */
-    {"defaultroute", 0, setdefaultroute}, /* Add default route */
-    {"proxyarp", 0, setproxyarp}, /* Add proxy ARP entry */
-    {"persist", 0, setpersist},        /* Keep on reopening connection after close */
-    {"login", 0, setdologin},  /* Use system password database for UPAP */
-    {"noipdefault", 0, setnoipdflt}, /* Don't use name for default IP adrs */
-    {"lcp-echo-failure", 1, setlcpechofails}, /* consecutive echo failures */
-    {"lcp-echo-interval", 1, setlcpechointv}, /* time for lcp echo events */
-    {"lcp-restart", 1, setlcptimeout}, /* Set timeout for LCP */
-    {"lcp-max-terminate", 1, setlcpterm}, /* Set max #xmits for term-reqs */
-    {"lcp-max-configure", 1, setlcpconf}, /* Set max #xmits for conf-reqs */
-    {"lcp-max-failure", 1, setlcpfails}, /* Set max #conf-naks for LCP */
-    {"ipcp-restart", 1, setipcptimeout}, /* Set timeout for IPCP */
-    {"ipcp-max-terminate", 1, setipcpterm}, /* Set max #xmits for term-reqs */
-    {"ipcp-max-configure", 1, setipcpconf}, /* Set max #xmits for conf-reqs */
-    {"ipcp-max-failure", 1, setipcpfails}, /* Set max #conf-naks for IPCP */
-    {"pap-restart", 1, setpaptimeout}, /* Set retransmit timeout for PAP */
-    {"pap-max-authreq", 1, setpapreqs}, /* Set max #xmits for auth-reqs */
-    {"pap-timeout", 1, setpapreqtime}, /* Set time limit for peer PAP auth. */
-    {"chap-restart", 1, setchaptimeout}, /* Set timeout for CHAP */
-    {"chap-max-challenge", 1, setchapchal}, /* Set max #xmits for challenge */
-    {"chap-interval", 1, setchapintv}, /* Set interval for rechallenge */
-    {"ipcp-accept-local", 0, setipcpaccl}, /* Accept peer's address for us */
-    {"ipcp-accept-remote", 0, setipcpaccr}, /* Accept peer's address for it */
-    {"bsdcomp", 1, setbsdcomp},                /* request BSD-Compress */
-    {"-bsdcomp", 0, setnobsdcomp},     /* don't allow BSD-Compress */
-    {"ipparam", 1, setipparam},                /* set ip script parameter */
-    {"papcrypt", 0, setpapcrypt},      /* PAP passwords encrypted */
-#ifdef _linux_
-    {"idle-disconnect", 1, setidle}, /* seconds for disconnect of idle IP */
+option_t general_options[] = {
+    { "debug", o_int, &debug,
+      "Increase debugging level", OPT_INC|OPT_NOARG|1 },
+    { "-d", o_int, &debug,
+      "Increase debugging level", OPT_INC|OPT_NOARG|1 },
+    { "kdebug", o_int, &kdebugflag,
+      "Set kernel driver debug level" },
+    { "nodetach", o_bool, &nodetach,
+      "Don't detach from controlling tty", 1 },
+    { "-detach", o_bool, &nodetach,
+      "Don't detach from controlling tty", 1 },
+    { "updetach", o_bool, &updetach,
+      "Detach from controlling tty once link is up", 1 },
+    { "holdoff", o_int, &holdoff,
+      "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, &connector_info },
+    { "disconnect", o_string, &disconnector,
+      "Program to disconnect serial device", OPT_A2INFO, &disconnector_info },
+    { "welcome", o_string, &welcomer,
+      "Script to welcome client", OPT_A2INFO, &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,
+      "Take options from a file" },
+    { "call", o_special, 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,
+      "Show version number" },
+    { "--help", o_special_noarg, showhelp,
+      "Show brief listing of options" },
+    { "-h", o_special_noarg, showhelp,
+      "Show brief listing of options" },
+
+#ifdef PPP_FILTER
+    { "pdebug", o_int, &dflag,
+      "libpcap debugging" },
+    { "pass-filter", 1, setpassfilter,
+      "set filter for packets to pass" },
+    { "active-filter", 1, setactivefilter,
+      "set filter for active pkts" },
 #endif
-    {NULL, 0, NULL}
-};
 
+    { NULL }
+};
 
 #ifndef IMPLEMENTATION
 #define IMPLEMENTATION ""
@@ -262,7 +224,7 @@ static struct cmd {
 
 static char *usage_string = "\
 pppd version %s patch level %d%s\n\
-Usage: %s [ arguments ], where arguments are:\n\
+Usage: %s [ options ], where options are:\n\
        <device>        Communicate over the named device\n\
        <speed>         Set the baud rate to <speed>\n\
        <loc>:<rem>     Set the local and/or remote interface IP\n\
@@ -275,71 +237,90 @@ Usage: %s [ arguments ], where arguments are:\n\
        file <f>        Take options from file <f>\n\
        modem           Use modem control lines\n\
        mru <n>         Set MRU value to <n> for negotiation\n\
-       netmask <n>     Set interface netmask to <n>\n\
 See pppd(8) for more options.\n\
 ";
 
-
 /*
- * parse_args - parse a string of arguments, from the command
- * line or from a file.
+ * parse_args - parse a string of arguments from the command line.
  */
 int
 parse_args(argc, argv)
     int argc;
     char **argv;
 {
-    char *arg, *val;
-    struct cmd *cmdp;
+    char *arg;
+    option_t *opt;
     int ret;
 
+    privileged_option = privileged;
+    option_source = "command line";
     while (argc > 0) {
        arg = *argv++;
        --argc;
 
        /*
-        * First see if it's a command.
+        * First see if it's an option in the new option list.
         */
-       for (cmdp = cmds; cmdp->cmd_name; cmdp++)
-           if (!strcmp(arg, cmdp->cmd_name))
-               break;
-
-       if (cmdp->cmd_name != NULL) {
-           if (argc < cmdp->num_args) {
-               fprintf(stderr, "Too few parameters for command %s\n", arg);
+       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;
            }
-           if (!(*cmdp->cmd_func)(argv))
+           current_option = arg;
+           if (!process_option(opt, argv))
                return 0;
-           argc -= cmdp->num_args;
-           argv += cmdp->num_args;
+           argc -= n;
+           argv += n;
+           continue;
+       }
 
-       } else {
-           /*
-            * Maybe a tty name, speed or IP address?
-            */
-           if ((ret = setdevname(arg)) == 0
-               && (ret = setspeed(arg)) == 0
-               && (ret = setipaddr(arg)) == 0) {
-               fprintf(stderr, "%s: unrecognized command\n", arg);
-               usage();
-               return 0;
-           }
-           if (ret < 0)        /* error */
-               return 0;
+       /*
+        * Maybe a tty name, speed or IP address?
+        */
+       if ((ret = setdevname(arg, 0)) == 0
+           && (ret = setspeed(arg)) == 0
+           && (ret = setipaddr(arg)) == 0) {
+           option_error("unrecognized option '%s'", arg);
+           usage();
+           return 0;
        }
+       if (ret < 0)    /* error */
+           return 0;
     }
     return 1;
 }
 
 /*
- * usage - print out a message telling how to use the program.
+ * scan_args - scan the command line arguments to get the tty name,
+ * if specified.
  */
 void
-usage()
+scan_args(argc, argv)
+    int argc;
+    char **argv;
 {
-    fprintf(stderr, usage_string, VERSION, PATCHLEVEL, IMPLEMENTATION,
-           progname);
+    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);
+    }
 }
 
 /*
@@ -347,14 +328,17 @@ usage()
  * and interpret them.
  */
 int
-options_from_file(filename, must_exist, check_prot)
+options_from_file(filename, must_exist, check_prot, priv)
     char *filename;
     int must_exist;
     int check_prot;
+    int priv;
 {
     FILE *f;
     int i, newline, ret;
-    struct cmd *cmdp;
+    option_t *opt;
+    int oldpriv;
+    char *oldsource;
     char *argv[MAXARGS];
     char args[MAXARGS][MAXWORDLEN];
     char cmd[MAXWORDLEN];
@@ -362,56 +346,64 @@ options_from_file(filename, must_exist, check_prot)
     if ((f = fopen(filename, "r")) == NULL) {
        if (!must_exist && errno == ENOENT)
            return 1;
-       perror(filename);
+       option_error("Can't open options file %s: %m", filename);
        return 0;
     }
     if (check_prot && !readable(fileno(f))) {
-       fprintf(stderr, "%s: access denied\n", filename);
+       option_error("Can't open options file %s: access denied", filename);
        fclose(f);
        return 0;
     }
 
+    oldpriv = privileged_option;
+    privileged_option = priv;
+    oldsource = option_source;
+    option_source = strdup(filename);
+    if (option_source == NULL)
+       option_source = "file";
+    ret = 0;
     while (getword(f, cmd, &newline, filename)) {
        /*
         * First see if it's a command.
         */
-       for (cmdp = cmds; cmdp->cmd_name; cmdp++)
-           if (!strcmp(cmd, cmdp->cmd_name))
-               break;
-
-       if (cmdp->cmd_name != NULL) {
-           for (i = 0; i < cmdp->num_args; ++i) {
+       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)) {
-                   fprintf(stderr,
-                           "In file %s: too few parameters for command %s\n",
-                           filename, cmd);
-                   fclose(f);
-                   return 0;
+                   option_error(
+                       "In file %s: too few parameters for option '%s'",
+                       filename, cmd);
+                   goto err;
                }
                argv[i] = args[i];
            }
-           if (!(*cmdp->cmd_func)(argv)) {
-               fclose(f);
-               return 0;
-           }
+           current_option = cmd;
+           if (!process_option(opt, argv))
+               goto err;
+           continue;
+       }
 
-       } else {
-           /*
-            * Maybe a tty name, speed or IP address?
-            */
-           if ((ret = setdevname(cmd)) == 0
-               && (ret = setspeed(cmd)) == 0
-               && (ret = setipaddr(cmd)) == 0) {
-               fprintf(stderr, "In file %s: unrecognized command %s\n",
-                       filename, cmd);
-               fclose(f);
-               return 0;
-           }
-           if (ret < 0)        /* error */
-               return 0;
+       /*
+        * Maybe a tty name, speed or IP address?
+        */
+       if ((i = setdevname(cmd, 0)) == 0
+           && (i = setspeed(cmd)) == 0
+           && (i = setipaddr(cmd)) == 0) {
+           option_error("In file %s: unrecognized option '%s'",
+                        filename, cmd);
+           goto err;
        }
+       if (i < 0)              /* error */
+           goto err;
     }
-    return 1;
+    ret = 1;
+
+err:
+    fclose(f);
+    privileged_option = oldpriv;
+    option_source = oldsource;
+    return ret;
 }
 
 /*
@@ -435,7 +427,7 @@ options_from_user()
     strcpy(path, user);
     strcat(path, "/");
     strcat(path, file);
-    ret = options_from_file(path, 0, 1);
+    ret = options_from_file(path, 0, 1, privileged);
     free(path);
     return ret;
 }
@@ -447,26 +439,249 @@ options_from_user()
 int
 options_for_tty()
 {
-    char *dev, *path;
+    char *dev, *path, *p;
     int ret;
 
-    dev = strrchr(devnam, '/');
-    if (dev == NULL)
-       dev = devnam;
-    else
-       ++dev;
+    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);
     if (path == NULL)
        novm("tty init file name");
     strcpy(path, _PATH_TTYOPT);
-    strcat(path, dev);
-    ret = options_from_file(path, 0, 0);
+    /* 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;
+    ret = options_from_file(path, 0, 0, 1);
     free(path);
     return ret;
 }
 
+/*
+ * find_option - scan the option lists for the various protocols
+ * looking for an entry with the given name.
+ * This could be optimized by using a hash table.
+ */
+static option_t *
+find_option(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;
+}
+
+/*
+ * process_option - process one new-style option.
+ */
+static int
+process_option(opt, argv)
+    option_t *opt;
+    char **argv;
+{
+    u_int32_t v;
+    int iv, a;
+    char *sv;
+    int (*parser) __P((char **));
+
+    if ((opt->flags & OPT_PRIV) && !privileged_option) {
+       option_error("using the %s option requires root privilege", opt->name);
+       return 0;
+    }
+    if ((opt->flags & OPT_ENABLE) && *(bool *)(opt->addr2) == 0) {
+       option_error("%s option is disabled", opt->name);
+       return 0;
+    }
+
+    switch (opt->type) {
+    case o_bool:
+       v = opt->flags & OPT_VALUE;
+       *(bool *)(opt->addr) = v;
+       if (opt->addr2 && (opt->flags & OPT_A2COPY))
+           *(bool *)(opt->addr2) = v;
+       break;
+
+    case o_int:
+       iv = 0;
+       if ((opt->flags & OPT_NOARG) == 0) {
+           if (!int_option(*argv, &iv))
+               return 0;
+           if ((((opt->flags & OPT_LLIMIT) && iv < opt->lower_limit)
+                || ((opt->flags & OPT_ULIMIT) && iv > opt->upper_limit))
+               && !((opt->flags & OPT_ZEROOK && iv == 0))) {
+               char *zok = (opt->flags & OPT_ZEROOK)? " zero or": "";
+               switch (opt->flags & OPT_LIMITS) {
+               case OPT_LLIMIT:
+                   option_error("%s value must be%s >= %d",
+                                opt->name, zok, opt->lower_limit);
+                   break;
+               case OPT_ULIMIT:
+                   option_error("%s value must be%s <= %d",
+                                opt->name, zok, opt->upper_limit);
+                   break;
+               case OPT_LIMITS:
+                   option_error("%s value must be%s between %d and %d",
+                               opt->name, opt->lower_limit, opt->upper_limit);
+                   break;
+               }
+               return 0;
+           }
+       }
+       a = opt->flags & OPT_VALUE;
+       if (a >= 128)
+           a -= 256;           /* sign extend */
+       iv += a;
+       if (opt->flags & OPT_INC)
+           iv += *(int *)(opt->addr);
+       if ((opt->flags & OPT_NOINCR) && !privileged_option) {
+           int oldv = *(int *)(opt->addr);
+           if ((opt->flags & OPT_ZEROINF) ?
+               (oldv != 0 && (iv == 0 || iv > oldv)) : (iv > oldv)) {
+               option_error("%s value cannot be increased", opt->name);
+               return 0;
+           }
+       }
+       *(int *)(opt->addr) = iv;
+       if (opt->addr2 && (opt->flags & OPT_A2COPY))
+           *(int *)(opt->addr2) = iv;
+       break;
+
+    case o_uint32:
+       if (opt->flags & OPT_NOARG) {
+           v = opt->flags & OPT_VALUE;
+       } else if (!number_option(*argv, &v, 16))
+           return 0;
+       if (opt->flags & OPT_OR)
+           v |= *(u_int32_t *)(opt->addr);
+       *(u_int32_t *)(opt->addr) = v;
+       if (opt->addr2 && (opt->flags & OPT_A2COPY))
+           *(u_int32_t *)(opt->addr2) = v;
+       break;
+
+    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);
+       } else {
+           sv = strdup(*argv);
+           if (sv == NULL)
+               novm("option argument");
+           *(char **)(opt->addr) = sv;
+       }
+       break;
+
+    case o_special_noarg:
+    case o_special:
+       parser = (int (*) __P((char **))) opt->addr;
+       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;
+    }
+
+    return 1;
+}
+
+/*
+ * n_arguments - tell how many arguments an option takes
+ */
+static int
+n_arguments(option_t *opt)
+{
+    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.
+ */
+void
+usage()
+{
+    if (phase == PHASE_INITIALIZE)
+       fprintf(stderr, usage_string, VERSION, PATCHLEVEL, IMPLEMENTATION,
+               progname);
+}
+
+/*
+ * showhelp - print out usage message and exit.
+ */
+static int
+showhelp(argv)
+    char **argv;
+{
+    if (phase == PHASE_INITIALIZE) {
+       usage();
+       exit(0);
+    }
+    return 0;
+}
+
+/*
+ * showversion - print out the version number and exit.
+ */
+static int
+showversion(argv)
+    char **argv;
+{
+    if (phase == PHASE_INITIALIZE) {
+       fprintf(stderr, "pppd version %s patch level %d%s\n",
+               VERSION, PATCHLEVEL, IMPLEMENTATION);
+       exit(0);
+    }
+    return 0;
+}
+
+/*
+ * option_error - print a message about an error in an option.
+ * The message is logged, and also sent to
+ * stderr if phase == PHASE_INITIALIZE.
+ */
+void
+option_error __V((char *fmt, ...))
+{
+    va_list args;
+    char buf[256];
+
+#if __STDC__
+    va_start(args, fmt);
+#else
+    char *fmt;
+    va_start(args);
+    fmt = va_arg(args, char *);
+#endif
+    vfmtmsg(buf, sizeof(buf), fmt, args);
+    va_end(args);
+    if (phase == PHASE_INITIALIZE)
+       fprintf(stderr, "%s: %s\n", progname, buf);
+    syslog(LOG_ERR, "%s", buf);
+}
+
 /*
  * readable - check if a file is readable by the real user.
  */
@@ -501,7 +716,6 @@ readable(fd)
  * Quotes, white-space and \ may be escaped with \.
  * \<newline> is ignored.
  */
-
 int
 getword(f, word, newlinep, filename)
     FILE *f;
@@ -715,7 +929,7 @@ getword(f, word, newlinep, filename)
        if (ferror(f)) {
            if (errno == 0)
                errno = EIO;
-           perror(filename);
+           option_error("Error reading %s: %m", filename);
            die(1);
        }
        /*
@@ -730,8 +944,8 @@ getword(f, word, newlinep, filename)
      * Warn if the word was too long, and append a terminating null.
      */
     if (len >= MAXWORDLEN) {
-       fprintf(stderr, "%s: warning: word in file %s too long (%.20s...)\n",
-               progname, filename, word);
+       option_error("warning: word in file %s too long (%.20s...)",
+                    filename, word);
        len = MAXWORDLEN - 1;
     }
     word[len] = 0;
@@ -745,7 +959,7 @@ getword(f, word, newlinep, filename)
 /*
  * number_option - parse an unsigned numeric parameter for an option.
  */
-static int
+int
 number_option(str, valp, base)
     char *str;
     u_int32_t *valp;
@@ -755,7 +969,8 @@ number_option(str, valp, base)
 
     *valp = strtoul(str, &ptr, base);
     if (ptr == str) {
-       fprintf(stderr, "%s: invalid number: %s\n", progname, str);
+       option_error("invalid numeric parameter '%s' for %s option",
+                    str, current_option);
        return 0;
     }
     return 1;
@@ -767,7 +982,7 @@ number_option(str, valp, base)
  * the base is assumed to be 0, and *valp is not changed
  * if there is an error.
  */
-static int
+int
 int_option(str, valp)
     char *str;
     int *valp;
@@ -782,7 +997,7 @@ int_option(str, valp)
 
 
 /*
- * The following procedures execute commands.
+ * The following procedures parse options.
  */
 
 /*
@@ -792,463 +1007,201 @@ static int
 readfile(argv)
     char **argv;
 {
-    return options_from_file(*argv, 1, 1);
-}
-
-/*
- * setdebug - Set debug (command line argument).
- */
-static int
-setdebug()
-{
-    debug++;
-    return (1);
+    return options_from_file(*argv, 1, 1, privileged_option);
 }
 
 /*
- * setkdebug - Set kernel debugging level.
+ * callfile - take commands from /etc/ppp/peers/<name>.
+ * Name may not contain /../, start with / or ../, or end in /..
  */
 static int
-setkdebug(argv)
+callfile(argv)
     char **argv;
 {
-    return int_option(*argv, &kdebugflag);
-}
+    char *fname, *arg, *p;
+    int l, ok;
+
+    arg = *argv;
+    ok = 1;
+    if (arg[0] == '/' || arg[0] == 0)
+       ok = 0;
+    else {
+       for (p = arg; *p != 0; ) {
+           if (p[0] == '.' && p[1] == '.' && (p[2] == '/' || p[2] == 0)) {
+               ok = 0;
+               break;
+           }
+           while (*p != '/' && *p != 0)
+               ++p;
+           if (*p == '/')
+               ++p;
+       }
+    }
+    if (!ok) {
+       option_error("call option value may not contain .. or start with /");
+       return 0;
+    }
 
-/*
- * noopt - Disable all options.
- */
-static int
-noopt()
-{
-    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);
+    l = strlen(arg) + strlen(_PATH_PEERFILES) + 1;
+    if ((fname = (char *) malloc(l)) == NULL)
+       novm("call file name");
+    strcpy(fname, _PATH_PEERFILES);
+    strcat(fname, arg);
+
+    ok = options_from_file(fname, 1, 1, 1);
+
+    free(fname);
+    return ok;
 }
 
+#ifdef PPP_FILTER
 /*
- * noaccomp - Disable Address/Control field compression negotiation.
+ * setpdebug - Set libpcap debugging level.
  */
 static int
-noaccomp()
+setpdebug(argv)
+    char **argv;
 {
-    lcp_wantoptions[0].neg_accompression = 0;
-    lcp_allowoptions[0].neg_accompression = 0;
-    return (1);
+    return int_option(*argv, &dflag);
 }
 
-
 /*
- * noasyncmap - Disable async map negotiation.
+ * setpassfilter - Set the pass filter for packets
  */
 static int
-noasyncmap()
+setpassfilter(argv)
+    char **argv;
 {
-    lcp_wantoptions[0].neg_asyncmap = 0;
-    lcp_allowoptions[0].neg_asyncmap = 0;
-    return (1);
+    pc.linktype = DLT_PPP;
+    pc.snapshot = PPP_HDRLEN;
+    if (pcap_compile(&pc, &pass_filter, *argv, 1, netmask) == 0)
+       return 1;
+    option_error("error in pass-filter expression: %s\n", pcap_geterr(&pc));
+    return 0;
 }
 
-
 /*
- * noipaddr - Disable IP address negotiation.
+ * setactivefilter - Set the active filter for packets
  */
 static int
-noipaddr()
+setactivefilter(argv)
+    char **argv;
 {
-    ipcp_wantoptions[0].neg_addr = 0;
-    ipcp_allowoptions[0].neg_addr = 0;
-    return (1);
+    pc.linktype = DLT_PPP;
+    pc.snapshot = PPP_HDRLEN;
+    if (pcap_compile(&pc, &active_filter, *argv, 1, netmask) == 0)
+       return 1;
+    option_error("error in active-filter expression: %s\n", pcap_geterr(&pc));
+    return 0;
 }
-
+#endif
 
 /*
- * nomagicnumber - Disable magic number negotiation.
+ * noopt - Disable all options.
  */
 static int
-nomagicnumber()
+noopt(argv)
+    char **argv;
 {
-    lcp_wantoptions[0].neg_magicnumber = 0;
-    lcp_allowoptions[0].neg_magicnumber = 0;
+    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);
 }
 
-
 /*
- * nomru - Disable mru negotiation.
+ * setdomain - Set domain name to append to hostname 
  */
 static int
-nomru()
+setdomain(argv)
+    char **argv;
 {
-    lcp_wantoptions[0].neg_mru = 0;
-    lcp_allowoptions[0].neg_mru = 0;
+    if (!privileged_option) {
+       option_error("using the domain option requires root privilege");
+       return 0;
+    }
+    gethostname(hostname, MAXNAMELEN);
+    if (**argv != 0) {
+       if (**argv != '.')
+           strncat(hostname, ".", MAXNAMELEN - strlen(hostname));
+       strncat(hostname, *argv, MAXNAMELEN - strlen(hostname));
+    }
+    hostname[MAXNAMELEN-1] = 0;
     return (1);
 }
 
 
 /*
- * setmru - Set MRU for negotiation.
+ * setspeed - Set the speed.
  */
 static int
-setmru(argv)
-    char **argv;
+setspeed(arg)
+    char *arg;
 {
-    u_int32_t mru;
+    char *ptr;
+    int spd;
 
-    if (!number_option(*argv, &mru, 0))
+    spd = strtol(arg, &ptr, 0);
+    if (ptr == arg || *ptr != 0 || spd == 0)
        return 0;
-    lcp_wantoptions[0].mru = mru;
-    lcp_wantoptions[0].neg_mru = 1;
-    return (1);
+    inspeed = spd;
+    return 1;
 }
 
 
 /*
- * setmru - Set the largest MTU we'll use.
+ * setdevname - Set the device name.
  */
 static int
-setmtu(argv)
-    char **argv;
+setdevname(cp, quiet)
+    char *cp;
+    int quiet;
 {
-    u_int32_t mtu;
+    struct stat statbuf;
+    char dev[MAXPATHLEN];
 
-    if (!number_option(*argv, &mtu, 0))
+    if (*cp == 0)
        return 0;
-    if (mtu < MINMRU || mtu > MAXMRU) {
-       fprintf(stderr, "mtu option value of %ld is too %s\n", mtu,
-               (mtu < MINMRU? "small": "large"));
-       return 0;
-    }
-    lcp_allowoptions[0].mru = mtu;
-    return (1);
-}
 
+    if (strncmp("/dev/", cp, 5) != 0) {
+       strcpy(dev, "/dev/");
+       strncat(dev, cp, MAXPATHLEN - 5);
+       dev[MAXPATHLEN-1] = 0;
+       cp = dev;
+    }
 
-/*
- * nopcomp - Disable Protocol field compression negotiation.
- */
-static int
-nopcomp()
-{
-    lcp_wantoptions[0].neg_pcompression = 0;
-    lcp_allowoptions[0].neg_pcompression = 0;
-    return (1);
-}
+    /*
+     * 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 (!privileged_option) {
+       if (!quiet)
+           option_error("setting the device name requires root privilege");
+       return -1;
+    }
 
-/*
- * setpassive - Set passive mode (don't give up if we time out sending
- * LCP configure-requests).
- */
-static int
-setpassive()
-{
-    lcp_wantoptions[0].passive = 1;
-    return (1);
+    (void) strncpy(devnam, cp, MAXPATHLEN);
+    devnam[MAXPATHLEN-1] = 0;
+    default_device = FALSE;
+  
+    return 1;
 }
 
 
 /*
- * setsilent - Set silent mode (don't start sending LCP configure-requests
- * until we get one from the peer).
+ * setipaddr - Set the IP address
  */
 static int
-setsilent()
-{
-    lcp_wantoptions[0].silent = 1;
-    return 1;
-}
-
-
-/*
- * nopap - Disable PAP authentication with peer.
- */
-static int
-nopap()
-{
-    lcp_allowoptions[0].neg_upap = 0;
-    return (1);
-}
-
-
-/*
- * reqpap - Require PAP authentication from peer.
- */
-static int
-reqpap()
-{
-    lcp_wantoptions[0].neg_upap = 1;
-    auth_required = 1;
-    return 1;
-}
-
-
-/*
- * setupapfile - specifies UPAP info for authenticating with peer.
- */
-static int
-setupapfile(argv)
-    char **argv;
-{
-    FILE * ufile;
-    int l;
-
-    lcp_allowoptions[0].neg_upap = 1;
-
-    /* open user info file */
-    if ((ufile = fopen(*argv, "r")) == NULL) {
-       fprintf(stderr, "unable to open user login data file %s\n", *argv);
-       return 0;
-    }
-    if (!readable(fileno(ufile))) {
-       fprintf(stderr, "%s: access denied\n", *argv);
-       return 0;
-    }
-    check_access(ufile, *argv);
-
-    /* get username */
-    if (fgets(user, MAXNAMELEN - 1, ufile) == NULL
-       || fgets(passwd, MAXSECRETLEN - 1, ufile) == NULL){
-       fprintf(stderr, "Unable to read user login data file %s.\n", *argv);
-       return 0;
-    }
-    fclose(ufile);
-
-    /* get rid of newlines */
-    l = strlen(user);
-    if (l > 0 && user[l-1] == '\n')
-       user[l-1] = 0;
-    l = strlen(passwd);
-    if (l > 0 && passwd[l-1] == '\n')
-       passwd[l-1] = 0;
-
-    return (1);
-}
-
-
-/*
- * nochap - Disable CHAP authentication with peer.
- */
-static int
-nochap()
-{
-    lcp_allowoptions[0].neg_chap = 0;
-    return (1);
-}
-
-
-/*
- * reqchap - Require CHAP authentication from peer.
- */
-static int
-reqchap()
-{
-    lcp_wantoptions[0].neg_chap = 1;
-    auth_required = 1;
-    return (1);
-}
-
-
-/*
- * setnovj - disable vj compression
- */
-static int
-setnovj()
-{
-    ipcp_wantoptions[0].neg_vj = 0;
-    ipcp_allowoptions[0].neg_vj = 0;
-    return (1);
-}
-
-
-/*
- * setnovjccomp - disable VJ connection-ID compression
- */
-static int
-setnovjccomp()
-{
-    ipcp_wantoptions[0].cflag = 0;
-    ipcp_allowoptions[0].cflag = 0;
-    return 1;
-}
-
-
-/*
- * setvjslots - set maximum number of connection slots for VJ compression
- */
-static int
-setvjslots(argv)
-    char **argv;
-{
-    int value;
-
-    if (!int_option(*argv, &value))
-       return 0;
-    if (value < 2 || value > 16) {
-       fprintf(stderr, "pppd: vj-max-slots value must be between 2 and 16\n");
-       return 0;
-    }
-    ipcp_wantoptions [0].maxslotindex =
-        ipcp_allowoptions[0].maxslotindex = value - 1;
-    return 1;
-}
-
-
-/*
- * setconnector - Set a program to connect to a serial line
- */
-static int
-setconnector(argv)
-    char **argv;
-{
-    connector = strdup(*argv);
-    if (connector == NULL)
-       novm("connector string");
-  
-    return (1);
-}
-
-/*
- * setdisconnector - Set a program to disconnect from the serial line
- */
-static int
-setdisconnector(argv)
-    char **argv;
-{
-    disconnector = strdup(*argv);
-    if (disconnector == NULL)
-       novm("disconnector string");
-  
-    return (1);
-}
-
-
-/*
- * setdomain - Set domain name to append to hostname 
- */
-static int
-setdomain(argv)
-    char **argv;
-{
-    gethostname(hostname, MAXNAMELEN);
-    if (**argv != 0) {
-       if (**argv != '.')
-           strncat(hostname, ".", MAXNAMELEN - strlen(hostname));
-       strncat(hostname, *argv, MAXNAMELEN - strlen(hostname));
-    }
-    hostname[MAXNAMELEN-1] = 0;
-    return (1);
-}
-
-
-/*
- * setasyncmap - add bits to asyncmap (what we request peer to escape).
- */
-static int
-setasyncmap(argv)
-    char **argv;
-{
-    u_int32_t asyncmap;
-
-    if (!number_option(*argv, &asyncmap, 16))
-       return 0;
-    lcp_wantoptions[0].asyncmap |= asyncmap;
-    lcp_wantoptions[0].neg_asyncmap = 1;
-    return(1);
-}
-
-
-/*
- * setescape - add chars to the set we escape on transmission.
- */
-static int
-setescape(argv)
-    char **argv;
-{
-    int n, ret;
-    char *p, *endp;
-
-    p = *argv;
-    ret = 1;
-    while (*p) {
-       n = strtol(p, &endp, 16);
-       if (p == endp) {
-           fprintf(stderr, "%s: invalid hex number: %s\n", progname, p);
-           return 0;
-       }
-       p = endp;
-       if (n < 0 || 0x20 <= n && n <= 0x3F || n == 0x5E || n > 0xFF) {
-           fprintf(stderr, "%s: can't escape character 0x%x\n", progname, n);
-           ret = 0;
-       } else
-           xmit_accm[0][n >> 5] |= 1 << (n & 0x1F);
-       while (*p == ',' || *p == ' ')
-           ++p;
-    }
-    return ret;
-}
-
-
-/*
- * 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.
- */
-int
-setdevname(cp)
-    char *cp;
-{
-    struct stat statbuf;
-    char *tty, *ttyname();
-    char dev[MAXPATHLEN];
-  
-    if (strncmp("/dev/", cp, 5) != 0) {
-       strcpy(dev, "/dev/");
-       strncat(dev, cp, MAXPATHLEN - 5);
-       dev[MAXPATHLEN-1] = 0;
-       cp = dev;
-    }
-
-    /*
-     * Check if there is a device by this name.
-     */
-    if (stat(cp, &statbuf) < 0) {
-       if (errno == ENOENT)
-           return 0;
-       syslog(LOG_ERR, cp);
-       return -1;
-    }
-  
-    (void) strncpy(devnam, cp, MAXPATHLEN);
-    devnam[MAXPATHLEN-1] = 0;
-    default_device = FALSE;
-  
-    return 1;
-}
-
-
-/*
- * setipaddr - Set the IP address
- */
-int
 setipaddr(arg)
     char *arg;
 {
@@ -1270,18 +1223,14 @@ setipaddr(arg)
        *colon = '\0';
        if ((local = inet_addr(arg)) == -1) {
            if ((hp = gethostbyname(arg)) == NULL) {
-               fprintf(stderr, "unknown host: %s\n", arg);
+               option_error("unknown host: %s", arg);
                return -1;
            } else {
                local = *(u_int32_t *)hp->h_addr;
-               if (our_name[0] == 0) {
-                   strncpy(our_name, arg, MAXNAMELEN);
-                   our_name[MAXNAMELEN-1] = 0;
-               }
            }
        }
        if (bad_ip_adrs(local)) {
-           fprintf(stderr, "bad local IP address %s\n", ip_ntoa(local));
+           option_error("bad local IP address %s", ip_ntoa(local));
            return -1;
        }
        if (local != 0)
@@ -1295,7 +1244,7 @@ setipaddr(arg)
     if (*++colon != '\0') {
        if ((remote = inet_addr(colon)) == -1) {
            if ((hp = gethostbyname(colon)) == NULL) {
-               fprintf(stderr, "unknown host: %s\n", colon);
+               option_error("unknown host: %s", colon);
                return -1;
            } else {
                remote = *(u_int32_t *)hp->h_addr;
@@ -1306,7 +1255,7 @@ setipaddr(arg)
            }
        }
        if (bad_ip_adrs(remote)) {
-           fprintf(stderr, "bad remote IP address %s\n", ip_ntoa(remote));
+           option_error("bad remote IP address %s", ip_ntoa(remote));
            return -1;
        }
        if (remote != 0)
@@ -1318,79 +1267,46 @@ setipaddr(arg)
 
 
 /*
- * setnoipdflt - disable setipdefault()
- */
-static int
-setnoipdflt()
-{
-    disable_defaultip = 1;
-    return 1;
-}
-
-
-/*
- * setipcpaccl - accept peer's idea of our address
- */
-static int
-setipcpaccl()
-{
-    ipcp_wantoptions[0].accept_local = 1;
-    return 1;
-}
-
-
-/*
- * setipcpaccr - accept peer's idea of its address
+ * setnetmask - set the netmask to be used on the interface.
  */
 static int
-setipcpaccr()
-{
-    ipcp_wantoptions[0].accept_remote = 1;
-    return 1;
-}
-
-
-/*
- * setipdefault - default our local IP address based on our hostname.
- */
-void
-setipdefault()
+setnetmask(argv)
+    char **argv;
 {
-    struct hostent *hp;
-    u_int32_t local;
-    ipcp_options *wo = &ipcp_wantoptions[0];
-
-    /*
-     * If local IP address already given, don't bother.
-     */
-    if (wo->ouraddr != 0 || disable_defaultip)
-       return;
+    u_int32_t mask, b;
+    int n, ok;
+    char *p, *endp;
 
     /*
-     * Look up our hostname (possibly with domain name appended)
-     * and take the first IP address as our local IP address.
-     * If there isn't an IP address for our hostname, too bad.
+     * 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.
      */
-    wo->accept_local = 1;      /* don't insist on this default value */
-    if ((hp = gethostbyname(hostname)) == NULL)
-       return;
-    local = *(u_int32_t *)hp->h_addr;
-    if (local != 0 && !bad_ip_adrs(local))
-       wo->ouraddr = local;
-}
-
+    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;
+    }
 
-/*
- * setnetmask - set the netmask to be used on the interface.
- */
-static int
-setnetmask(argv)
-    char **argv;
-{
-    u_int32_t mask;
+    mask = htonl(mask);
 
-    if ((mask = inet_addr(*argv)) == -1 || (netmask & ~mask) != 0) {
-       fprintf(stderr, "Invalid netmask %s\n", *argv);
+    if (*p != 0 || (netmask & ~mask) != 0) {
+       option_error("invalid netmask value '%s'", *argv);
        return 0;
     }
 
@@ -1399,314 +1315,12 @@ setnetmask(argv)
 }
 
 static int
-setcrtscts()
-{
-    crtscts = 1;
-    return (1);
-}
-
-static int
-setnocrtscts()
-{
-    crtscts = -1;
-    return (1);
-}
-
-static int
-setxonxoff()
+setxonxoff(argv)
+    char **argv;
 {
     lcp_wantoptions[0].asyncmap |= 0x000A0000; /* escape ^S and ^Q */
     lcp_wantoptions[0].neg_asyncmap = 1;
 
-    crtscts = 2;
-    return (1);
-}
-
-static int
-setnodetach()
-{
-    nodetach = 1;
+    crtscts = -2;
     return (1);
 }
-
-static int
-setmodem()
-{
-    modem = 1;
-    return 1;
-}
-
-static int
-setlocal()
-{
-    modem = 0;
-    return 1;
-}
-
-static int
-setlock()
-{
-    lockflag = 1;
-    return 1;
-}
-
-static int
-setusehostname()
-{
-    usehostname = 1;
-    return 1;
-}
-
-static int
-setname(argv)
-    char **argv;
-{
-    if (our_name[0] == 0) {
-       strncpy(our_name, argv[0], MAXNAMELEN);
-       our_name[MAXNAMELEN-1] = 0;
-    }
-    return 1;
-}
-
-static int
-setuser(argv)
-    char **argv;
-{
-    strncpy(user, argv[0], MAXNAMELEN);
-    user[MAXNAMELEN-1] = 0;
-    return 1;
-}
-
-static int
-setremote(argv)
-    char **argv;
-{
-    strncpy(remote_name, argv[0], MAXNAMELEN);
-    remote_name[MAXNAMELEN-1] = 0;
-    return 1;
-}
-
-static int
-setauth()
-{
-    auth_required = 1;
-    return 1;
-}
-
-static int
-setdefaultroute()
-{
-    ipcp_wantoptions[0].default_route = 1;
-    return 1;
-}
-
-static int
-setproxyarp()
-{
-    ipcp_wantoptions[0].proxy_arp = 1;
-    return 1;
-}
-
-static int
-setpersist()
-{
-    persist = 1;
-    return 1;
-}
-
-static int
-setdologin()
-{
-    uselogin = 1;
-    return 1;
-}
-
-/*
- * Functions to set the echo interval for modem-less monitors
- */
-
-static int
-setlcpechointv(argv)
-    char **argv;
-{
-    return int_option(*argv, &lcp_echo_interval);
-}
-
-static int
-setlcpechofails(argv)
-    char **argv;
-{
-    return int_option(*argv, &lcp_echo_fails);
-}
-
-/*
- * Functions to set timeouts, max transmits, etc.
- */
-static int
-setlcptimeout(argv)
-    char **argv;
-{
-    return int_option(*argv, &lcp_fsm[0].timeouttime);
-}
-
-static int
-setlcpterm(argv)
-    char **argv;
-{
-    return int_option(*argv, &lcp_fsm[0].maxtermtransmits);
-}
-
-static int
-setlcpconf(argv)
-    char **argv;
-{
-    return int_option(*argv, &lcp_fsm[0].maxconfreqtransmits);
-}
-
-static int
-setlcpfails(argv)
-    char **argv;
-{
-    return int_option(*argv, &lcp_fsm[0].maxnakloops);
-}
-
-static int
-setipcptimeout(argv)
-    char **argv;
-{
-    return int_option(*argv, &ipcp_fsm[0].timeouttime);
-}
-
-static int
-setipcpterm(argv)
-    char **argv;
-{
-    return int_option(*argv, &ipcp_fsm[0].maxtermtransmits);
-}
-
-static int
-setipcpconf(argv)
-    char **argv;
-{
-    return int_option(*argv, &ipcp_fsm[0].maxconfreqtransmits);
-}
-
-static int
-setipcpfails(argv)
-    char **argv;
-{
-    return int_option(*argv, &lcp_fsm[0].maxnakloops);
-}
-
-static int
-setpaptimeout(argv)
-    char **argv;
-{
-    return int_option(*argv, &upap[0].us_timeouttime);
-}
-
-static int
-setpapreqtime(argv)
-    char **argv;
-{
-    return int_option(*argv, &upap[0].us_reqtimeout);
-}
-
-static int
-setpapreqs(argv)
-    char **argv;
-{
-    return int_option(*argv, &upap[0].us_maxtransmits);
-}
-
-static int
-setchaptimeout(argv)
-    char **argv;
-{
-    return int_option(*argv, &chap[0].timeouttime);
-}
-
-static int
-setchapchal(argv)
-    char **argv;
-{
-    return int_option(*argv, &chap[0].max_transmits);
-}
-
-static int
-setchapintv(argv)
-    char **argv;
-{
-    return int_option(*argv, &chap[0].chal_interval);
-}
-
-static int
-setbsdcomp(argv)
-    char **argv;
-{
-    int rbits, abits;
-    char *str, *endp;
-
-    str = *argv;
-    abits = rbits = strtol(str, &endp, 0);
-    if (endp != str && *endp == ',') {
-       str = endp + 1;
-       abits = strtol(str, &endp, 0);
-    }
-    if (*endp != 0 || endp == str) {
-       fprintf(stderr, "%s: invalid argument format for bsdcomp option\n",
-               progname);
-       return 0;
-    }
-    if (rbits != 0 && (rbits < BSD_MIN_BITS || rbits > BSD_MAX_BITS)
-       || abits != 0 && (abits < BSD_MIN_BITS || abits > BSD_MAX_BITS)) {
-       fprintf(stderr, "%s: bsdcomp option values must be 0 or %d .. %d\n",
-               progname, BSD_MIN_BITS, BSD_MAX_BITS);
-       return 0;
-    }
-    if (rbits > 0) {
-       ccp_wantoptions[0].bsd_compress = 1;
-       ccp_wantoptions[0].bsd_bits = rbits;
-    } else
-       ccp_wantoptions[0].bsd_compress = 0;
-    if (abits > 0) {
-       ccp_allowoptions[0].bsd_compress = 1;
-       ccp_allowoptions[0].bsd_bits = abits;
-    } else
-       ccp_allowoptions[0].bsd_compress = 0;
-    return 1;
-}
-
-static int
-setnobsdcomp()
-{
-    ccp_wantoptions[0].bsd_compress = 0;
-    ccp_allowoptions[0].bsd_compress = 0;
-    return 1;
-}
-
-static int
-setipparam(argv)
-    char **argv;
-{
-    ipparam = strdup(*argv);
-    if (ipparam == NULL)
-       novm("ipparam string");
-
-    return 1;
-}
-
-static int
-setpapcrypt()
-{
-    cryptpap = 1;
-    return 1;
-}
-
-#ifdef _linux_
-static int setidle (argv)
-    char **argv;
-{
-    return int_option(*argv, &idle_time_limit);
-}
-#endif