]> git.ozlabs.org Git - ppp.git/blobdiff - pppd/sys-NeXT.c
Large patch from Frank Cusack <fcusack@fcusack.com> to add proper
[ppp.git] / pppd / sys-NeXT.c
index 7a8d7946d3638a4c39893b4c6a0d38618b2940a4..8df34e55b9f51be0ae96aeb7e69a36f63ecbf4bb 100644 (file)
@@ -4,7 +4,6 @@
  *
  * Copyright (c) 1989 Carnegie Mellon University.
  * Copyright (c) 1994 Philippe-Andre Prindeville.
- * Copyright (c) 1995 The Australian National University.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms are permitted
  * duplicated in all such forms and that any documentation,
  * advertising materials, and other materials related to such
  * distribution and use acknowledge that the software was developed
- * by Carnegie Mellon University, Philippe-Andre Prindeville and
- * The Australian National University.  The names of the developers
- * may not be used to endorse or promote products derived from this
- * software without specific prior written permission.
+ * by Carnegie Mellon University.  The name of the
+ * University may not be used to endorse or promote products derived
+ * from this software without specific prior written permission.
  * THIS SOFTWARE IS PROVIDED ``AS IS'' AND WITHOUT ANY EXPRESS OR
  * IMPLIED WARRANTIES, INCLUDING, WITHOUT LIMITATION, THE IMPLIED
  * WARRANTIES OF MERCHANTIBILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
-#ifndef lint
-static char rcsid[] = "$Id: sys-NeXT.c,v 1.5 1996/04/04 04:04:17 paulus Exp $";
-#endif
+#define RCSID  "$Id: sys-NeXT.c,v 1.20 1999/08/13 06:46:17 paulus Exp $"
 
 #include <stdio.h>
-#include <syslog.h>
 #include <termios.h>
 #include <utmp.h>
 #include <unistd.h>
 #include <stdlib.h>
+#include <libc.h>
 #include <strings.h>
 #include <sys/types.h>
 #include <sys/file.h>
@@ -48,7 +44,10 @@ static char rcsid[] = "$Id: sys-NeXT.c,v 1.5 1996/04/04 04:04:17 paulus Exp $";
 #include <netinet/in.h>
 #include <netinet/in_systm.h>
 #include <netinet/in_var.h>
+#if !(NS_TARGET >= 40)
+/* XXX get an error "duplicate member ip_v under 4.1 GAMMA */
 #include <netinet/ip.h>
+#endif /* NS_TARGET */
 #include <netinet/if_ether.h>
 #include <net/route.h>
 #include <netinet/in.h>
@@ -57,32 +56,31 @@ static char rcsid[] = "$Id: sys-NeXT.c,v 1.5 1996/04/04 04:04:17 paulus Exp $";
 
 #include "pppd.h"
 
-static int initdisc = -1;      /* Initial TTY discipline for ppp_fd */
-static int initfdflags = -1;   /* Initial file descriptor flags for ppp_fd */
-static int ppp_fd = -1;                /* fd which is set to PPP discipline */
-
-extern int errno;
-
-static int restore_term;       /* 1 => we've munged the terminal */
-static struct termios inittermios; /* Initial TTY termios */
-static struct winsize wsinfo;  /* Initial window size info */
-
-static char *lock_file;                /* name of lock file created */
+static const char rcsid[] = RCSID;
 
+static int initdisc = -1;      /* Initial TTY discipline */
+static int initfdflags = -1;   /* Initial file descriptor flags for fd */
+static int ppp_fd = -1;                /* fd which is set to PPP discipline */
 static int loop_slave = -1;
 static int loop_master;
 static char loop_name[20];
 
-static unsigned char inbuf[512]; /* buffer for chars read from loopback */
+static fd_set in_fds;          /* set of fds that wait_input waits for */
+static int max_in_fd;          /* highest fd set in in_fds */
+
+extern int errno;
+
+static int     restore_term;   /* 1 => we've munged the terminal */
+static struct termios inittermios; /* Initial TTY termios */
 
 static int sockfd;             /* socket for doing interface ioctls */
+static int pppdev;  /* +++ */
 
 #if defined(i386) && defined(HAS_BROKEN_IOCTL)
 #define        ioctl   myioctl
 #endif
 
 static int if_is_up;           /* the interface is currently up */
-static u_int32_t ifaddrs[2];   /* local and remote addresses */
 static u_int32_t default_route_gateway;        /* gateway addr for default route */
 static u_int32_t proxy_arp_addr;       /* remote addr for proxy arp */
 
@@ -102,14 +100,16 @@ sys_init()
 {
     openlog("pppd", LOG_PID | LOG_NDELAY, LOG_PPP);
     setlogmask(LOG_UPTO(LOG_INFO));
-    if (debug)
-       setlogmask(LOG_UPTO(LOG_DEBUG));
 
     /* Get an internet socket for doing socket ioctl's on. */
-    if ((sockfd = socket(AF_INET, SOCK_DGRAM, 0)) < 0) {
-       syslog(LOG_ERR, "Couldn't create IP socket: %m");
-       die(1);
-    }
+    if ((sockfd = socket(AF_INET, SOCK_DGRAM, 0)) < 0)
+       fatal("Couldn't create IP socket: %m");
+
+    if((pppdev = open("/dev/ppp0", O_RDWR, O_NONBLOCK)) == NULL)
+       fatal("Couldn't open /dev/ppp0: %m");
+      
+    FD_ZERO(&in_fds);
+    max_in_fd = 0;
 }
 
 /*
@@ -123,56 +123,20 @@ sys_cleanup()
     struct ifreq ifr;
 
     if (if_is_up) {
-       strncpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name));
+       strlcpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name));
        if (ioctl(sockfd, SIOCGIFFLAGS, &ifr) >= 0
            && ((ifr.ifr_flags & IFF_UP) != 0)) {
            ifr.ifr_flags &= ~IFF_UP;
            ioctl(sockfd, SIOCSIFFLAGS, &ifr);
        }
     }
-    if (ifaddrs[0])
-       cifaddr(0, ifaddrs[0], ifaddrs[1]);
+
     if (default_route_gateway)
-       cifdefaultroute(0, default_route_gateway);
+       cifdefaultroute(0, 0, default_route_gateway);
     if (proxy_arp_addr)
        cifproxyarp(0, proxy_arp_addr);
-}
 
-/*
- * sys_close - Clean up in a child process before execing.
- */
-void
-sys_close()
-{
-    close(sockfd);
-    if (loop_slave >= 0) {
-       close(loop_slave);
-       close(loop_master);
-    }
-    closelog();
-}
-
-/*
- * sys_check_options - check the options that the user specified
- */
-void
-sys_check_options()
-{
-}
-
-
-/*
- * note_debug_level - note a change in the debug level.
- */
-void
-note_debug_level()
-{
-    if (debug) {
-       syslog(LOG_INFO, "Debug turned ON, Level %d", debug);
-       setlogmask(LOG_UPTO(LOG_DEBUG));
-    } else {
-       setlogmask(LOG_UPTO(LOG_WARNING));
-    }
+    close(pppdev);
 }
 
 /*
@@ -184,132 +148,67 @@ ppp_available()
 {
     int s, ok;
     struct ifreq ifr;
+    extern char *no_ppp_msg;
 
     if ((s = socket(AF_INET, SOCK_DGRAM, 0)) < 0)
        return 1;               /* can't tell - maybe we're not root */
 
