]> git.ozlabs.org Git - ppp.git/blobdiff - pppd/options.c
don't need to be group kmem any more
[ppp.git] / pppd / options.c
index 622ca31ebd5b5abaf73c77d20f273228fc2b16d2..6c146a04c28ecb57cc47218296e60f5ae6f4fb06 100644 (file)
  */
 
 #ifndef lint
-static char rcsid[] = "$Id: options.c,v 1.5 1994/04/18 04:07:48 paulus Exp $";
+static char rcsid[] = "$Id: options.c,v 1.15 1994/09/16 02:34:52 paulus Exp $";
 #endif
 
 #include <stdio.h>
 #include <errno.h>
 #include <unistd.h>
+#include <limits.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 "ppp.h"
 #include "pppd.h"
@@ -41,6 +44,7 @@ static char rcsid[] = "$Id: options.c,v 1.5 1994/04/18 04:07:48 paulus Exp $";
 #include "ipcp.h"
 #include "upap.h"
 #include "chap.h"
+#include "ccp.h"
 
 #define FALSE  0
 #define TRUE   1
@@ -49,6 +53,39 @@ static char rcsid[] = "$Id: options.c,v 1.5 1994/04/18 04:07:48 paulus Exp $";
 char *strdup __ARGS((char *));
 #endif
 
+#ifndef GIDSET_TYPE
+#define GIDSET_TYPE    gid_t
+#endif
+
+/*
+ * Option variables and default values.
+ */
+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 */
+int    inspeed = 0;            /* Input/Output speed requested */
+uint32 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 */
+char   *connector = NULL;      /* Script to establish physical link */
+char   *disconnector = NULL;   /* Script to disestablish physical link */
+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 */
+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 */
+
 /*
  * Prototypes
  */
@@ -58,6 +95,8 @@ static int setpassive __ARGS((void));
 static int setsilent __ARGS((void));
 static int noopt __ARGS((void));
 static int setnovj __ARGS((void));
+static int setnovjccomp __ARGS((void));
+static int setvjslots __ARGS((char **));
 static int reqpap __ARGS((void));
 static int nopap __ARGS((void));
 static int setupapfile __ARGS((char **));
@@ -79,6 +118,8 @@ static int setdisconnector __ARGS((char **));
 static int setdomain __ARGS((char **));
 static int setnetmask __ARGS((char **));
 static int setcrtscts __ARGS((void));
+static int setnocrtscts __ARGS((void));
+static int setxonxoff __ARGS((void));
 static int setnodetach __ARGS((void));
 static int setmodem __ARGS((void));
 static int setlocal __ARGS((void));
@@ -109,37 +150,13 @@ static int setchapchal __ARGS((char **));
 static int setchapintv __ARGS((char **));
 static int setipcpaccl __ARGS((void));
 static int setipcpaccr __ARGS((void));
+static int setlcpechointv __ARGS((char **));
+static int setlcpechofails __ARGS((char **));
+static int setbsdcomp __ARGS((char **));
+static int setnobsdcomp __ARGS((void));
 
 static int number_option __ARGS((char *, long *, int));
-
-
-/*
- * Option variables
- */
-extern char *progname;
-extern int debug;
-extern int kdebugflag;
-extern int modem;
-extern int lockflag;
-extern int crtscts;
-extern int nodetach;
-extern char *connector;
-extern char *disconnector;
-extern int inspeed;
-extern char devname[];
-extern int default_device;
-extern u_long netmask;
-extern int detach;
-extern char user[];
-extern char passwd[];
-extern int auth_required;
-extern int proxyarp;
-extern int persist;
-extern int uselogin;
-extern char our_name[];
-extern char remote_name[];
-int usehostname;
-int disable_defaultip;
+static int readable __ARGS((int fd));
 
 /*
  * Valid arguments.
@@ -149,71 +166,83 @@ static struct cmd {
     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 */
