]> git.ozlabs.org Git - ppp.git/blobdiff - pppd/main.c
added maxconnect option; extend protocol table; optional IPX support;
[ppp.git] / pppd / main.c
index ea5784a2e544a595271fb8bfc441989c61fa9e7b..159ecfb983985e8e9a56e3f23d207afecf5026ce 100644 (file)
  */
 
 #ifndef lint
-static char rcsid[] = "$Id: main.c,v 1.16 1994/08/25 06:55:21 paulus Exp $";
+static char rcsid[] = "$Id: main.c,v 1.29 1995/12/18 03:50:17 paulus Exp $";
 #endif
 
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
 #include <signal.h>
 #include <errno.h>
@@ -39,7 +40,6 @@ static char rcsid[] = "$Id: main.c,v 1.16 1994/08/25 06:55:21 paulus Exp $";
 #include <sys/socket.h>
 #include <net/if.h>
 
-#include "ppp.h"
 #include "pppd.h"
 #include "magic.h"
 #include "fsm.h"
@@ -51,6 +51,10 @@ static char rcsid[] = "$Id: main.c,v 1.16 1994/08/25 06:55:21 paulus Exp $";
 #include "pathnames.h"
 #include "patchlevel.h"
 
+#ifdef IPX_CHANGE
+#include "ipxcp.h"
+#endif /* IPX_CHANGE */
+
 /*
  * If REQ_SYSOPTIONS is defined to 1, pppd will not run unless
  * /etc/ppp/options exists.
@@ -59,98 +63,95 @@ static char rcsid[] = "$Id: main.c,v 1.16 1994/08/25 06:55:21 paulus Exp $";
 #define REQ_SYSOPTIONS 1
 #endif
 
-static char *pidpath = _PATH_PIDFILE;
-
 /* interface vars */
 char ifname[IFNAMSIZ];         /* Interface name */
 int ifunit;                    /* Interface unit number */
 
 char *progname;                        /* Name of this program */
 char hostname[MAXNAMELEN];     /* Our hostname */
-static char pidfilename[MAXPATHLEN];
-
+static char pidfilename[MAXPATHLEN];   /* name of pid file */
+static char default_devnam[MAXPATHLEN];        /* name of default device */
 static pid_t   pid;            /* Our pid */
 static pid_t   pgrpid;         /* Process Group ID */
 static uid_t uid;              /* Our real user-id */
 
 int fd = -1;                   /* Device file descriptor */
-int s;                         /* Socket file descriptor */
 
 int phase;                     /* where the link is at */
 int kill_link;
+int open_ccp_flag;
 
-static int initfdflags = -1;   /* Initial file descriptor flags */
-
-u_char outpacket_buf[MTU+DLLHEADERLEN]; /* buffer for outgoing packet */
-static u_char inpacket_buf[MTU+DLLHEADERLEN]; /* buffer for incoming packet */
+u_char outpacket_buf[PPP_MRU+PPP_HDRLEN]; /* buffer for outgoing packet */
+static u_char inpacket_buf[PPP_MRU+PPP_HDRLEN]; /* buffer for incoming packet */
 
 int hungup;                    /* terminal has been hung up */
 static int n_children;         /* # child processes still running */
 
-int baud_rate;
-
-/* prototypes */
-static void hup __ARGS((int));
-static void term __ARGS((int));
-static void chld __ARGS((int));
-static void incdebug __ARGS((int));
-static void nodebug __ARGS((int));
-
-static void get_input __ARGS((void));
-void establish_ppp __ARGS((void));
-void calltimeout __ARGS((void));
-struct timeval *timeleft __ARGS((struct timeval *));
-void reap_kids __ARGS((void));
-void cleanup __ARGS((int, caddr_t));
-void close_fd __ARGS((void));
-void die __ARGS((int));
-void novm __ARGS((char *));
-
-void log_packet __ARGS((u_char *, int, char *));
-void format_packet __ARGS((u_char *, int,
-                          void (*) (void *, char *, ...), void *));
-void pr_log __ARGS((void *, char *, ...));
-
-extern char    *ttyname __ARGS((int));
-extern char    *getlogin __ARGS((void));
+int baud_rate;                 /* Actual bits/second for serial device */
+
+char *no_ppp_msg = "Sorry - this system lacks PPP kernel support\n";
+
+/* Prototypes for procedures local to this file. */
+
+static void cleanup __P((void));
+static void close_fd __P((void));
+static void get_input __P((void));
+static void connect_time_expired __P((caddr_t));
+static void calltimeout __P((void));
+static struct timeval *timeleft __P((struct timeval *));
+static void hup __P((int));
+static void term __P((int));
+static void chld __P((int));
+static void toggle_debug __P((int));
+static void open_ccp __P((int));
+static void bad_signal __P((int));
+static int device_script __P((char *, int, int));
+static void reap_kids __P((void));
+static void pr_log __P((void *, char *, ...));
+
+extern char    *ttyname __P((int));
+extern char    *getlogin __P((void));
+
+#ifdef ultrix
+#undef O_NONBLOCK
+#define        O_NONBLOCK      O_NDELAY
+#endif
 
 /*
  * PPP Data Link Layer "protocol" table.
  * One entry per supported protocol.
+ * The last entry must be NULL.
  */