-    strncpy(ifr.ifr_name, "ppp0", sizeof (ifr.ifr_name));
+    strlcpy(ifr.ifr_name, "ppp0", sizeof (ifr.ifr_name));
     ok = ioctl(s, SIOCGIFFLAGS, (caddr_t) &ifr) >= 0;
     close(s);
 
+    no_ppp_msg = "\
+This system lacks kernel support for PPP.  To include PPP support\n\
+in the kernel, please follow the steps detailed in the README.NeXT\n\
+file in the ppp-2.2 distribution.\n";
+
     return ok;
 }
 
 /*
  * establish_ppp - Turn the serial port into a ppp interface.
  */
-void
+int
 establish_ppp(fd)
     int fd;
 {
     int pppdisc = PPPDISC;
     int x;
 
-    if (demand) {
-       /*
-        * Demand mode - prime the old ppp device to relinquish the unit.
-        */
-       if (ioctl(ppp_fd, PPPIOCXFERUNIT, 0) < 0) {
-           syslog(LOG_ERR, "ioctl(transfer ppp unit): %m");
-           die(1);
-       }
-    }
+    if (ioctl(fd, TIOCGETD, &initdisc) < 0)
+       fatal("ioctl(TIOCGETD): %m");
+    if (ioctl(fd, TIOCSETD, &pppdisc) < 0)
+       fatal("ioctl(establish TIOCSETD): %m");
 
     /*
-     * Save the old line discipline of fd, and set it to PPP.
+     * Find out which interface we were given.
      */
-    if (ioctl(fd, TIOCGETD, &initdisc) < 0) {
-       syslog(LOG_ERR, "ioctl(TIOCGETD): %m");
-       die(1);
-    }
-    if (ioctl(fd, TIOCSETD, &pppdisc) < 0) {
-       syslog(LOG_ERR, "ioctl(establish TIOCSETD): %m");
-       die(1);
-    }
-
-    if (!demand) {
-       /*
-        * Find out which interface we were given.
-        */
-       if (ioctl(fd, PPPIOCGUNIT, &ifunit) < 0) {      
-           syslog(LOG_ERR, "ioctl(PPPIOCGUNIT): %m");
-           die(1);
-       }
-    } else {
-       /*
-        * Check that we got the same unit again.
-        */
-       if (ioctl(fd, PPPIOCGUNIT, &x) < 0) {   
-           syslog(LOG_ERR, "ioctl(PPPIOCGUNIT): %m");
-           die(1);
-       }
-       if (x != ifunit) {
-           syslog(LOG_ERR, "transfer_ppp failed: wanted unit %d, got %d",
-                  ifunit, x);
-           die(1);
-       }
-       x = TTYDISC;
-       ioctl(loop_slave, TIOCSETD, &x);
-    }
-
-    ppp_fd = fd;
+    if (ioctl(fd, PPPIOCGUNIT, &ifunit) < 0)
+       fatal("ioctl(PPPIOCGUNIT): %m");
 
     /*
      * Enable debug in the driver if requested.
      */
     if (kdebugflag) {
        if (ioctl(fd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
-           syslog(LOG_WARNING, "ioctl(PPPIOCGFLAGS): %m");
+           warn("ioctl(PPPIOCGFLAGS): %m");
        } else {
            x |= (kdebugflag & 0xFF) * SC_DEBUG;
            if (ioctl(fd, PPPIOCSFLAGS, (caddr_t) &x) < 0)
-               syslog(LOG_WARNING, "ioctl(PPPIOCSFLAGS): %m");
+               warn("ioctl(PPPIOCSFLAGS): %m");
        }
     }
 
     /*
-     * Set device for non-blocking reads.
+     * Set device for non-blocking reads so PPPD can poll for
+     * input from the kernel.
      */
     if ((initfdflags = fcntl(fd, F_GETFL)) == -1
        || fcntl(fd, F_SETFL, initfdflags | O_NONBLOCK) == -1) {
-       syslog(LOG_WARNING, "Couldn't set device to non-blocking mode: %m");
+       warn("Couldn't set device to non-blocking mode: %m");
     }
-}
 
-/*
- * restore_loop - reattach the ppp unit to the loopback.
- */
-void
-restore_loop()
-{
-    int x;
-
-    /*
-     * Transfer the ppp interface back to the loopback.
-     */
-    if (ioctl(ppp_fd, PPPIOCXFERUNIT, 0) < 0) {
-       syslog(LOG_ERR, "ioctl(transfer ppp unit): %m");
-       die(1);
-    }
-    x = PPPDISC;
-    if (ioctl(loop_slave, TIOCSETD, &x) < 0) {
-       syslog(LOG_ERR, "ioctl(TIOCSETD): %m");
-       die(1);
-    }
-
-    /*
-     * Check that we got the same unit again.
-     */
-    if (ioctl(loop_slave, PPPIOCGUNIT, &x) < 0) {      
-       syslog(LOG_ERR, "ioctl(PPPIOCGUNIT): %m");
-       die(1);
-    }
-    if (x != ifunit) {
-       syslog(LOG_ERR, "transfer_ppp failed: wanted unit %d, got %d",
-              ifunit, x);
-       die(1);
-    }
-    ppp_fd = loop_slave;
+    return fd;
 }
 
 
@@ -323,16 +222,13 @@ disestablish_ppp(fd)
 {
     /* Reset non-blocking mode on fd. */
     if (initfdflags != -1 && fcntl(fd, F_SETFL, initfdflags) < 0)
-       syslog(LOG_WARNING, "Couldn't restore device fd flags: %m");
+       warn("Couldn't restore device fd flags: %m");
     initfdflags = -1;
 
     /* Restore old line discipline. */
     if (initdisc >= 0 && ioctl(fd, TIOCSETD, &initdisc) < 0)
-       syslog(LOG_ERR, "ioctl(disestablish TIOCSETD): %m");
+       error("ioctl(TIOCSETD): %m");
     initdisc = -1;
-
-    if (fd == ppp_fd)
-       ppp_fd = -1;
 }
 
 /*
@@ -344,7 +240,7 @@ clean_check()
     int x;
     char *s;
 
-    if (ioctl(ppp_fd, PPPIOCGFLAGS, (caddr_t) &x) == 0) {
+    if (ioctl(ttyfd, PPPIOCGFLAGS, (caddr_t) &x) == 0) {
        s = NULL;
        switch (~x & (SC_RCV_B7_0|SC_RCV_B7_1|SC_RCV_EVNP|SC_RCV_ODDP)) {
        case SC_RCV_B7_0:
@@ -361,8 +257,8 @@ clean_check()
            break;
        }
        if (s != NULL) {
-           syslog(LOG_WARNING, "Serial link is not 8-bit clean:");
-           syslog(LOG_WARNING, "All received characters had %s", s);
+           warn("Serial link is not 8-bit clean:");
+           warn("All received characters had %s", s);
        }
     }
 }
@@ -471,7 +367,7 @@ translate_speed(bps)
     for (speedp = speeds; speedp->speed_int; speedp++)
        if (bps == speedp->speed_int)
            return speedp->speed_val;
-    syslog(LOG_WARNING, "speed %d not supported", bps);
+    warn("speed %d not supported", bps);
     return 0;
 }
 
@@ -505,21 +401,13 @@ set_up_tty(fd, local)
     int speed, x, modembits;
     struct termios tios;
 
-    if (tcgetattr(fd, &tios) < 0) {
-       syslog(LOG_ERR, "tcgetattr: %m");
-       die(1);
-    }
+    if (tcgetattr(fd, &tios) < 0)
+       fatal("tcgetattr: %m");
 
-    if (!restore_term) {
+    if (!restore_term)
        inittermios = tios;
-       ioctl(fd, TIOCGWINSZ, &wsinfo);
-    }
 
     tios.c_cflag &= ~(CSIZE | CSTOPB | PARENB | CLOCAL);
-    if (crtscts > 0 && !local)
-       tios.c_cflag |= CRTSCTS;
-    else if (crtscts < 0)
-       tios.c_cflag &= ~CRTSCTS;
 
     tios.c_cflag |= CS8 | CREAD | HUPCL;
     if (local || !modem)
@@ -543,18 +431,23 @@ set_up_tty(fd, local)
        cfsetispeed(&tios, speed);
     } else {
        speed = cfgetospeed(&tios);
+       /*
+        * We can't proceed if the serial port speed is B0,
+        * since that implies that the serial port is disabled.
+        */
+       if (speed == B0)
+           fatal("Baud rate for %s is 0; need explicit baud rate",
+                 devnam);
     }
 
     if (modem) {
       modembits = TIOCM_RTS | TIOCM_CTS;
       if (ioctl(fd, (crtscts ? TIOCMBIS : TIOCMBIC), &modembits) < 0)
-       syslog(LOG_ERR, "ioctl: TIOCMBIS/BIC: %m");
+       error("ioctl: TIOCMBIS/BIC: %m");
     }
 