-    "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 */
-    "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-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 timeout for UPAP */
-    "pap-max-authreq", 1, setpapreqs,  /* Set max #xmits for auth-reqs */
-    "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 */
-    NULL
+    {"-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 timeout for UPAP */
+    {"pap-max-authreq", 1, setpapreqs}, /* Set max #xmits for auth-reqs */
+    {"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 */
+    {NULL, 0, NULL}
 };
 
 
+#ifndef IMPLEMENTATION
+#define IMPLEMENTATION ""
+#endif
+
 static char *usage_string = "\
-pppd version %s patch level %d\n\
+pppd version %s patch level %d%s\n\
 Usage: %s [ arguments ], where arguments are:\n\
        <device>        Communicate over the named device\n\
        <speed>         Set the baud rate to <speed>\n\
@@ -243,6 +272,7 @@ parse_args(argc, argv)
 {
     char *arg, *val;
     struct cmd *cmdp;
+    int ret;
 
     while (argc > 0) {
        arg = *argv++;
@@ -269,11 +299,15 @@ parse_args(argc, argv)
            /*
             * Maybe a tty name, speed or IP address?
             */
-           if (!setdevname(arg) && !setspeed(arg) && !setipaddr(arg)) {
+           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;
        }
     }
     return 1;
@@ -284,7 +318,8 @@ parse_args(argc, argv)
  */
 usage()
 {
-    fprintf(stderr, usage_string, VERSION, PATCHLEVEL, progname);
+    fprintf(stderr, usage_string, VERSION, PATCHLEVEL, IMPLEMENTATION,
+           progname);
 }
 
 /*
@@ -292,12 +327,13 @@ usage()
  * and interpret them.
  */
 int
-options_from_file(filename, must_exist)
+options_from_file(filename, must_exist, check_prot)
     char *filename;
     int must_exist;
+    int check_prot;
 {
     FILE *f;
-    int i, newline;
+    int i, newline, ret;
     struct cmd *cmdp;
     char *argv[MAXARGS];
     char args[MAXARGS][MAXWORDLEN];
@@ -307,8 +343,14 @@ options_from_file(filename, must_exist)
        if (!must_exist && errno == ENOENT)
            return 1;
        perror(filename);
-       exit(1);
+       return 0;
     }
+    if (check_prot && !readable(fileno(f))) {
+       fprintf(stderr, "%s: access denied\n", filename);
+       fclose(f);
+       return 0;
+    }
+
     while (getword(f, cmd, &newline, filename)) {
        /*
         * First see if it's a command.
@@ -337,12 +379,16 @@ options_from_file(filename, must_exist)
            /*
             * Maybe a tty name, speed or IP address?
             */
-           if (!setdevname(cmd) && !setspeed(cmd) && !setipaddr(cmd)) {
+           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;
        }
     }
     return 1;
@@ -357,20 +403,78 @@ options_from_user()
 {
     char *user, *path, *file;
     int ret;
+    struct passwd *pw;
 
-    if ((user = getenv("HOME")) == NULL)
-       return;
-    file = "/.ppprc";
-    path = malloc(strlen(user) + strlen(file) + 1);
+    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);
     if (path == NULL)
        novm("init file name");
     strcpy(path, user);
+    strcat(path, "/");
     strcat(path, file);
-    ret = options_from_file(path, 0);
+    ret = options_from_file(path, 0, 1);
     free(path);
     return ret;
 }
 
+/*
+ * options_for_tty - See if an options file exists for the serial
+ * device, and if so, interpret options from it.
+ */
+int
+options_for_tty()
+{
+    char *dev, *path;
+    int ret;
+
+    dev = strrchr(devnam, '/');
+    if (dev == NULL)
+       dev = devnam;
+    else
+       ++dev;
+    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);
+    free(path);
+    return ret;
+}
+
+/*
+ * readable - check if a file is readable by the real user.
+ */
+static int
+readable(fd)
+    int fd;
+{
+    uid_t uid;
+    int ngroups, i;
+    struct stat sbuf;
+    GIDSET_TYPE groups[NGROUPS_MAX];
+
+    uid = getuid();
+    if (uid == 0)
+       return 1;
+    if (fstat(fd, &sbuf) != 0)
+       return 0;
+    if (sbuf.st_uid == uid)
+       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;
+}
+
 /*
  * Read a word from a file.
  * Words are delimited by white-space or by quotes (").
@@ -527,7 +631,7 @@ static int
 readfile(argv)
     char **argv;
 {
-    return options_from_file(*argv, 1);
+    return options_from_file(*argv, 1, 1);
 }
 
 /*
@@ -734,7 +838,11 @@ setupapfile(argv)
     /* open user info file */
     if ((ufile = fopen(*argv, "r")) == NULL) {
        fprintf(stderr, "unable to open user login data file %s\n", *argv);
-       exit(1);
+       return 0;
+    }
+    if (!readable(fileno(ufile))) {
+       fprintf(stderr, "%s: access denied\n", *argv);
+       return 0;
     }
     check_access(ufile, *argv);
 