-static struct protent {
-    u_short protocol;
-    void (*init)();
-    void (*input)();
-    void (*protrej)();
-    int  (*printpkt)();
-    void (*datainput)();
-    char *name;
-} prottbl[] = {
-    { LCP, lcp_init, lcp_input, lcp_protrej, lcp_printpkt, NULL, "LCP" },
-    { IPCP, ipcp_init, ipcp_input, ipcp_protrej, ipcp_printpkt, NULL, "IPCP" },
-    { UPAP, upap_init, upap_input, upap_protrej, upap_printpkt, NULL, "PAP" },
-    { CHAP, ChapInit, ChapInput, ChapProtocolReject, ChapPrintPkt, NULL, "CHAP" },
-    { CCP, ccp_init, ccp_input, ccp_protrej, ccp_printpkt, ccp_datainput, "CCP" },
+struct protent *protocols[] = {
+    &lcp_protent,
+    &pap_protent,
+    &chap_protent,
+    &ipcp_protent,
+    &ccp_protent,
+#ifdef IPX_CHANGE
+    &ipxcp_protent,
+#endif
+    NULL
 };
 
-#define N_PROTO                (sizeof(prottbl) / sizeof(prottbl[0]))
-
 main(argc, argv)
     int argc;
     char *argv[];
 {
-    int mask, i, nonblock;
+    int i, nonblock, fdflags;
     struct sigaction sa;
     struct cmd *cmdp;
     FILE *pidfile;
     char *p;
     struct passwd *pw;
     struct timeval timo;
+    sigset_t mask;
+    struct protent *protp;
 
     p = ttyname(0);
     if (p)
        strcpy(devnam, p);
+    strcpy(default_devnam, devnam);
 
     if (gethostname(hostname, MAXNAMELEN) < 0 ) {
        perror("couldn't get hostname");
@@ -160,18 +161,13 @@ main(argc, argv)
 
     uid = getuid();
 
-    if (!ppp_available()) {
-       fprintf(stderr, "Sorry - PPP is not available on this system\n");
-       exit(1);
-    }
-
     /*
      * Initialize to the standard option set, then parse, in order,
      * the system options file, the user's options file, and the command
      * line arguments.
      */
-    for (i = 0; i < N_PROTO; i++)
-       (*prottbl[i].init)(0);
+    for (i = 0; (protp = protocols[i]) != NULL; ++i)
+        (*protp->init)(0);
   
     progname = *argv;
 
@@ -180,9 +176,25 @@ main(argc, argv)
        !parse_args(argc-1, argv+1) ||
        !options_for_tty())
        die(1);
+
+    if (!ppp_available()) {
+       fprintf(stderr, no_ppp_msg);
+       exit(1);
+    }
+
+#ifdef IPX_CHANGE
+    remove_sys_options();
+#endif /* IPX_CHANGE */
     check_auth_options();
     setipdefault();
 
+    /*
+     * If the user has specified the default device name explicitly,
+     * pretend they hadn't.
+     */
+    if (!default_device && strcmp(devnam, default_devnam) == 0)
+       default_device = 1;
+
     /*
      * Initialize system-dependent stuff and magic number package.
      */
@@ -208,12 +220,6 @@ main(argc, argv)
     }
     syslog(LOG_NOTICE, "pppd %s.%d started by %s, uid %d",
           VERSION, PATCHLEVEL, p, uid);
-
-    /* Get an internet socket for doing socket ioctl's on. */
-    if ((s = socket(AF_INET, SOCK_DGRAM, 0)) < 0) {
-       syslog(LOG_ERR, "Couldn't create IP socket: %m");
-       die(1);
-    }
   
     /*
      * Compute mask of all interesting signals and install signal handlers
@@ -241,8 +247,47 @@ main(argc, argv)
     SIGNAL(SIGTERM, term);             /* Terminate */
     SIGNAL(SIGCHLD, chld);
 
-    signal(SIGUSR1, incdebug);         /* Increment debug flag */
-    signal(SIGUSR2, nodebug);          /* Reset debug flag */
+    SIGNAL(SIGUSR1, toggle_debug);     /* Toggle debug flag */
+    SIGNAL(SIGUSR2, open_ccp);         /* Reopen CCP */
+
+    /*
+     * Install a handler for other signals which would otherwise
+     * cause pppd to exit without cleaning up.
+     */
+    SIGNAL(SIGABRT, bad_signal);
+    SIGNAL(SIGALRM, bad_signal);
+    SIGNAL(SIGFPE, bad_signal);
+    SIGNAL(SIGILL, bad_signal);
+    SIGNAL(SIGPIPE, bad_signal);
+    SIGNAL(SIGQUIT, bad_signal);
+    SIGNAL(SIGSEGV, bad_signal);
+#ifdef SIGBUS
+    SIGNAL(SIGBUS, bad_signal);
+#endif
+#ifdef SIGEMT
+    SIGNAL(SIGEMT, bad_signal);
+#endif
+#ifdef SIGPOLL
+    SIGNAL(SIGPOLL, bad_signal);
+#endif
+#ifdef SIGPROF
+    SIGNAL(SIGPROF, bad_signal);
+#endif
+#ifdef SIGSYS
+    SIGNAL(SIGSYS, bad_signal);
+#endif
+#ifdef SIGTRAP
+    SIGNAL(SIGTRAP, bad_signal);
+#endif
+#ifdef SIGVTALRM
+    SIGNAL(SIGVTALRM, bad_signal);
+#endif
+#ifdef SIGXCPU
+    SIGNAL(SIGXCPU, bad_signal);
+#endif
+#ifdef SIGXFSZ
+    SIGNAL(SIGXFSZ, bad_signal);
+#endif
 
     /*
      * Lock the device if we've been asked to.
@@ -264,13 +309,11 @@ main(argc, argv)
            syslog(LOG_ERR, "Failed to open %s: %m", devnam);
            die(1);
        }
-       if ((initfdflags = fcntl(fd, F_GETFL)) == -1) {
-           syslog(LOG_ERR, "Couldn't get device fd flags: %m");
-           die(1);
-       }
        if (nonblock) {
-           initfdflags &= ~O_NONBLOCK;
-           fcntl(fd, F_SETFL, initfdflags);
+           if ((fdflags = fcntl(fd, F_GETFL)) == -1
+               || fcntl(fd, F_SETFL, fdflags & ~O_NONBLOCK) < 0)
+               syslog(LOG_WARNING,
+                      "Couldn't reset non-blocking mode on device: %m");
        }
        hungup = 0;
        kill_link = 0;
@@ -295,10 +338,10 @@ main(argc, argv)
                die(1);
            }
 
-           syslog(LOG_INFO, "Connected...");
+           syslog(LOG_INFO, "Serial connection established.");
            sleep(1);           /* give it time to set up its terminal */
        }