-    if (tcsetattr(fd, TCSAFLUSH, &tios) < 0) {
-       syslog(LOG_ERR, "tcsetattr: %m");
-       die(1);
-    }
+    if (tcsetattr(fd, TCSAFLUSH, &tios) < 0)
+       fatal("tcsetattr: %m");
 
    baud_rate = inspeed = baud_rate_of(speed);
    restore_term = 1;
@@ -568,19 +461,9 @@ restore_tty(fd)
     int fd;
 {
     if (restore_term) {
-       if (!default_device) {
-           /*
-            * Turn off echoing, because otherwise we can get into
-            * a loop with the tty and the modem echoing to each other.
-            * We presume we are the sole user of this tty device, so
-            * when we close it, it will revert to its defaults anyway.
-            */
-           inittermios.c_lflag &= ~(ECHO | ECHONL);
-       }
        if (tcsetattr(fd, TCSAFLUSH, &inittermios) < 0)
            if (errno != ENXIO)
-               syslog(LOG_WARNING, "tcsetattr: %m");
-       ioctl(fd, TIOCSWINSZ, &wsinfo);
+               warn("tcsetattr: %m");
        restore_term = 0;
     }
 }
@@ -611,68 +494,6 @@ int fd, on;
 }
 
 
-/*
- * open_ppp_loopback - open the device we use for getting
- * packets in demand mode, and connect it to a ppp interface.
- * Here we use a pty.
- */
-void
-open_ppp_loopback()
-{
-    int flags;
-    struct termios tios;
-    int pppdisc = PPPDISC;
-
-    if (openpty(&loop_master, &loop_slave, loop_name, NULL, NULL) < 0) {
-       syslog(LOG_ERR, "No free pty for loopback");
-       die(1);
-    }
-    SYSDEBUG((LOG_DEBUG, "using %s for loopback", loop_name));
-
-    if (tcgetattr(loop_slave, &tios) == 0) {
-       tios.c_cflag &= ~(CSIZE | CSTOPB | PARENB);
-       tios.c_cflag |= CS8 | CREAD;
-       tios.c_iflag = IGNPAR;
-       tios.c_oflag = 0;
-       tios.c_lflag = 0;
-       if (tcsetattr(loop_slave, TCSAFLUSH, &tios) < 0)
-           syslog(LOG_WARNING, "couldn't set attributes on loopback: %m");
-    }
-
-    if ((flags = fcntl(loop_master, F_GETFL)) != -1) 
-       if (fcntl(loop_master, F_SETFL, flags | O_NONBLOCK) == -1)
-           syslog(LOG_WARNING, "couldn't set loopback to nonblock: %m");
-
-    ppp_fd = loop_slave;
-    if (ioctl(ppp_fd, TIOCSETD, &pppdisc) < 0) {
-       syslog(LOG_ERR, "ioctl(TIOCSETD): %m");
-       die(1);
-    }
-
-    /*
-     * Find out which interface we were given.
-     */
-    if (ioctl(ppp_fd, PPPIOCGUNIT, &ifunit) < 0) {     
-       syslog(LOG_ERR, "ioctl(PPPIOCGUNIT): %m");
-       die(1);
-    }
-
-    /*
-     * Enable debug in the driver if requested.
-     */
-    if (kdebugflag) {
-       if (ioctl(ppp_fd, PPPIOCGFLAGS, (caddr_t) &flags) < 0) {
-           syslog(LOG_WARNING, "ioctl (PPPIOCGFLAGS): %m");
-       } else {
-           flags |= (kdebugflag & 0xFF) * SC_DEBUG;
-           if (ioctl(ppp_fd, PPPIOCSFLAGS, (caddr_t) &flags) < 0)
-               syslog(LOG_WARNING, "ioctl(PPPIOCSFLAGS): %m");
-       }
-    }
-
-}
-
-
 /*
  * output - Output PPP packet.
  */
@@ -683,22 +504,21 @@ output(unit, p, len)
     int len;
 {
     if (debug)
-       log_packet(p, len, "sent ");
+       dbglog("sent %P", p, len);
 
     if (write(ttyfd, p, len) < 0) {
        if (errno == EWOULDBLOCK || errno == ENOBUFS
            || errno == ENXIO || errno == EIO) {
-           syslog(LOG_WARNING, "write: warning: %m");
+           warn("write: warning: %m");
        } else {
-           syslog(LOG_ERR, "write: %m");
-           die(1);
+           fatal("write: %m");
        }
     }
 }
 
 
 /*
- * wait_input - wait until there is data available on ttyfd,
+ * wait_input - wait until there is data available,
  * for the length of time specified by *timo (indefinite
  * if timo is NULL).
  */
@@ -709,54 +529,33 @@ wait_input(timo)
     fd_set ready;
     int n;
 
-    FD_ZERO(&ready);
-    FD_SET(ttyfd, &ready);
-    n = select(ttyfd+1, &ready, NULL, &ready, timo);
-    if (n < 0 && errno != EINTR) {
-       syslog(LOG_ERR, "select: %m");
-       die(1);
-    }
+    ready = in_fds;
+    n = select(max_in_fd + 1, &ready, NULL, &ready, timo);
+    if (n < 0 && errno != EINTR)
+       fatal("select: %m");
 }
 
 
 /*
- * wait_loop_output - wait until there is data available on the
- * loopback, for the length of time specified by *timo (indefinite
- * if timo is NULL).
+ * add_fd - add an fd to the set that wait_input waits for.
  */
-wait_loop_output(timo)
-    struct timeval *timo;
+void add_fd(fd)
+    int fd;
 {
-    fd_set ready;
-    int n;
-
-    FD_ZERO(&ready);
-    FD_SET(loop_master, &ready);
-    n = select(loop_master + 1, &ready, NULL, &ready, timo);
-    if (n < 0 && errno != EINTR) {
-       syslog(LOG_ERR, "select: %m");
-       die(1);
-    }
+    FD_SET(fd, &in_fds);
+    if (fd > max_in_fd)
+       max_in_fd = fd;
 }
 