@@ -742,7 +850,7 @@ setupapfile(argv)
     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);
-       exit(2);
+       return 0;
     }
     fclose(ufile);
 
@@ -782,7 +890,7 @@ reqchap()
 
 
 /*
- * setnovj - diable vj compression
+ * setnovj - disable vj compression
  */
 static int
 setnovj()
@@ -792,6 +900,39 @@ setnovj()
     return (1);
 }
 
+
+/*
+ * setnovjccomp - disable VJ connection-ID compression
+ */
+static int
+setnovjccomp()
+{
+    ipcp_wantoptions[0].cflag = 0;
+    ipcp_allowoptions[0].cflag = 0;
+}
+
+
+/*
+ * 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
  */
@@ -923,16 +1064,16 @@ setdevname(cp)
      */
     if (stat(cp, &statbuf) < 0) {
        if (errno == ENOENT)
-           return (0);
+           return 0;
        syslog(LOG_ERR, cp);
-       exit(1);
+       return -1;
     }
   
-    (void) strncpy(devname, cp, MAXPATHLEN);
-    devname[MAXPATHLEN-1] = 0;
+    (void) strncpy(devnam, cp, MAXPATHLEN);
+    devnam[MAXPATHLEN-1] = 0;
     default_device = FALSE;
   
-    return (1);
+    return 1;
 }
 
 
@@ -945,14 +1086,14 @@ setipaddr(arg)
 {
     struct hostent *hp;
     char *colon, *index();
-    u_long local, remote;
+    uint32 local, remote;
     ipcp_options *wo = &ipcp_wantoptions[0];
   
     /*
      * IP address pair separated by ":".
      */
     if ((colon = index(arg, ':')) == NULL)
-       return (0);
+       return 0;
   
     /*
      * If colon first character, then no local addr.
@@ -961,8 +1102,8 @@ setipaddr(arg)
        *colon = '\0';
        if ((local = inet_addr(arg)) == -1) {
            if ((hp = gethostbyname(arg)) == NULL) {
-               fprintf(stderr, "unknown host: %s", arg);
-               local = 0;
+               fprintf(stderr, "unknown host: %s\n", arg);
+               return -1;
            } else {
                local = *(long *)hp->h_addr;
                if (our_name[0] == 0) {
@@ -971,6 +1112,10 @@ setipaddr(arg)
                }
            }
        }
+       if (bad_ip_adrs(local)) {
+           fprintf(stderr, "bad local IP address %s\n", ip_ntoa(local));
+           return -1;
+       }
        if (local != 0)
            wo->ouraddr = local;
        *colon = ':';
@@ -982,8 +1127,8 @@ setipaddr(arg)
     if (*++colon != '\0') {
        if ((remote = inet_addr(colon)) == -1) {
            if ((hp = gethostbyname(colon)) == NULL) {
-               fprintf(stderr, "unknown host: %s", colon);
-               remote = 0;
+               fprintf(stderr, "unknown host: %s\n", colon);
+               return -1;
            } else {
                remote = *(long *)hp->h_addr;
                if (remote_name[0] == 0) {
@@ -992,11 +1137,15 @@ setipaddr(arg)
                }
            }
        }
+       if (bad_ip_adrs(remote)) {
+           fprintf(stderr, "bad remote IP address %s\n", ip_ntoa(remote));
+           return -1;
+       }
        if (remote != 0)
            wo->hisaddr = remote;
     }
 
-    return (1);
+    return 1;
 }
 
 
@@ -1040,7 +1189,7 @@ void
 setipdefault()
 {
     struct hostent *hp;
-    u_long local;
+    uint32 local;
     ipcp_options *wo = &ipcp_wantoptions[0];
 
     /*
@@ -1058,7 +1207,7 @@ setipdefault()
     if ((hp = gethostbyname(hostname)) == NULL)
        return;
     local = *(long *)hp->h_addr;
-    if (local != 0)
+    if (local != 0 && !bad_ip_adrs(local))
        wo->ouraddr = local;
 }
 
@@ -1070,11 +1219,11 @@ static int
 setnetmask(argv)
     char **argv;
 {
-    u_long mask;
-       
-    if ((mask = inet_addr(*argv)) == -1) {
+    uint32 mask;
+
+    if ((mask = inet_addr(*argv)) == -1 || (netmask & ~mask) != 0) {
        fprintf(stderr, "Invalid netmask %s\n", *argv);
-       exit(1);
+       return 0;
     }
 
     netmask = mask;
@@ -1086,9 +1235,9 @@ setnetmask(argv)
  * been set. 
  */
 /* ARGSUSED */
-u_long
+uint32
 GetMask(addr)
-    u_long addr;
+    uint32 addr;
 {
     return(netmask);
 }
@@ -1101,6 +1250,20 @@ setcrtscts()
     return (1);
 }
 