-  
+
        /* set line speed, flow control, etc.; clear CLOCAL if modem option */
        set_up_tty(fd, 0);
 
@@ -309,7 +352,7 @@ main(argc, argv)
        (void) sprintf(ifname, "ppp%d", ifunit);
 
        /* write pid to file */
-       (void) sprintf(pidfilename, "%s/%s.pid", pidpath, ifname);
+       (void) sprintf(pidfilename, "%s%s.pid", _PATH_VARRUN, ifname);
        if ((pidfile = fopen(pidfilename, "w")) != NULL) {
            fprintf(pidfile, "%d\n", pid);
            (void) fclose(pidfile);
@@ -319,13 +362,12 @@ main(argc, argv)
        }
 
        /*
-        * Set device for non-blocking reads.
+        * Set a timeout to close the connection once the maximum
+        * connect time has expired.
         */
-       if (fcntl(fd, F_SETFL, initfdflags | O_NONBLOCK) == -1) {
-           syslog(LOG_ERR, "Couldn't set device to non-blocking mode: %m");
-           die(1);
-       }
-  
+       if (maxconnect > 0)
+           TIMEOUT(connect_time_expired, 0, maxconnect);
+
        /*
         * Block all signals, start opening the connection, and wait for
         * incoming events (reply, timeout, etc.).
@@ -336,24 +378,34 @@ main(argc, argv)
        for (phase = PHASE_ESTABLISH; phase != PHASE_DEAD; ) {
            wait_input(timeleft(&timo));
            calltimeout();
+           get_input();
            if (kill_link) {
-               lcp_close(0);
+               lcp_close(0, "User request");
+               phase = PHASE_TERMINATE;
                kill_link = 0;
            }
-           get_input();
+           if (open_ccp_flag) {
+               if (phase == PHASE_NETWORK) {
+                   ccp_fsm[0].flags = OPT_RESTART; /* clears OPT_SILENT */
+                   ccp_open(0);
+               }
+               open_ccp_flag = 0;
+           }
            reap_kids();        /* Don't leave dead kids lying around */
        }
 
        /*
-        * Run disconnector script, if requested
+        * Run disconnector script, if requested.
+        * XXX we may not be able to do this if the line has hung up!
         */