-
 /*
- * wait_time - wait for a given length of time or until a
- * signal is received.
+ * remove_fd - remove an fd from the set that wait_input waits for.
  */
-wait_time(timo)
-    struct timeval *timo;
+void remove_fd(fd)
+    int fd;
 {
-    int n;
-
-    n = select(0, NULL, NULL, NULL, timo);
-    if (n < 0 && errno != EINTR) {
-       syslog(LOG_ERR, "select: %m");
-       die(1);
-    }
+    FD_CLR(fd, &in_fds);
 }
 
-
 /*
  * read_packet - get a PPP packet from the serial device.
  */
@@ -767,43 +566,16 @@ read_packet(buf)
     int len;
 
     if ((len = read(ttyfd, buf, PPP_MTU + PPP_HDRLEN)) < 0) {
-       if (errno == EWOULDBLOCK || errno == EINTR)
+       if (errno == EWOULDBLOCK || errno == EINTR) {
+           SYSDEBUG(("read: %m"));
            return -1;
-       syslog(LOG_ERR, "read: %m");
-       die(1);
+       }
+       fatal("read: %m");
     }
     return len;
 }
 
 
-/*
- * get_loop_output - read characters from the loopback, form them
- * into frames, and detect when we want to bring the real link up.
- * Return value is 1 if we need to bring up the link, 0 otherwise.
- */
-int
-get_loop_output()
-{
-    int rv = 0;
-    int n;
-
-    while ((n = read(loop_master, inbuf, sizeof(inbuf))) >= 0) {
-       if (loop_chars(inbuf, n))
-           rv = 1;
-    }
-
-    if (n == 0) {
-       syslog(LOG_ERR, "eof on loopback");
-       die(1);
-    } else if (errno != EWOULDBLOCK){
-       syslog(LOG_ERR, "read from loopback: %m");
-       die(1);
-    }
-
-    return rv;
-}
-
-
 /*
  * ppp_send_config - configure the transmit characteristics of
  * the ppp interface.
@@ -817,28 +589,21 @@ ppp_send_config(unit, mtu, asyncmap, pcomp, accomp)
     u_int x;
     struct ifreq ifr;
 
-    strncpy(ifr.ifr_name, ifname, sizeof (ifr.ifr_name));
+    strlcpy(ifr.ifr_name, ifname, sizeof (ifr.ifr_name));
     ifr.ifr_mtu = mtu;
-    if (ioctl(sockfd, SIOCSIFMTU, (caddr_t) &ifr) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFMTU): %m");
-       quit();
-    }
+    if (ioctl(sockfd, SIOCSIFMTU, (caddr_t) &ifr) < 0)
+       fatal("ioctl(SIOCSIFMTU): %m");
 
-    if (ioctl(ppp_fd, PPPIOCSASYNCMAP, (caddr_t) &asyncmap) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCSASYNCMAP): %m");
-       quit();
-    }
+    if (ioctl(ttyfd, PPPIOCSASYNCMAP, (caddr_t) &asyncmap) < 0)
+       fatal("ioctl(PPPIOCSASYNCMAP): %m");
+
+    if (ioctl(ttyfd, PPPIOCGFLAGS, (caddr_t) &x) < 0)
+       fatal("ioctl(PPPIOCGFLAGS): %m");
 
-    if (ioctl(ppp_fd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCGFLAGS): %m");
-       quit();
-    }
     x = pcomp? x | SC_COMP_PROT: x &~ SC_COMP_PROT;
     x = accomp? x | SC_COMP_AC: x &~ SC_COMP_AC;
-    if (ioctl(ppp_fd, PPPIOCSFLAGS, (caddr_t) &x) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCSFLAGS): %m");
-       quit();
-    }
+    if (ioctl(ttyfd, PPPIOCSFLAGS, (caddr_t) &x) < 0)
+       fatal("ioctl(PPPIOCSFLAGS): %m");
 }
 
 
@@ -850,8 +615,8 @@ ppp_set_xaccm(unit, accm)
     int unit;
     ext_accm accm;
 {
-    if (ioctl(ppp_fd, PPPIOCSXASYNCMAP, accm) < 0 && errno != ENOTTY)
-       syslog(LOG_WARNING, "ioctl(set extended ACCM): %m");
+    if (ioctl(ttyfd, PPPIOCSXASYNCMAP, accm) < 0 && errno != ENOTTY)
+       warn("ioctl(PPPIOCSXASYNCMAP): %m");
 }
 
 
@@ -867,30 +632,20 @@ ppp_recv_config(unit, mru, asyncmap, pcomp, accomp)
 {
     int x;
 
-    if (ioctl(ppp_fd, PPPIOCSMRU, (caddr_t) &mru) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCSMRU): %m");
-       quit();
-    }
-    if (ioctl(ppp_fd, PPPIOCSRASYNCMAP, (caddr_t) &asyncmap) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCSRASYNCMAP): %m");
-       quit();
-    }
-    if (ioctl(ppp_fd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCGFLAGS): %m");
-       quit();
-    }
+    if (ioctl(ttyfd, PPPIOCSMRU, (caddr_t) &mru) < 0)
+       fatal("ioctl(PPPIOCSMRU): %m");
+    if (ioctl(ttyfd, PPPIOCSRASYNCMAP, (caddr_t) &asyncmap) < 0)
+       fatal("ioctl(PPPIOCSRASYNCMAP): %m");
+    if (ioctl(ttyfd, PPPIOCGFLAGS, (caddr_t) &x) < 0)
+       fatal("ioctl(PPPIOCGFLAGS): %m");
     x = !accomp? x | SC_REJ_COMP_AC: x &~ SC_REJ_COMP_AC;
-    if (ioctl(ppp_fd, PPPIOCSFLAGS, (caddr_t) &x) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCSFLAGS): %m");
-       quit();
-    }
+    if (ioctl(ttyfd, PPPIOCSFLAGS, (caddr_t) &x) < 0)
+       fatal("ioctl(PPPIOCSFLAGS): %m");
 }
 
 /*
  * ccp_test - ask kernel whether a given compression method
- * is acceptable for use.  Returns 1 if the method and parameters
- * are OK, 0 if the method is known but the parameters are not OK
- * (e.g. code size should be reduced), or -1 if the method is unknown.
+ * is acceptable for use.
  */
 int
 ccp_test(unit, opt_ptr, opt_len, for_transmit)