+static int
+setnocrtscts()
+{
+    crtscts = -1;
+    return (1);
+}
+
+static int
+setxonxoff()
+{
+    crtscts = 2;
+    return (1);
+}
+
 static int
 setnodetach()
 {
@@ -1200,6 +1363,24 @@ setdologin()
     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.
  */
@@ -1207,77 +1388,119 @@ static int
 setlcptimeout(argv)
     char **argv;
 {
-    return int_option(*argv, &lcp_fsm[0].timeouttime, 0);
+    return int_option(*argv, &lcp_fsm[0].timeouttime);
 }
 
 static int setlcpterm(argv)
     char **argv;
 {
-    return int_option(*argv, &lcp_fsm[0].maxtermtransmits, 0);
+    return int_option(*argv, &lcp_fsm[0].maxtermtransmits);
 }
 
 static int setlcpconf(argv)
     char **argv;
 {
-    return int_option(*argv, &lcp_fsm[0].maxconfreqtransmits, 0);
+    return int_option(*argv, &lcp_fsm[0].maxconfreqtransmits);
 }
 
 static int setlcpfails(argv)
     char **argv;
 {
-    return int_option(*argv, &lcp_fsm[0].maxnakloops, 0);
+    return int_option(*argv, &lcp_fsm[0].maxnakloops);
 }
 
 static int setipcptimeout(argv)
     char **argv;
 {
-    return int_option(*argv, &ipcp_fsm[0].timeouttime, 0);
+    return int_option(*argv, &ipcp_fsm[0].timeouttime);
 }
 
 static int setipcpterm(argv)
     char **argv;
 {
-    return int_option(*argv, &ipcp_fsm[0].maxtermtransmits, 0);
+    return int_option(*argv, &ipcp_fsm[0].maxtermtransmits);
 }
 
 static int setipcpconf(argv)
     char **argv;
 {
-    return int_option(*argv, &ipcp_fsm[0].maxconfreqtransmits, 0);
+    return int_option(*argv, &ipcp_fsm[0].maxconfreqtransmits);
 }
 
 static int setipcpfails(argv)
     char **argv;
 {
-    return int_option(*argv, &lcp_fsm[0].maxnakloops, 0);
+    return int_option(*argv, &lcp_fsm[0].maxnakloops);
 }
 
 static int setpaptimeout(argv)
     char **argv;
 {
-    return int_option(*argv, &upap[0].us_timeouttime, 0);
+    return int_option(*argv, &upap[0].us_timeouttime);
 }
 
 static int setpapreqs(argv)
     char **argv;
 {
-    return int_option(*argv, &upap[0].us_maxtransmits, 0);
+    return int_option(*argv, &upap[0].us_maxtransmits);
 }
 
 static int setchaptimeout(argv)
     char **argv;
 {
-    return int_option(*argv, &chap[0].timeouttime, 0);
+    return int_option(*argv, &chap[0].timeouttime);
 }
 
 static int setchapchal(argv)
     char **argv;
 {
-    return int_option(*argv, &chap[0].max_transmits, 0);
+    return int_option(*argv, &chap[0].max_transmits);
 }
 
 static int setchapintv(argv)
     char **argv;
 {
-    return int_option(*argv, &chap[0].chal_interval, 0);
+    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 < MIN_BSD_BITS || rbits > MAX_BSD_BITS)
+       || abits != 0 && (abits < MIN_BSD_BITS || abits > MAX_BSD_BITS)) {
+       fprintf(stderr, "%s: bsdcomp option values must be 0 or %d .. %d\n",
+               progname, MIN_BSD_BITS, MAX_BSD_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;
 }