old DLPI doesn't have dl_sap_length
[ppp.git] / pppd / main.c
index 37aeaeca9629e79d617ad29f2b1c9bb6ae713821..d278b071d95b3bbc40f13423de2e6db26db8a6bd 100644 (file)
  */
 
 #ifndef lint
-static char rcsid[] = "$Id: main.c,v 1.17 1994/09/01 00:28:14 paulus Exp $";
+static char rcsid[] = "$Id: main.c,v 1.28 1995/10/27 03:41:01 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.17 1994/09/01 00:28:14 paulus Exp $";
 #include <sys/socket.h>
 #include <net/if.h>
 
-#include "ppp.h"
 #include "pppd.h"
 #include "magic.h"
 #include "fsm.h"
@@ -59,61 +59,58 @@ static char rcsid[] = "$Id: main.c,v 1.17 1994/09/01 00:28:14 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 toggle_debug __ARGS((int));
-static void open_ccp __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 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.
@@ -128,11 +125,16 @@ static struct protent {
     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" },
+    { PPP_LCP, lcp_init, lcp_input, lcp_protrej,
+         lcp_printpkt, NULL, "LCP" },
+    { PPP_IPCP, ipcp_init, ipcp_input, ipcp_protrej,
+         ipcp_printpkt, NULL, "IPCP" },
+    { PPP_PAP, upap_init, upap_input, upap_protrej,
+         upap_printpkt, NULL, "PAP" },
+    { PPP_CHAP, ChapInit, ChapInput, ChapProtocolReject,
+         ChapPrintPkt, NULL, "CHAP" },
+    { PPP_CCP, ccp_init, ccp_input, ccp_protrej,
+         ccp_printpkt, ccp_datainput, "CCP" },
 };
 
 #define N_PROTO                (sizeof(prottbl) / sizeof(prottbl[0]))
@@ -141,7 +143,7 @@ main(argc, argv)
     int argc;
     char *argv[];
 {
-    int i, nonblock;
+    int i, nonblock, fdflags;
     struct sigaction sa;
     struct cmd *cmdp;
     FILE *pidfile;
@@ -153,6 +155,7 @@ main(argc, argv)
     p = ttyname(0);
     if (p)
        strcpy(devnam, p);
+    strcpy(default_devnam, devnam);
 
     if (gethostname(hostname, MAXNAMELEN) < 0 ) {
        perror("couldn't get hostname");
@@ -163,7 +166,7 @@ main(argc, argv)
     uid = getuid();
 
     if (!ppp_available()) {
-       fprintf(stderr, "Sorry - PPP is not available on this system\n");
+       fprintf(stderr, no_ppp_msg);
        exit(1);
     }
 
@@ -185,6 +188,13 @@ main(argc, argv)
     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.
      */
@@ -210,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
@@ -243,8 +247,47 @@ main(argc, argv)
     SIGNAL(SIGTERM, term);             /* Terminate */
     SIGNAL(SIGCHLD, chld);
 
-    signal(SIGUSR1, toggle_debug);     /* Toggle debug flag */
-    signal(SIGUSR2, open_ccp);         /* Reopen CCP */
+    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.
@@ -266,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;
@@ -297,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);
 
@@ -311,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);
@@ -320,14 +361,6 @@ main(argc, argv)
            pidfilename[0] = 0;
        }
 
-       /*
-        * Set device for non-blocking reads.
-        */
-       if (fcntl(fd, F_SETFL, initfdflags | O_NONBLOCK) == -1) {
-           syslog(LOG_ERR, "Couldn't set device to non-blocking mode: %m");
-           die(1);
-       }
-  
        /*
         * Block all signals, start opening the connection, and wait for
         * incoming events (reply, timeout, etc.).
@@ -338,6 +371,7 @@ main(argc, argv)
        for (phase = PHASE_ESTABLISH; phase != PHASE_DEAD; ) {
            wait_input(timeleft(&timo));
            calltimeout();
+           get_input();
            if (kill_link) {
                lcp_close(0);
                kill_link = 0;
@@ -349,20 +383,21 @@ main(argc, argv)
                }
                open_ccp_flag = 0;
            }
-           get_input();
            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();
@@ -372,10 +407,7 @@ main(argc, argv)
 
     } while (persist);
 
-    if (lockflag && !default_device)
-       unlock();
-
-    exit(0);
+    die(0);
 }
 
 
@@ -389,62 +421,59 @@ get_input()
     u_char *p;
     u_short protocol;
 
-    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;
+    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 (len == 0) {
+       syslog(LOG_NOTICE, "Modem hangup");
+       hungup = 1;
+       lcp_lowerdown(0);       /* serial link is no longer available */
+       link_terminated(0);
+       return;
+    }
 