@@ -916,14 +671,14 @@ ccp_flags_set(unit, isopen, isup)
 {
     int x;
 
-    if (ioctl(ppp_fd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCGFLAGS): %m");
+    if (ioctl(ttyfd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
+       error("ioctl(PPPIOCGFLAGS): %m");
        return;
     }
     x = isopen? x | SC_CCP_OPEN: x &~ SC_CCP_OPEN;
     x = isup? x | SC_CCP_UP: x &~ SC_CCP_UP;
-    if (ioctl(ppp_fd, PPPIOCSFLAGS, (caddr_t) &x) < 0)
-       syslog(LOG_ERR, "ioctl(PPPIOCSFLAGS): %m");
+    if (ioctl(ttyfd, PPPIOCSFLAGS, (caddr_t) &x) < 0)
+       error("ioctl(PPPIOCSFLAGS): %m");
 }
 
 /*
@@ -937,50 +692,13 @@ ccp_fatal_error(unit)
 {
     int x;
 
-    if (ioctl(ppp_fd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCGFLAGS): %m");
+    if (ioctl(ttyfd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
+       error("ioctl(PPPIOCGFLAGS): %m");
        return 0;
     }
     return x & SC_DC_FERROR;
 }
 
-/*
- * get_idle_time - return how long the link has been idle.
- */
-int
-get_idle_time(u, ip)
-    int u;
-    struct ppp_idle *ip;
-{
-    return ioctl(ppp_fd, PPPIOCGIDLE, ip) >= 0;
-}
-
-
-/*
- * set_filters - transfer the pass and active filters to the kernel.
- */
-int
-set_filters(pass, active)
-    struct bpf_program *pass, *active;
-{
-    int ret = 1;
-
-    if (pass->bf_len > 0) {
-       if (ioctl(ppp_fd, PPPIOCSPASS, pass) < 0) {
-           syslog(LOG_ERR, "Couldn't set pass-filter in kernel: %m");
-           ret = 0;
-       }
-    }
-    if (active->bf_len > 0) {
-       if (ioctl(ppp_fd, PPPIOCSACTIVE, active) < 0) {
-           syslog(LOG_ERR, "Couldn't set active-filter in kernel: %m");
-           ret = 0;
-       }
-    }
-    return ret;
-}
-
-
 /*
  * sifvjcomp - config tcp header compression
  */
@@ -990,18 +708,18 @@ sifvjcomp(u, vjcomp, cidcomp, maxcid)
 {
     u_int x;
 
-    if (ioctl(ppp_fd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCGFLAGS): %m");
+    if (ioctl(ttyfd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
+       error("ioctl(PPIOCGFLAGS): %m");
        return 0;
     }
     x = vjcomp ? x | SC_COMP_TCP: x &~ SC_COMP_TCP;
     x = cidcomp? x & ~SC_NO_TCP_CCID: x | SC_NO_TCP_CCID;
-    if (ioctl(ppp_fd, PPPIOCSFLAGS, (caddr_t) &x) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCSFLAGS): %m");
+    if (ioctl(ttyfd, PPPIOCSFLAGS, (caddr_t) &x) < 0) {
+       error("ioctl(PPPIOCSFLAGS): %m");
        return 0;
     }
-    if (ioctl(ppp_fd, PPPIOCSMAXCID, (caddr_t) &maxcid) < 0) {
-       syslog(LOG_ERR, "ioctl(PPPIOCSFLAGS): %m");
+    if (ioctl(ttyfd, PPPIOCSMAXCID, (caddr_t) &maxcid) < 0) {
+       error("ioctl(PPPIOCSFLAGS): %m");
        return 0;
     }
     return 1;
@@ -1010,49 +728,46 @@ sifvjcomp(u, vjcomp, cidcomp, maxcid)
 /*
  * sifup - Config the interface up and enable IP packets to pass.
  */
+#ifndef SC_ENABLE_IP
+#define SC_ENABLE_IP   0x100   /* compat for old versions of kernel code */
+#endif
+
 int
 sifup(u)
     int u;
 {
     struct ifreq ifr;
+    u_int x;
     struct npioctl npi;
 
-    strncpy(ifr.ifr_name, ifname, sizeof (ifr.ifr_name));
+    strlcpy(ifr.ifr_name, ifname, sizeof (ifr.ifr_name));
     if (ioctl(sockfd, SIOCGIFFLAGS, (caddr_t) &ifr) < 0) {
-       syslog(LOG_ERR, "ioctl (SIOCGIFFLAGS): %m");
+       error("ioctl (SIOCGIFFLAGS): %m");
        return 0;
     }
     ifr.ifr_flags |= IFF_UP;
     if (ioctl(sockfd, SIOCSIFFLAGS, (caddr_t) &ifr) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFFLAGS): %m");
+       error("ioctl(SIOCSIFFLAGS): %m");
        return 0;
     }
     if_is_up = 1;
     npi.protocol = PPP_IP;
     npi.mode = NPMODE_PASS;
-    if (ioctl(ppp_fd, PPPIOCSNPMODE, &npi) < 0) {
-       syslog(LOG_ERR, "ioctl(set IP mode to PASS): %m");
-       return 0;
-    }
-    return 1;
-}
-
-/*
- * sifnpmode - Set the mode for handling packets for a given NP.
- */
-int
-sifnpmode(u, proto, mode)
-    int u;
-    int proto;
-    enum NPmode mode;
-{
-    struct npioctl npi;
-
-    npi.protocol = proto;
-    npi.mode = mode;
-    if (ioctl(ppp_fd, PPPIOCSNPMODE, &npi) < 0) {
-       syslog(LOG_ERR, "ioctl(set NP %d mode to %d): %m", proto, mode);
-       return 0;
+    if (ioctl(ttyfd, PPPIOCSNPMODE, &npi) < 0) {
+       if (errno != ENOTTY) {
+           error("ioctl(PPPIOCSNPMODE): %m");
+           return 0;
+       }
+       /* for backwards compatibility */
+       if (ioctl(ttyfd, PPPIOCGFLAGS, (caddr_t) &x) < 0) {
+           error("ioctl (PPPIOCGFLAGS): %m");
+           return 0;
+       }
+       x |= SC_ENABLE_IP;
+       if (ioctl(ttyfd, PPPIOCSFLAGS, (caddr_t) &x) < 0) {
+           error("ioctl(PPPIOCSFLAGS): %m");
+           return 0;
+       }
     }
     return 1;
 }
@@ -1065,23 +780,25 @@ sifdown(u)
     int u;
 {
     struct ifreq ifr;
+    u_int x;
     int rv;
     struct npioctl npi;
 
     rv = 1;
     npi.protocol = PPP_IP;
     npi.mode = NPMODE_ERROR;
-    ioctl(ppp_fd, PPPIOCSNPMODE, (caddr_t) &npi);
-    /* ignore errors, because ppp_fd might have been closed by now. */
+    ioctl(ttyfd, PPPIOCSNPMODE, (caddr_t) &npi);
+    /* ignore errors, because ttyfd might have been closed by now. */
 
-    strncpy(ifr.ifr_name, ifname, sizeof (ifr.ifr_name));
+
+    strlcpy(ifr.ifr_name, ifname, sizeof (ifr.ifr_name));
     if (ioctl(sockfd, SIOCGIFFLAGS, (caddr_t) &ifr) < 0) {
-       syslog(LOG_ERR, "ioctl (SIOCGIFFLAGS): %m");
+       error("ioctl (SIOCGIFFLAGS): %m");
        rv = 0;
     } else {
        ifr.ifr_flags &= ~IFF_UP;
        if (ioctl(sockfd, SIOCSIFFLAGS, (caddr_t) &ifr) < 0) {
-           syslog(LOG_ERR, "ioctl(SIOCSIFFLAGS): %m");
+           error("ioctl(SIOCSIFFLAGS): %m");
            rv = 0;
        } else
            if_is_up = 0;
@@ -1109,28 +826,26 @@ sifaddr(u, o, h, m)
     struct ifreq ifr;
 
     ret = 1;
-    strncpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name));
+    strlcpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name));
     SET_SA_FAMILY(ifr.ifr_addr, AF_INET);
-    if (m != 0) {
-       ((struct sockaddr_in *) &ifr.ifr_addr)->sin_addr.s_addr = m;
-       syslog(LOG_INFO, "Setting interface mask to %s\n", ip_ntoa(m));
-       if (ioctl(sockfd, SIOCSIFNETMASK, (caddr_t) &ifr) < 0) {
-           syslog(LOG_ERR, "ioctl(SIOCSIFNETMASK): %m");
-           ret = 0;
-       }
-    }
     ((struct sockaddr_in *) &ifr.ifr_addr)->sin_addr.s_addr = o;
     if (ioctl(sockfd, SIOCSIFADDR, (caddr_t) &ifr) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCAIFADDR): %m");
+       error("ioctl(SIOCAIFADDR): %m");
        ret = 0;
     }
     ((struct sockaddr_in *) &ifr.ifr_dstaddr)->sin_addr.s_addr = h;
     if (ioctl(sockfd, SIOCSIFDSTADDR, (caddr_t) &ifr) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFDSTADDR): %m");
+       error("ioctl(SIOCSIFDSTADDR): %m");
        ret = 0;
     }
-    ifaddrs[0] = o;
-    ifaddrs[1] = h;
+    if (m != 0) {
+       ((struct sockaddr_in *) &ifr.ifr_addr)->sin_addr.s_addr = m;
+       info("Setting interface mask to %s\n", ip_ntoa(m));
+       if (ioctl(sockfd, SIOCSIFNETMASK, (caddr_t) &ifr) < 0) {
+           error("ioctl(SIOCSIFNETMASK): %m");
+           ret = 0;
+       }
+    }
     return ret;
 }
 
@@ -1153,15 +868,13 @@ cifaddr(u, o, h)
     h = o = 0L;
     (void) sifaddr(u, o, h, 0L);
 #endif
-    ifaddrs[0] = 0;
-    BZERO(&rt, sizeof(rt));
     SET_SA_FAMILY(rt.rt_dst, AF_INET);
     ((struct sockaddr_in *) &rt.rt_dst)->sin_addr.s_addr = h;
     SET_SA_FAMILY(rt.rt_gateway, AF_INET);
     ((struct sockaddr_in *) &rt.rt_gateway)->sin_addr.s_addr = o;
     rt.rt_flags = RTF_HOST;
     if (ioctl(sockfd, SIOCDELRT, (caddr_t) &rt) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCDELRT): %m");
+       error("ioctl(SIOCDELRT): %m");
        return 0;
     }
     return 1;