+       disestablish_ppp();
        if (disconnector) {
+           set_up_tty(fd, 1);
            if (device_script(disconnector, fd, fd) < 0) {
                syslog(LOG_WARNING, "disconnect script failed");
-               die(1);
+           } else {
+               syslog(LOG_INFO, "Serial link disconnected.");
            }
-
-           syslog(LOG_INFO, "Disconnected...");
        }
 
        close_fd();
@@ -363,10 +415,7 @@ main(argc, argv)
 
     } while (persist);
 
-    if (lockflag && !default_device)
-       unlock();
-
-    exit(0);
+    die(0);
 }
 
 
@@ -379,98 +428,71 @@ get_input()
     int len, i;
     u_char *p;
     u_short protocol;
+    struct protent *protp;
 
-    for (;;) {                 /* Read all available packets */
-       p = inpacket_buf;       /* point to beginning of packet buffer */
+    p = inpacket_buf;  /* point to beginning of packet buffer */
 
-       len = read_packet(inpacket_buf);
-       if (len < 0)
-           return;
-
-       if (len == 0) {
-           MAINDEBUG((LOG_DEBUG, "End of file on fd!"));
-           hungup = 1;
-           lcp_lowerdown(0);   /* serial link is no longer available */
-           phase = PHASE_DEAD;
-           return;
-       }
-
-       if (debug /*&& (debugflags & DBG_INPACKET)*/)
-           log_packet(p, len, "rcvd ");
-
-       if (len < DLLHEADERLEN) {
-           MAINDEBUG((LOG_INFO, "io(): Received short packet."));
-           return;
-       }
-
-       p += 2;                         /* Skip address and control */
-       GETSHORT(protocol, p);
-       len -= DLLHEADERLEN;
+    len = read_packet(inpacket_buf);
+    if (len < 0)
+       return;
 
-       /*
-        * Toss all non-LCP packets unless LCP is OPEN.
-        */
-       if (protocol != LCP && lcp_fsm[0].state != OPENED) {
-           MAINDEBUG((LOG_INFO,
-                      "io(): Received non-LCP packet when LCP not open."));
-           return;
-       }
+    if (len == 0) {
+       syslog(LOG_NOTICE, "Modem hangup");
+       hungup = 1;
+       lcp_lowerdown(0);       /* serial link is no longer available */
+       link_terminated(0);
+       return;
+    }
 
-       /*
-        * Upcall the proper protocol input routine.
-        */
-       for (i = 0; i < sizeof (prottbl) / sizeof (struct protent); i++)
-           if (prottbl[i].protocol == protocol) {
-               (*prottbl[i].input)(0, p, len);
-               break;
-           } else if (protocol == (prottbl[i].protocol & ~0x8000)
-                      && prottbl[i].datainput != NULL) {
-               (*prottbl[i].datainput)(0, p, len);
-               break;
-           }
+    if (debug /*&& (debugflags & DBG_INPACKET)*/)
+       log_packet(p, len, "rcvd ");
 
-       if (i == sizeof (prottbl) / sizeof (struct protent)) {
-           if (debug)
-               syslog(LOG_WARNING, "Unknown protocol (%x) received",
-                      protocol);
-           lcp_sprotrej(0, p - DLLHEADERLEN, len + DLLHEADERLEN);
-       }
+    if (len < PPP_HDRLEN) {
+       MAINDEBUG((LOG_INFO, "io(): Received short packet."));
+       return;
     }
-}
 