-       if (debug /*&& (debugflags & DBG_INPACKET)*/)
-           log_packet(p, len, "rcvd ");
+    if (debug /*&& (debugflags & DBG_INPACKET)*/)
+       log_packet(p, len, "rcvd ");
 
-       if (len < DLLHEADERLEN) {
-           MAINDEBUG((LOG_INFO, "io(): Received short packet."));
-           return;
-       }
+    if (len < PPP_HDRLEN) {
+       MAINDEBUG((LOG_INFO, "io(): Received short packet."));
+       return;
+    }
 
-       p += 2;                         /* Skip address and control */
-       GETSHORT(protocol, p);
-       len -= DLLHEADERLEN;
+    p += 2;                            /* Skip address and control */
+    GETSHORT(protocol, p);
+    len -= PPP_HDRLEN;
 
-       /*
-        * 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."));
+    /*
+     * 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 input routine.
+     */
+    for (i = 0; i < sizeof (prottbl) / sizeof (struct protent); i++) {
+       if (prottbl[i].protocol == protocol) {
+           (*prottbl[i].input)(0, p, len);
            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 (i == sizeof (prottbl) / sizeof (struct protent)) {
-           if (debug)
-               syslog(LOG_WARNING, "Unknown protocol (0x%x) received",
-                      protocol);
-           lcp_sprotrej(0, p - DLLHEADERLEN, len + DLLHEADERLEN);
+        if (protocol == (prottbl[i].protocol & ~0x8000)
+           && prottbl[i].datainput != NULL) {
+           (*prottbl[i].datainput)(0, p, len);
+           return;
        }
     }
+
+    if (debug)
+       syslog(LOG_WARNING, "Unknown protocol (0x%x) received", protocol);
+    lcp_sprotrej(0, p - PPP_HDRLEN, len + PPP_HDRLEN);
 }
 
 
@@ -467,19 +496,18 @@ demuxprotrej(unit, protocol)
            return;
        }
 
-    syslog(LOG_WARNING,
-          "demuxprotrej: Unrecognized Protocol-Reject for protocol 0x%x",
+    syslog(LOG_WARNING, "Unrecognized Protocol-Reject for protocol 0x%x",
           protocol);
 }
 
 
 /*
- * quit - Clean up state and exit.
+ * quit - Clean up state and exit (with an error indication).
  */
 void 
 quit()
 {
-    die(0);
+    die(1);
 }
 
 /*
@@ -489,7 +517,7 @@ void
 die(status)
     int status;
 {
-    cleanup(0, NULL);
+    cleanup();
     syslog(LOG_INFO, "Exit.");
     exit(status);
 }
@@ -498,11 +526,11 @@ die(status)
  * 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();
 
@@ -517,19 +545,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);
@@ -561,8 +585,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.
@@ -581,9 +605,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;
@@ -602,7 +626,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.
@@ -619,7 +643,7 @@ untimeout(func, arg)
 /*
  * calltimeout - Call any timeout routines which are now due.
  */
-void
+static void
 calltimeout()
 {
     struct callout *p;
@@ -647,7 +671,7 @@ calltimeout()
 /*
  * timeleft - return the length of time until the next timeout is due.
  */
-struct timeval *
+static struct timeval *
 timeleft(tvp)
     struct timeval *tvp;
 {
@@ -741,23 +765,30 @@ open_ccp(sig)
 }
 
 
+/*
+ * 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);
+}
+
+
 /*
  * 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();
 
@@ -767,11 +798,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);
@@ -784,7 +817,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);
 }
@@ -803,6 +835,7 @@ run_program(prog, args, must_exist)
     int must_exist;
 {
     int pid;
+    char *nullenv[1];
 
     pid = fork();
     if (pid == -1) {
@@ -810,20 +843,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) {
@@ -835,14 +870,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);
@@ -857,7 +895,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;
@@ -907,17 +945,17 @@ 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;
 
-    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;
+       len -= PPP_HDRLEN;
        for (i = 0; i < N_PROTO; ++i)
            if (proto == prottbl[i].protocol)
                break;
@@ -941,7 +979,7 @@ format_packet(p, len, printer, arg)
 #ifdef __STDC__
 #include <stdarg.h>
 
-void
+static void
 pr_log(void *arg, char *fmt, ...)
 {
     int n;
@@ -964,7 +1002,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;
@@ -996,7 +1034,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;