@@ -1171,9 +884,9 @@ cifaddr(u, o, h)
  * sifdefaultroute - assign a default route through the address given.
  */
 int
-sifdefaultroute(u, g)
+sifdefaultroute(u, l, g)
     int u;
-    u_int32_t g;
+    u_int32_t l, g;
 {
     return dodefaultroute(g, 's');
 }
@@ -1182,9 +895,9 @@ sifdefaultroute(u, g)
  * cifdefaultroute - delete a default route through the address given.
  */
 int
-cifdefaultroute(u, g)
+cifdefaultroute(u, l, g)
     int u;
-    u_int32_t g;
+    u_int32_t l, g;
 {
     return dodefaultroute(g, 'c');
 }
@@ -1199,14 +912,13 @@ dodefaultroute(g, cmd)
 {
     struct rtentry rt;
 
-    BZERO(&rt, sizeof(rt));
     SET_SA_FAMILY(rt.rt_dst, AF_INET);
     ((struct sockaddr_in *) &rt.rt_dst)->sin_addr.s_addr = 0L;
     SET_SA_FAMILY(rt.rt_gateway, AF_INET);
     ((struct sockaddr_in *) &rt.rt_gateway)->sin_addr.s_addr = g;
     rt.rt_flags = RTF_GATEWAY;
     if (ioctl(sockfd, (cmd == 's') ? SIOCADDRT : SIOCDELRT, &rt) < 0) {
-       syslog(LOG_ERR, "%cifdefaultroute: ioctl(%s): %m", cmd,
+       error("%cifdefaultroute: ioctl(%s): %m", cmd,
               (cmd == 's') ? "SIOCADDRT" : "SIOCDELRT");
        return 0;
     }
@@ -1231,7 +943,7 @@ sifproxyarp(unit, hisaddr)
      * as our local address.
      */
     if (!get_ether_addr(hisaddr, &arpreq.arp_ha)) {
-       syslog(LOG_ERR, "Cannot determine ethernet address for proxy ARP");
+       error("Cannot determine ethernet address for proxy ARP");
        return 0;
     }
 
@@ -1239,7 +951,7 @@ sifproxyarp(unit, hisaddr)
     ((struct sockaddr_in *) &arpreq.arp_pa)->sin_addr.s_addr = hisaddr;
     arpreq.arp_flags = ATF_PERM | ATF_PUBL;
     if (ioctl(sockfd, SIOCSARP, (caddr_t)&arpreq) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSARP): %m");
+       error("ioctl(SIOCSARP): %m");
        return 0;
     }
 
@@ -1261,7 +973,7 @@ cifproxyarp(unit, hisaddr)
     SET_SA_FAMILY(arpreq.arp_pa, AF_INET);
     ((struct sockaddr_in *) &arpreq.arp_pa)->sin_addr.s_addr = hisaddr;
     if (ioctl(sockfd, SIOCDARP, (caddr_t)&arpreq) < 0) {
-       syslog(LOG_WARNING, "ioctl(SIOCDARP): %m");
+       warn("ioctl(SIOCDARP): %m");
        return 0;
     }
     proxy_arp_addr = 0;
@@ -1290,7 +1002,7 @@ get_ether_addr(ipaddr, hwaddr)
     ifc.ifc_len = sizeof(ifs);
     ifc.ifc_req = ifs;
     if (ioctl(sockfd, SIOCGIFCONF, &ifc) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCGIFCONF): %m");
+       error("ioctl(SIOCGIFCONF): %m");
        return 0;
     }
 