+    p += 2;                            /* Skip address and control */
+    GETSHORT(protocol, p);
+    len -= PPP_HDRLEN;
 
-/*
- * demuxprotrej - Demultiplex a Protocol-Reject.
- */
-void
-demuxprotrej(unit, protocol)
-    int unit;
-    u_short protocol;
-{
-    int i;
+    /*
+     * Toss all non-LCP packets unless LCP is OPEN.
+     */
+    if (protocol != PPP_LCP && lcp_fsm[0].state != OPENED) {
+       MAINDEBUG((LOG_INFO,
+                  "io(): Received non-LCP packet when LCP not open."));
+       return;
+    }
 
     /*
-     * Upcall the proper Protocol-Reject routine.
+     * Upcall the proper protocol input routine.
      */
-    for (i = 0; i < sizeof (prottbl) / sizeof (struct protent); i++)
-       if (prottbl[i].protocol == protocol) {
-           (*prottbl[i].protrej)(unit);
+    for (i = 0; (protp = protocols[i]) != NULL; ++i) {
+       if (protp->protocol == protocol && protp->enabled_flag) {
+           (*protp->input)(0, p, len);
            return;
        }
+        if (protocol == (protp->protocol & ~0x8000) && protp->enabled_flag
+           && protp->datainput != NULL) {
+           (*protp->datainput)(0, p, len);
+           return;
+       }
+    }
 
-    syslog(LOG_WARNING,
-          "demuxprotrej: Unrecognized Protocol-Reject for protocol 0x%x",
-          protocol);
+    if (debug)
+       syslog(LOG_WARNING, "Unsupported protocol (0x%x) received", protocol);
+    lcp_sprotrej(0, p - PPP_HDRLEN, len + PPP_HDRLEN);
 }
 
 
 /*
- * quit - Clean up state and exit.
+ * quit - Clean up state and exit (with an error indication).
  */
 void 
 quit()
 {
-    die(0);
+    die(1);
 }
 
 /*
@@ -480,20 +502,33 @@ void
 die(status)
     int status;
 {
-    cleanup(0, NULL);
+    cleanup();
     syslog(LOG_INFO, "Exit.");
     exit(status);
 }
 
+/*
+ * connect_time_expired - log a message and close the connection.
+ */
+static void
+connect_time_expired(arg)
+    caddr_t arg;
+{
+    syslog(LOG_INFO, "Connect time expired");
+
+    phase = PHASE_TERMINATE;
+    lcp_close(0, "Connect time expired");      /* Close connection */
+}
+
 /*
  * cleanup - restore anything which needs to be restored before we exit
  */
 /* ARGSUSED */
-void
-cleanup(status, arg)
-    int status;
-    caddr_t arg;
+static void
+cleanup()
 {
+    sys_cleanup();
+
     if (fd >= 0)
        close_fd();
 
@@ -508,19 +543,15 @@ cleanup(status, arg)
 /*
  * close_fd - restore the terminal device and close it.
  */
-void
+static void
 close_fd()
 {
+    disestablish_ppp();
+
     /* drop dtr to hang up */
     if (modem)
        setdtr(fd, FALSE);
 
-    if (initfdflags != -1 && fcntl(fd, F_SETFL, initfdflags) < 0)
-       syslog(LOG_WARNING, "Couldn't restore device fd flags: %m");
-    initfdflags = -1;
-
-    disestablish_ppp();
-
     restore_tty();
 
     close(fd);
@@ -552,8 +583,8 @@ timeout(func, arg, time)
 {
     struct callout *newp, *p, **pp;
   
-    MAINDEBUG((LOG_DEBUG, "Timeout %x:%x in %d seconds.",
-              (int) func, (int) arg, time));
+    MAINDEBUG((LOG_DEBUG, "Timeout %lx:%lx in %d seconds.",
+              (long) func, (long) arg, time));
   
     /*
      * Allocate timeout.
@@ -572,9 +603,9 @@ timeout(func, arg, time)
      * Find correct place and link it in.
      */
     for (pp = &callout; (p = *pp); pp = &p->c_next)
-       if (p->c_time.tv_sec < newp->c_time.tv_sec
-           || (p->c_time.tv_sec == newp->c_time.tv_sec
-               && p->c_time.tv_usec <= newp->c_time.tv_sec))
+       if (newp->c_time.tv_sec < p->c_time.tv_sec
+           || (newp->c_time.tv_sec == p->c_time.tv_sec
+               && newp->c_time.tv_usec < p->c_time.tv_sec))
            break;
     newp->c_next = p;
     *pp = newp;
@@ -593,7 +624,7 @@ untimeout(func, arg)
     struct callout **copp, *freep;
     int reschedule = 0;
   
-    MAINDEBUG((LOG_DEBUG, "Untimeout %x:%x.", (int) func, (int) arg));
+    MAINDEBUG((LOG_DEBUG, "Untimeout %lx:%lx.", (long) func, (long) arg));
   
     /*
      * Find first matching timeout and remove it from the list.
@@ -610,7 +641,7 @@ untimeout(func, arg)
 /*
  * calltimeout - Call any timeout routines which are now due.
  */
-void
+static void
 calltimeout()
 {
     struct callout *p;
@@ -638,7 +669,7 @@ calltimeout()
 /*
  * timeleft - return the length of time until the next timeout is due.
  */
-struct timeval *
+static struct timeval *
 timeleft(tvp)
     struct timeval *tvp;
 {
@@ -704,32 +735,43 @@ chld(sig)
 
 
 /*
- * incdebug - Catch SIGUSR1 signal.
+ * toggle_debug - Catch SIGUSR1 signal.
  *
- * Increment debug flag.
+ * Toggle debug flag.
  */
 /*ARGSUSED*/
 static void
-incdebug(sig)
+toggle_debug(sig)
     int sig;
 {
-    debug++;
+    debug = !debug;
     note_debug_level();
 }
 
 
 /*
- * nodebug - Catch SIGUSR2 signal.
+ * open_ccp - Catch SIGUSR2 signal.
  *
- * Turn off debugging.
+ * Try to (re)negotiate compression.
  */
 /*ARGSUSED*/
 static void
-nodebug(sig)
+open_ccp(sig)
     int sig;
 {
-    debug = 0;
-    note_debug_level();
+    open_ccp_flag = 1;
+}
+
+
+/*
+ * bad_signal - We've caught a fatal signal.  Clean up state and exit.
+ */
+static void
+bad_signal(sig)
+    int sig;
+{
+    syslog(LOG_ERR, "Fatal signal %d", sig);
+    die(1);
 }
 
 
@@ -737,19 +779,14 @@ nodebug(sig)
  * device_script - run a program to connect or disconnect the
  * serial device.
  */
-int
+static int
 device_script(program, in, out)
     char *program;
     int in, out;
 {
     int pid;
     int status;
-    sigset_t mask;
-
-    sigemptyset(&mask);
-    sigaddset(&mask, SIGINT);
-    sigaddset(&mask, SIGHUP);
-    sigprocmask(SIG_BLOCK, &mask, &mask);
+    int errfd;
 
     pid = fork();
 
@@ -759,11 +796,13 @@ device_script(program, in, out)
     }
 
     if (pid == 0) {
-       setreuid(getuid(), getuid());
-       setregid(getgid(), getgid());
-       sigprocmask(SIG_SETMASK, &mask, NULL);
        dup2(in, 0);
        dup2(out, 1);
+       errfd = open(_PATH_CONNERRS, O_WRONLY | O_APPEND | O_CREAT, 0644);
+       if (errfd >= 0)
+           dup2(errfd, 2);
+       setuid(getuid());
+       setgid(getgid());
        execl("/bin/sh", "sh", "-c", program, (char *)0);
        syslog(LOG_ERR, "could not exec /bin/sh: %m");
        _exit(99);
@@ -776,7 +815,6 @@ device_script(program, in, out)
        syslog(LOG_ERR, "error waiting for (dis)connection process: %m");
        die(1);
     }
-    sigprocmask(SIG_SETMASK, &mask, NULL);
 
     return (status == 0 ? 0 : -1);
 }
@@ -795,6 +833,7 @@ run_program(prog, args, must_exist)
     int must_exist;
 {
     int pid;
+    char *nullenv[1];
 
     pid = fork();
     if (pid == -1) {
@@ -802,20 +841,22 @@ run_program(prog, args, must_exist)
        return -1;
     }
     if (pid == 0) {
-        int new_fd;
+       int new_fd;
 
        /* Leave the current location */
        (void) setsid();    /* No controlling tty. */
-       (void) umask (0);   /* no umask. Must change in script. */
+       (void) umask (S_IRWXG|S_IRWXO);
        (void) chdir ("/"); /* no current directory. */
+       setuid(geteuid());
+       setgid(getegid());
 
        /* Ensure that nothing of our device environment is inherited. */
        close (0);
        close (1);
        close (2);
-       close (s);   /* Socket interface to the ppp device */
        close (fd);  /* tty interface to the ppp device */
-       
+       /* XXX should call sysdep cleanup procedure here */
+
         /* Don't pass handles to the PPP device, even by accident. */
        new_fd = open (_PATH_DEVNULL, O_RDWR);
        if (new_fd >= 0) {
@@ -827,14 +868,17 @@ run_program(prog, args, must_exist)
            dup2 (0, 2); /* stderr -> /dev/null */
        }
 
+#ifdef BSD
        /* Force the priority back to zero if pppd is running higher. */
        if (setpriority (PRIO_PROCESS, 0, 0) < 0)
            syslog (LOG_WARNING, "can't reset priority to 0: %m"); 
+#endif
 
        /* SysV recommends a second fork at this point. */
 
-       /* run the program */
-       execv(prog, args);
+       /* run the program; give it a null environment */
+       nullenv[0] = NULL;
+       execve(prog, args, nullenv);
        if (must_exist || errno != ENOENT)
            syslog(LOG_WARNING, "Can't execute %s: %m", prog);
        _exit(-1);
@@ -849,7 +893,7 @@ run_program(prog, args, must_exist)
  * reap_kids - get status from any dead child processes,
  * and log a message for abnormal terminations.
  */
-void
+static void
 reap_kids()
 {
     int pid, status;
@@ -899,23 +943,24 @@ void
 format_packet(p, len, printer, arg)
     u_char *p;
     int len;
-    void (*printer) __ARGS((void *, char *, ...));
+    void (*printer) __P((void *, char *, ...));
     void *arg;
 {
     int i, n;
     u_short proto;
     u_char x;
+    struct protent *protp;
 
-    if (len >= DLLHEADERLEN && p[0] == ALLSTATIONS && p[1] == UI) {
+    if (len >= PPP_HDRLEN && p[0] == PPP_ALLSTATIONS && p[1] == PPP_UI) {
        p += 2;
        GETSHORT(proto, p);
-       len -= DLLHEADERLEN;
-       for (i = 0; i < N_PROTO; ++i)
-           if (proto == prottbl[i].protocol)
+       len -= PPP_HDRLEN;
+       for (i = 0; (protp = protocols[i]) != NULL; ++i)
+           if (proto == protp->protocol)
                break;
-       if (i < N_PROTO) {
-           printer(arg, "[%s", prottbl[i].name);
-           n = (*prottbl[i].printpkt)(p, len, printer, arg);
+       if (protp != NULL) {
+           printer(arg, "[%s", protp->name);
+           n = (*protp->printpkt)(p, len, printer, arg);
            printer(arg, "]");
            p += n;
            len -= n;
@@ -933,7 +978,7 @@ format_packet(p, len, printer, arg)
 #ifdef __STDC__
 #include <stdarg.h>
 
-void
+static void
 pr_log(void *arg, char *fmt, ...)
 {
     int n;
@@ -956,7 +1001,7 @@ pr_log(void *arg, char *fmt, ...)
 #else /* __STDC__ */
 #include <varargs.h>
 
-void
+static void
 pr_log(arg, fmt, va_alist)
 void *arg;
 char *fmt;
@@ -988,7 +1033,7 @@ void
 print_string(p, len, printer, arg)
     char *p;
     int len;
-    void (*printer) __ARGS((void *, char *, ...));
+    void (*printer) __P((void *, char *, ...));
     void *arg;
 {
     int c;