@@ -1303,7 +1015,7 @@ get_ether_addr(ipaddr, hwaddr)
                ((char *)&ifr->ifr_addr + sizeof(struct sockaddr))) {
        if (ifr->ifr_addr.sa_family == AF_INET) {
            ina = ((struct sockaddr_in *) &ifr->ifr_addr)->sin_addr.s_addr;
-           strncpy(ifreq.ifr_name, ifr->ifr_name, sizeof(ifreq.ifr_name));
+           strlcpy(ifreq.ifr_name, ifr->ifr_name, sizeof(ifreq.ifr_name));
            /*
             * Check that the interface is up, and not point-to-point
             * or loopback.
@@ -1329,7 +1041,7 @@ get_ether_addr(ipaddr, hwaddr)
 
     if (ifr >= ifend)
        return 0;
-    syslog(LOG_INFO, "found interface %s for proxy arp", ifr->ifr_name);
+    info("found interface %s for proxy arp", ifr->ifr_name);
 
     /*
      * Get the hostname and look for an entry using the ethers database.
@@ -1339,7 +1051,7 @@ get_ether_addr(ipaddr, hwaddr)
        return 0;
 
     if (ether_by_host(hostent->h_name, &dla)) {
-       syslog(LOG_INFO, "Add entry for %s in /etc/ethers", hostent->h_name);
+       info("Add entry for %s in /etc/ethers", hostent->h_name);
        return 0;       /* it's not there */
     }
     hwaddr->sa_family = AF_UNSPEC;
@@ -1365,7 +1077,7 @@ ether_by_host(hostname, etherptr)
      * find the address in the
      * top domain of netinfo.
      */
-    strcat(strcpy(path, "/machines/"), hostname);
+    slprintf(path, sizeof(path), "/machines/%s", hostname);
 
     if (ni_open((void *)0, "/", &conn)
      || ni_root(conn, &root)
@@ -1376,7 +1088,7 @@ ether_by_host(hostname, etherptr)
     /*
      * Now we can convert the returned string into an ethernet address.
      */
-    strcpy(path, val.ni_namelist_val[0]);
+    strlcpy(path, val.ni_namelist_val[0], sizeof(path));
     ni_free(conn);
     if ((thisptr = (struct ether_addr*)ether_aton(path)) == NULL)
        return 1;
@@ -1419,7 +1131,7 @@ GetMask(addr)
     ifc.ifc_len = sizeof(ifs);
     ifc.ifc_req = ifs;
     if (ioctl(sockfd, SIOCGIFCONF, &ifc) < 0) {
-       syslog(LOG_WARNING, "ioctl(SIOCGIFCONF): %m");
+       warn("ioctl(SIOCGIFCONF): %m");
        return mask;
     }
     ifend = (struct ifreq *) (ifc.ifc_buf + ifc.ifc_len);
@@ -1436,7 +1148,7 @@ GetMask(addr)
        /*
         * Check that the interface is up, and not point-to-point or loopback.
         */
-       strncpy(ifreq.ifr_name, ifr->ifr_name, sizeof(ifreq.ifr_name));
+       strlcpy(ifreq.ifr_name, ifr->ifr_name, sizeof(ifreq.ifr_name));
        if (ioctl(sockfd, SIOCGIFFLAGS, &ifreq) < 0)
            continue;
        if ((ifreq.ifr_flags & (IFF_UP|IFF_POINTOPOINT|IFF_LOOPBACK))
@@ -1453,8 +1165,18 @@ GetMask(addr)
     return mask;
 }
 
+/*
+ * have_route_to - determine if the system has any route to
+ * a given IP address.
+ * For demand mode to work properly, we have to ignore routes
+ * through our own interface.
+ */
+int have_route_to(u_int32_t addr)
+{
+    return -1;
+}
 
-
+#if 0
 /*
  * daemon - Detach us from the terminal session.
  */
@@ -1478,7 +1200,7 @@ daemon(nochdir, noclose)
     }
     return 0;
 }
-
+#endif
 
 char *
 strdup(s)
@@ -1511,9 +1233,9 @@ strdup(s)
 
 #define WTMPFILE        "/usr/adm/wtmp"
 
-int
+void
 logwtmp(line, name, host)
-    char *line, *name, *host;
+    const char *line, *name, *host;
 {
     int fd;
     struct stat buf;
@@ -1522,9 +1244,9 @@ logwtmp(line, name, host)
     if ((fd = open(WTMPFILE, O_WRONLY|O_APPEND, 0)) < 0)
        return;
     if (!fstat(fd, &buf)) {
-       (void)strncpy(ut.ut_line, line, sizeof(ut.ut_line));
-       (void)strncpy(ut.ut_name, name, sizeof(ut.ut_name));
-       (void)strncpy(ut.ut_host, host, sizeof(ut.ut_host));
+       strncpy(ut.ut_line, line, sizeof(ut.ut_line));
+       strncpy(ut.ut_name, name, sizeof(ut.ut_name));
+       strncpy(ut.ut_host, host, sizeof(ut.ut_host));
        (void)time(&ut.ut_time);
        if (write(fd, (char *)&ut, sizeof(struct utmp)) != sizeof(struct utmp))
            (void)ftruncate(fd, buf.st_size);
@@ -1532,6 +1254,7 @@ logwtmp(line, name, host)
     close(fd);
 }
 
+#if 0
 /*
  * Routines for locking and unlocking the serial device, moved here
  * from chat.c.
@@ -1539,6 +1262,8 @@ logwtmp(line, name, host)
 
 #define LOCK_PREFIX    "/usr/spool/uucp/LCK/LCK.."
 
+static char *lock_file;
+
 /*
  * lock - create a lock file for the named device
  */
@@ -1548,13 +1273,15 @@ lock(dev)
 {
     int fd, pid, n;
     char *p;
+    size_t l;
 
     if ((p = strrchr(dev, '/')) != NULL)
        dev = p + 1;
-    lock_file = malloc(strlen(LOCK_PREFIX) + strlen(dev) + 1);
+    l = strlen(LOCK_PREFIX) + strlen(dev) + 1;
+    lock_file = malloc(l);
     if (lock_file == NULL)
        novm("lock file name");
-    strcat(strcpy(lock_file, LOCK_PREFIX), dev);
+    slprintf(lock_file, l, "%s%s", LOCK_PREFIX, dev);
 
     while ((fd = open(lock_file, O_EXCL | O_CREAT | O_RDWR, 0644)) < 0) {
        if (errno == EEXIST
@@ -1562,26 +1289,25 @@ lock(dev)
            /* Read the lock file to find out who has the device locked */
            n = read(fd, &pid, sizeof(pid));
            if (n <= 0) {
-               syslog(LOG_ERR, "Can't read pid from lock file %s", lock_file);
+               error("Can't read pid from lock file %s", lock_file);
                close(fd);
            } else {
                if (kill(pid, 0) == -1 && errno == ESRCH) {
                    /* pid no longer exists - remove the lock file */
                    if (unlink(lock_file) == 0) {
                        close(fd);
-                       syslog(LOG_NOTICE, "Removed stale lock on %s (pid %d)",
+                       notice("Removed stale lock on %s (pid %d)",
                               dev, pid);
                        continue;
                    } else
-                       syslog(LOG_WARNING, "Couldn't remove stale lock on %s",
-                              dev);
+                       warn("Couldn't remove stale lock on %s", dev);
                } else
-                   syslog(LOG_NOTICE, "Device %s is locked by pid %d",
+                   notice("Device %s is locked by pid %d",
                           dev, pid);
            }
            close(fd);
        } else
-           syslog(LOG_ERR, "Can't create lock file %s: %m", lock_file);
+           error("Can't create lock file %s: %m", lock_file);
        free(lock_file);
        lock_file = NULL;
        return -1;
@@ -1606,6 +1332,7 @@ unlock()
        lock_file = NULL;
     }
 }
+#endif
 
 #if defined(i386) && defined(HAS_BROKEN_IOCTL)
 int
@@ -1625,7 +1352,7 @@ ioctl(fd, cmd, c)
     code = cmd;
 
     if (let == 't' && (75 <= code && code <= 90))
-    syslog(LOG_INFO, "ioctl(%d, 0x%x ('%c', %d, %d), 0x%x)\n", fd, cmd,
+    info("ioctl(%d, 0x%x ('%c', %d, %d), 0x%x)\n", fd, cmd,
           let, code, size, c);
 #endif
 
@@ -1634,12 +1361,12 @@ ioctl(fd, cmd, c)
 #ifdef DEBUGIOCTL
     serrno = errno;
     if (ret == -1)
-       syslog(LOG_INFO, "ioctl('%c', %d, %d) errno = %d (%m)\n",
+       info("ioctl('%c', %d, %d) errno = %d (%m)\n",
                let, code, size, errno);
     if (let == 't' && (75 <= code && code <= 90) && (cmd & IOC_OUT)) {
        int i, len = ((cmd >> 16) & IOCPARM_MASK);
        for (i = 0; i < len / 4; ++i)
-               syslog(LOG_INFO, "word[%d] @ 0x%06x = 0x%x\n",
+               info("word[%d] @ 0x%06x = 0x%x\n",
                       i, &((int *) c)[i],((int *)c)[i]);
     }
     errno = serrno;
@@ -1700,10 +1427,265 @@ int sigaction(int sig, const struct sigaction *act, struct sigaction *oact)
    if (!in)
      {
        in = 1;
-       syslog(LOG_WARNING, "PPPD: Inside modified HP and SPARC sigaction\n");
+       warn("PPPD: Inside modified HP and SPARC sigaction\n");
      }
 
    return sigvec(sig, &sv, NULL);
 }
 
 #endif
+
+
+/*
+ * Code following is added for 2.3 compatibility
+ */
+
+/*
+ * get_idle_time - return how long the link has been idle.
+ */
+int
+get_idle_time(u, ip)
+    int u;
+    struct ppp_idle *ip;
+{
+  return (ioctl(ttyfd, PPPIOCGIDLE, ip) >= 0); 
+}
+
+/*
+ * get_ppp_stats - return statistics for the link.
+ */
+int
+get_ppp_stats(u, stats)
+    int u;
+    struct pppd_stats *stats;
+{
+    struct ifpppstatsreq req;
+
+    memset (&req, 0, sizeof (req));
+    strlcpy(req.ifr_name, interface, sizeof(req.ifr_name));
+    if (ioctl(sockfd, SIOCGPPPSTATS, &req) < 0) {
+       error("Couldn't get PPP statistics: %m");
+       return 0;
+    }
+    stats->bytes_in = req.stats.p.ppp_ibytes;
+    stats->bytes_out = req.stats.p.ppp_obytes;
+    return 1;
+}
+
+
+/*
+ * get_loop_output - read characters from the loopback, form them
+ * into frames, and detect when we want to bring the real link up.
+ * Return value is 1 if we need to bring up the link, 0 otherwise.
+ */
+int
+get_loop_output()
+{
+
+#if 0
+    int rv = 0;
+    int n;
+
+    while ((n = read(loop_master, inbuf, sizeof(inbuf))) >= 0) {
+       if (loop_chars(inbuf, n))
+           rv = 1;
+    }
+
+    if (n == 0)
+       fatal("eof on loopback");
+    if (errno != EWOULDBLOCK)
+       fatal("read from loopback: %m");
+
+    return rv;
+#endif
+
+    return 0;
+}
+
+/*
+ * sifnpmode - Set the mode for handling packets for a given NP.
+ */
+int
+sifnpmode(u, proto, mode)
+    int u;
+    int proto;
+    enum NPmode mode;
+{
+    struct npioctl npi;
+
+    npi.protocol = proto;
+    npi.mode = mode;
+    if (ioctl(ttyfd, PPPIOCSNPMODE, &npi) < 0) {
+       error("ioctl(set NP %d mode to %d): %m", proto, mode);
+       return 0;
+    }
+    return 1;
+}
+
+
+/*
+ * open_ppp_loopback - open the device we use for getting
+ * packets in demand mode, and connect it to a ppp interface.
+ * Here we use a pty.
+ */
+int
+open_ppp_loopback()
+{
+
+#if 0
+    int flags;
+    struct termios tios;
+    int pppdisc = PPPDISC;
+
+    fatal("open_ppp_loopback called!");
+
+    if (openpty(&loop_master, &loop_slave, loop_name, NULL, NULL) < 0)
+       fatal("No free pty for loopback");
+    SYSDEBUG(("using %s for loopback", loop_name));
+
+    if (tcgetattr(loop_slave, &tios) == 0) {
+       tios.c_cflag &= ~(CSIZE | CSTOPB | PARENB);
+       tios.c_cflag |= CS8 | CREAD;
+       tios.c_iflag = IGNPAR;
+       tios.c_oflag = 0;
+       tios.c_lflag = 0;
+       if (tcsetattr(loop_slave, TCSAFLUSH, &tios) < 0)
+           warn("couldn't set attributes on loopback: %m");
+    }
+
+    if ((flags = fcntl(loop_master, F_GETFL)) != -1) 
+       if (fcntl(loop_master, F_SETFL, flags | O_NONBLOCK) == -1)
+           warn("couldn't set loopback to nonblock: %m");
+
+    ttyfd = loop_slave;
+    if (ioctl(ttyfd, TIOCSETD, &pppdisc) < 0)
+       fatal("ioctl(TIOCSETD): %m");
+
+    /*
+     * Find out which interface we were given.
+     */
+    if (ioctl(ttyfd, PPPIOCGUNIT, &ifunit) < 0)
+       fatal("ioctl(PPPIOCGUNIT): %m");
+
+    /*
+     * Enable debug in the driver if requested.
+     */
+    if (kdebugflag) {
+       if (ioctl(ttyfd, PPPIOCGFLAGS, (caddr_t) &flags) < 0) {
+           warn("ioctl (PPPIOCGFLAGS): %m");
+       } else {
+           flags |= (kdebugflag & 0xFF) * SC_DEBUG;
+           if (ioctl(ttyfd, PPPIOCSFLAGS, (caddr_t) &flags) < 0)
+               warn("ioctl(PPPIOCSFLAGS): %m");
+       }
+    }
+
+    return loop_master;
+#endif
+
+}
+
+/*
+ * restore_loop - reattach the ppp unit to the loopback.
+ */
+void
+restore_loop()
+{
+    int x;
+
+    /*
+     * Transfer the ppp interface back to the loopback.
+     */
+    if (ioctl(ttyfd, PPPIOCXFERUNIT, 0) < 0)
+       fatal("ioctl(transfer ppp unit): %m");
+    x = PPPDISC;
+    if (ioctl(loop_slave, TIOCSETD, &x) < 0)
+       fatal("ioctl(TIOCSETD): %m");
+
+    /*
+     * Check that we got the same unit again.
+     */
+    if (ioctl(loop_slave, PPPIOCGUNIT, &x) < 0)
+       fatal("ioctl(PPPIOCGUNIT): %m");
+    if (x != ifunit)
+       fatal("transfer_ppp failed: wanted unit %d, got %d",
+             ifunit, x);
+    ttyfd = loop_slave;
+}
+
+
+/*
+ * Use the hostid as part of the random number seed.
+ */
+int
+get_host_seed()
+{
+    return gethostid();
+}
+
+
+/*
+ * sys_check_options - check the options that the user specified
+ */
+int
+sys_check_options()
+{
+  /*
+   * We don't support demand dialing yet.
+   */
+  if (demand)
+    {
+      option_error("PPP-2.3 for NeXTSTEP does not yet support demand dialing");
+      return 0;
+    }
+  return 1;
+}
+
+
+/*
+ * sys_close - Clean up in a child process before execing.
+ */
+void
+sys_close()
+{
+    close(sockfd);
+    if (loop_slave >= 0) {
+       close(loop_slave);
+       close(loop_master);
+    }
+    closelog();
+}
+
+#if 0
+/*
+ * wait_loop_output - wait until there is data available on the
+ * loopback, for the length of time specified by *timo (indefinite
+ * if timo is NULL).
+ */
+void wait_loop_output(timo)
+    struct timeval *timo;
+{
+    fd_set ready;
+    int n;
+
+    FD_ZERO(&ready);
+    FD_SET(loop_master, &ready);
+    n = select(loop_master + 1, &ready, NULL, &ready, timo);
+    if (n < 0 && errno != EINTR)
+       fatal("select: %m");
+}
+
+/*
+ * wait_time - wait for a given length of time or until a
+ * signal is received.
+ */
+void wait_time(timo)
+    struct timeval *timo;
+{
+    int n;
+
+    n = select(0, NULL, NULL, NULL, timo);
+    if (n < 0 && errno != EINTR)
+       fatal("select: %m");
+}
+#endif