]> git.ozlabs.org Git - ppp.git/blobdiff - pppd/sys-aix4.c
Patch from Frank Cusack to add support for MSCHAPv2.
[ppp.git] / pppd / sys-aix4.c
index 76feef21e489e0111fcefc94ed014666b49633c1..c244c3ecad56385a57179cc0d8101b577c95b4e4 100644 (file)
@@ -3,6 +3,7 @@
  * PPP interfaces on AIX systems which use the STREAMS ppp interface.
  *
  * Copyright (c) 1989 Carnegie Mellon University.
+ * 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.  The name of the
- * University may not be used to endorse or promote products derived
- * from this software without specific prior written permission.
+ * by Carnegie Mellon University and The Australian National University.
+ * The names of the Universities 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-aix4.c,v 1.8 1996/01/01 23:04:13 paulus Exp $";
-#endif
+#define RCSID  "$Id: sys-aix4.c,v 1.22 1999/08/13 06:46:17 paulus Exp $"
 
 /*
  * TODO:
@@ -31,7 +31,6 @@ static char rcsid[] = "$Id: sys-aix4.c,v 1.8 1996/01/01 23:04:13 paulus Exp $";
 #include <stdlib.h>
 */
 #include <errno.h>
-#include <syslog.h>
 #include <termios.h>
 #include <sys/termiox.h>
 #include <fcntl.h>
@@ -47,13 +46,14 @@ static char rcsid[] = "$Id: sys-aix4.c,v 1.8 1996/01/01 23:04:13 paulus Exp $";
 
 #include <net/if.h>
 #include <net/ppp_defs.h>
-#include <net/ppp_str.h>
 #include <net/route.h>
 #include <net/if_arp.h>
 #include <netinet/in.h>
 
 #include "pppd.h"
 
+static const char rcsid[] = RCSID;
+
 #ifndef ifr_mtu
 #define ifr_mtu                ifr_metric
 #endif
@@ -73,9 +73,14 @@ static int   initfdflags = -1; /* Initial file descriptor flags for fd */
 static int sockfd;             /* socket for doing interface ioctls */
 
 static int     if_is_up;       /* Interface has been marked up */
+static u_int32_t ifaddrs[2];   /* local and remote addresses */
 static u_int32_t default_route_gateway;        /* Gateway for default route added */
 static u_int32_t proxy_arp_addr;       /* Addr for proxy arp entry added */
 
+#define MAX_POLLFDS    32
+static struct pollfd pollfds[MAX_POLLFDS];
+static int n_pollfds;
+
 /* Prototypes for procedures local to this file. */
 static int translate_speed __P((int));
 static int baud_rate_of __P((int));
@@ -88,16 +93,11 @@ static int get_ether_addr __P((u_int32_t, struct sockaddr *));
 void
 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");
+
+    n_pollfds = 0;
 }
 
 /*
@@ -111,34 +111,33 @@ 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);
 }
 
 /*
- * note_debug_level - note a change in the debug level.
+ * 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.
  */
-void
-note_debug_level()
+int have_route_to(u_int32_t addr)
 {
-    if (debug) {
-       syslog(LOG_INFO, "Debug turned ON, Level %d", debug);
-       setlogmask(LOG_UPTO(LOG_DEBUG));
-    } else {
-       setlogmask(LOG_UPTO(LOG_WARNING));
-    }
+    return -1;
 }
 
+#if 0
 /*
  * daemon - Detach us from the terminal session.
  */
@@ -162,7 +161,7 @@ daemon(nochdir, noclose)
     }
     return 0;
 }
-
+#endif
 
 /*
  * ppp_available - check if this kernel supports PPP.
@@ -184,7 +183,7 @@ ppp_available()
 /*
  * establish_ppp - Turn the serial port into a ppp interface.
  */
-void
+int
 establish_ppp(fd)
     int fd;
 {
@@ -193,45 +192,37 @@ establish_ppp(fd)
        if (ioctl(fd, I_LOOK, str_modules[str_module_count].modname) < 0 ||
            ioctl(fd, I_POP, 0) < 0)
            break;
-       MAINDEBUG((LOG_DEBUG, "popped stream module : %s",
+       SYSDEBUG(("popped stream module : %s",
                   str_modules[str_module_count].modname));
        str_module_count++;
     }
 
     /* now push the async/fcs module */
-    if (ioctl(fd, I_PUSH, "pppasync") < 0) {
-       syslog(LOG_ERR, "ioctl(I_PUSH, ppp_async): %m");
-       die(1);
-    }
+    if (ioctl(fd, I_PUSH, "pppasync") < 0)
+       fatal("ioctl(I_PUSH, ppp_async): %m");
     /* push the compress module */
     if (ioctl(fd, I_PUSH, "pppcomp") < 0) {
-       syslog(LOG_WARNING, "ioctl(I_PUSH, ppp_comp): %m");
+       warn("ioctl(I_PUSH, ppp_comp): %m");
     }
     /* finally, push the ppp_if module that actually handles the */
     /* network interface */ 
-    if (ioctl(fd, I_PUSH, "pppif") < 0) {
-       syslog(LOG_ERR, "ioctl(I_PUSH, ppp_if): %m");
-       die(1);
-    }
+    if (ioctl(fd, I_PUSH, "pppif") < 0)
+       fatal("ioctl(I_PUSH, ppp_if): %m");
     pushed_ppp = 1;
     /* read mode, message non-discard mode
-    if (ioctl(fd, I_SRDOPT, RMSGN) < 0) {
-       syslog(LOG_ERR, "ioctl(I_SRDOPT, RMSGN): %m");
-       die(1);
-    }
+    if (ioctl(fd, I_SRDOPT, RMSGN) < 0)
+       fatal("ioctl(I_SRDOPT, RMSGN): %m");
 */
     /*
      * Find out which interface we were given.
      * (ppp_if handles this ioctl)
      */
-    if (ioctl(fd, SIOCGETU, &ifunit) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCGETU): %m");
-       die(1);
-    }
+    if (ioctl(fd, SIOCGETU, &ifunit) < 0)
+       fatal("ioctl(SIOCGETU): %m");
 
     /* Set debug flags in driver */
     if (ioctl(fd, SIOCSIFDEBUG, kdebugflag) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFDEBUG): %m");
+       error("ioctl(SIOCSIFDEBUG): %m");
     }
 
     /* close stdin, stdout, stderr if they might refer to the device */
@@ -249,8 +240,10 @@ establish_ppp(fd)
      */
     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");
     }
+
+    return fd;
 }
 
 /*
@@ -267,7 +260,7 @@ disestablish_ppp(fd)
 
     /* Reset non-blocking mode on the file descriptor. */
     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;
 
     if (hungup) {
@@ -298,8 +291,8 @@ disestablish_ppp(fd)
                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);
            }
        }
     }
@@ -311,11 +304,11 @@ disestablish_ppp(fd)
     for (; str_module_count > 0; str_module_count--) {
        if (ioctl(fd, I_PUSH, str_modules[str_module_count-1].modname)) {
            if (errno != ENXIO)
-               syslog(LOG_WARNING, "str_restore: couldn't push module %s: %m",
-                      str_modules[str_module_count-1].modname);
+               warn("str_restore: couldn't push module %s: %m",
+                    str_modules[str_module_count-1].modname);
        } else {
-           MAINDEBUG((LOG_INFO, "str_restore: pushed module %s",
-                      str_modules[str_module_count-1].modname));
+           SYSDEBUG(("str_restore: pushed module %s",
+                     str_modules[str_module_count-1].modname));
        }
     }
 }
@@ -410,7 +403,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;
 }
 
@@ -444,10 +437,9 @@ set_up_tty(fd, local)
     struct termios tios;
     struct termiox tiox;
 
-    if (tcgetattr(fd, &tios) < 0) {
-       syslog(LOG_ERR, "tcgetattr: %m");
-       die(1);
-    }
+    if (tcgetattr(fd, &tios) < 0)
+       fatal("tcgetattr: %m");
+
 
     if (!restore_term)
        inittermios = tios;
@@ -492,17 +484,12 @@ set_up_tty(fd, local)
         * We can't proceed if the serial port speed is B0,
         * since that implies that the serial port is disabled.
         */
-       if (speed == B0) {
-           syslog(LOG_ERR, "Baud rate for %s is 0; need explicit baud rate",
-                  devnam);
-           die(1);
-       }
+       if (speed == B0)
+           fatal("Baud rate for %s is 0; need explicit baud rate", devnam);
     }
 
-    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;
@@ -527,7 +514,7 @@ restore_tty(fd)
        }
        if (tcsetattr(fd, TCSAFLUSH, &inittermios) < 0)
            if (errno != ENXIO)
-               syslog(LOG_WARNING, "tcsetattr: %m");
+               warn("tcsetattr: %m");
        restore_term = 0;
     }
 }
@@ -560,7 +547,7 @@ output(unit, p, len)
     struct pollfd pfd;
 
     if (debug)
-       log_packet(p, len, "sent ");
+       dbglog("sent %P", p, len);
 
     str.len = len;
     str.buf = (caddr_t) p;
@@ -568,7 +555,7 @@ output(unit, p, len)
     while (putmsg(ttyfd, NULL, &str, 0) < 0) {
        if (--retries < 0 || (errno != EWOULDBLOCK && errno != EAGAIN)) {
            if (errno != ENXIO)
-               syslog(LOG_ERR, "Couldn't send packet: %m");
+               error("Couldn't send packet: %m");
            break;
        }
        pfd.fd = ttyfd;
@@ -585,17 +572,50 @@ wait_input(timo)
     struct timeval *timo;
 {
     int t;
-    struct pollfd pfd;
 
     t = timo == NULL? -1: timo->tv_sec * 1000 + timo->tv_usec / 1000;
-    pfd.fd = ttyfd;
-    pfd.events = POLLIN | POLLPRI | POLLHUP;
-    if (poll(&pfd, 1, t) < 0 && errno != EINTR) {
-       syslog(LOG_ERR, "poll: %m");
-       die(1);
+    if (poll(pollfds, n_pollfds, t) < 0 && errno != EINTR)
+       fatal("poll: %m");
+}
+
+/*
+ * add_fd - add an fd to the set that wait_input waits for.
+ */
+void add_fd(fd)
+    int fd;
+{
+    int n;
+
+    for (n = 0; n < n_pollfds; ++n)
+       if (pollfds[n].fd == fd)
+           return;
+    if (n_pollfds < MAX_POLLFDS) {
+       pollfds[n_pollfds].fd = fd;
+       pollfds[n_pollfds].events = POLLIN | POLLPRI | POLLHUP;
+       ++n_pollfds;
+    } else
+       error("Too many inputs!");
+}
+
+/*
+ * remove_fd - remove an fd from the set that wait_input waits for.
+ */
+void remove_fd(fd)
+    int fd;
+{
+    int n;
+
+    for (n = 0; n < n_pollfds; ++n) {
+       if (pollfds[n].fd == fd) {
+           while (++n < n_pollfds)
+               pollfds[n-1] = pollfds[n];
+           --n_pollfds;
+           break;
+       }
     }
 }
 
+
 /*
  * read_packet - get a PPP packet from the serial device.
  */
@@ -617,17 +637,15 @@ read_packet(buf)
        if (errno == EAGAIN || errno == EWOULDBLOCK || errno == EINTR) {
            return -1;
        }
-       syslog(LOG_ERR, "getmsg %m");
-       die(1);
+       fatal("getmsg %m");
     }
     if (len) 
-       MAINDEBUG((LOG_DEBUG, "getmsg returned 0x%x",len));
+       SYSDEBUG(("getmsg returned 0x%x", len));
     if (ctl.len > 0)
-       syslog(LOG_NOTICE, "got ctrl msg len %d %x %x\n", ctl.len,
-              ctlbuf[0], ctlbuf[1]);
+       notice("got ctrl msg len %d %x %x\n", ctl.len, ctlbuf[0], ctlbuf[1]);
 
     if (str.len < 0) {
-       MAINDEBUG((LOG_DEBUG, "getmsg short return length %d", str.len));
+       SYSDEBUG(("getmsg short return length %d", str.len));
        return -1;
     }
 
@@ -648,29 +666,21 @@ ppp_send_config(unit, mtu, asyncmap, pcomp, accomp)
     int c;
     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(ttyfd, SIOCSIFASYNCMAP, asyncmap) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFASYNCMAP): %m");
-       quit();
-    }
+    if(ioctl(ttyfd, SIOCSIFASYNCMAP, asyncmap) < 0)
+       fatal("ioctl(SIOCSIFASYNCMAP): %m");
 
     c = (pcomp? 1: 0);
-    if(ioctl(ttyfd, SIOCSIFCOMPPROT, c) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFCOMPPROT): %m");
-       quit();
-    }
+    if(ioctl(ttyfd, SIOCSIFCOMPPROT, c) < 0)
+       fatal("ioctl(SIOCSIFCOMPPROT): %m");
 
     c = (accomp? 1: 0);
-    if(ioctl(ttyfd, SIOCSIFCOMPAC, c) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFCOMPAC): %m");
-       quit();
-    }
+    if(ioctl(ttyfd, SIOCSIFCOMPAC, c) < 0)
+       fatal("ioctl(SIOCSIFCOMPAC): %m");
 }
 
 
@@ -683,7 +693,7 @@ ppp_set_xaccm(unit, accm)
     ext_accm accm;
 {
     if (ioctl(ttyfd, SIOCSIFXASYNCMAP, accm) < 0 && errno != ENOTTY)
-       syslog(LOG_WARNING, "ioctl(set extended ACCM): %m");
+       warn("ioctl(set extended ACCM): %m");
 }
 
 
@@ -700,21 +710,21 @@ ppp_recv_config(unit, mru, asyncmap, pcomp, accomp)
     char c;
 
     if (ioctl(ttyfd, SIOCSIFMRU, mru) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFMRU): %m");
+       error("ioctl(SIOCSIFMRU): %m");
     }
 
     if (ioctl(ttyfd, SIOCSIFRASYNCMAP, (caddr_t) asyncmap) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFRASYNCMAP): %m");
+       error("ioctl(SIOCSIFRASYNCMAP): %m");
     }
 
     c = 2 + (pcomp? 1: 0);
     if(ioctl(ttyfd, SIOCSIFCOMPPROT, c) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFCOMPPROT): %m");
+       error("ioctl(SIOCSIFCOMPPROT): %m");
     }
 
     c = 2 + (accomp? 1: 0);
     if (ioctl(ttyfd, SIOCSIFCOMPAC, c) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFCOMPAC): %m");
+       error("ioctl(SIOCSIFCOMPAC): %m");
     }
 }
 
@@ -750,7 +760,7 @@ ccp_flags_set(unit, isopen, isup)
 
     x = (isopen? 1: 0) + (isup? 2: 0);
     if (ioctl(ttyfd, SIOCSIFCOMP, x) < 0 && errno != ENOTTY)
-       syslog(LOG_ERR, "ioctl (SIOCSIFCOMP): %m");
+       error("ioctl (SIOCSIFCOMP): %m");
 }
 
 /*
@@ -765,7 +775,7 @@ ccp_fatal_error(unit)
     int x;
 
     if (ioctl(ttyfd, SIOCGIFCOMP, &x) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCGIFCOMP): %m");
+       error("ioctl(SIOCGIFCOMP): %m");
        return 0;
     }
     return x & CCP_FATALERROR;
@@ -782,7 +792,7 @@ sifvjcomp(u, vjcomp, cidcomp, maxcid)
 
     x = (vjcomp? 1: 0) + (cidcomp? 0: 2) + (maxcid << 4);
     if (ioctl(ttyfd, SIOCSIFVJCOMP, x) < 0) {
-       syslog(LOG_ERR, "ioctl(SIOCSIFVJCOMP): %m");
+       error("ioctl(SIOCSIFVJCOMP): %m");
        return 0;
     }
     return 1;
@@ -796,28 +806,39 @@ sifup(u)
     int u;
 {
     struct ifreq ifr;
-    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(ttyfd, SIOCSETNPMODE, &npi) < 0) {
-        if (errno != ENOTTY) {
-            syslog(LOG_ERR, "ioctl(SIOCSETNPMODE): %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) {
+       error("ioctl(set NP %d mode to %d): %m", proto, mode);
+       return 0;
+    }
     return 1;
 }
 
@@ -835,21 +856,17 @@ sifdown(u)
     rv = 1;
     npi.protocol = PPP_IP;
     npi.mode = NPMODE_ERROR;
-    if (ioctl(ttyfd, SIOCSETNPMODE, &npi) < 0) {
-        if (errno != ENOTTY) {
-            syslog(LOG_ERR, "ioctl(SIOCSETNPMODE): %m");
-            rv = 0;
-        }
-    }
+    ioctl(ttyfd, SIOCSETNPMODE, &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;
@@ -876,34 +893,39 @@ 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) {
-        syslog(LOG_INFO, "Setting interface mask to %s\n", ip_ntoa(m));
+        info("Setting interface mask to %s\n", ip_ntoa(m));
         ((struct sockaddr_in *) &ifr.ifr_addr)->sin_addr.s_addr = m;
         if (ioctl(sockfd, SIOCSIFNETMASK, (caddr_t) &ifr) < 0) {
-            syslog(LOG_ERR, "ioctl(SIOCSIFNETMASK): %m");
+            error("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(SIOCSIFADDR): %m");
+        error("ioctl(SIOCSIFADDR): %m");
         ret = 0;
     }
 
     SET_SA_FAMILY(ifr.ifr_dstaddr, AF_INET);
     ((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;
     }
+
+    /* XXX is this necessary? */
     ((struct sockaddr_in *) &ifr.ifr_addr)->sin_addr.s_addr = o;
     if (ioctl(sockfd, SIOCSIFADDR, (caddr_t) &ifr) < 0) {
-        syslog(LOG_ERR, "ioctl(SIOCSIFADDR): %m");
+        error("ioctl(SIOCSIFADDR): %m");
         ret = 0;
     }
+
+    ifaddrs[0] = o;
+    ifaddrs[1] = h;
     return ret;
 }
 
@@ -918,13 +940,15 @@ cifaddr(u, o, h)
 {
     struct ortentry rt;
 
+    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;
@@ -934,18 +958,19 @@ 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;
 {
     struct ortentry rt;
 
+    BZERO(&rt, sizeof(rt));
     SET_SA_FAMILY(rt.rt_dst, AF_INET);
     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, SIOCADDRT, &rt) < 0) {
-       syslog(LOG_ERR, "default route ioctl(SIOCADDRT): %m");
+       error("default route ioctl(SIOCADDRT): %m");
        return 0;
     }
     default_route_gateway = g;
@@ -956,18 +981,19 @@ 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;
 {
     struct ortentry rt;
 
+    BZERO(&rt, sizeof(rt));
     SET_SA_FAMILY(rt.rt_dst, AF_INET);
     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, SIOCDELRT, &rt) < 0) {
-       syslog(LOG_ERR, "default route ioctl(SIOCDELRT): %m");
+       error("default route ioctl(SIOCDELRT): %m");
        return 0;
     }
     default_route_gateway = 0;
@@ -991,7 +1017,7 @@ sifproxyarp(unit, hisaddr)
      * as our local address.
      */
     if (!get_ether_addr(hisaddr, &arpreq.arp_ha)) {
-       syslog(LOG_WARNING, "Cannot determine ethernet address for proxy ARP");
+       warn("Cannot determine ethernet address for proxy ARP");
        return 0;
     }
 
@@ -999,7 +1025,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;
     }
 
@@ -1021,7 +1047,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_ERR, "ioctl(SIOCDARP): %m");
+       error("ioctl(SIOCDARP): %m");
        return 0;
     }
     proxy_arp_addr = 0;
@@ -1074,7 +1100,7 @@ static struct nlist nl[] = {
 int kvm_read(int fd, off_t offset, void *buf, int nbytes)
 {
     if (lseek(fd, offset, SEEK_SET) != offset) {
-        syslog(LOG_ERR,"lseek in kmem: %m");
+        error("lseek in kmem: %m");
         return(0);
     }
     return(read(fd, buf, nbytes));
@@ -1106,13 +1132,13 @@ get_ether_addr(ipaddr, hwaddr)
 
 /* Open kernel memory for reading */
     if ((kd = open("/dev/kmem", O_RDONLY)) < 0) {
-        syslog(LOG_ERR, "/dev/kmem: %m");
+        error("/dev/kmem: %m");
         return 0;
     }
 
 /* Fetch namelist */
     if (nlist("/unix", nl) != 0) {
-        syslog(LOG_ERR, "nlist(): %m");
+        error("nlist(): %m");
         return 0;
     }
 
@@ -1123,12 +1149,12 @@ get_ether_addr(ipaddr, hwaddr)
 
     if (kvm_read(kd, nl[N_IFNET].n_value, (char *)&addr, sizeof(addr))
         != sizeof(addr)) {
-        syslog(LOG_ERR, "error reading ifnet addr");
+        error("error reading ifnet addr");
         return 0;
     }
     for ( ; addr && !found; addr = (u_long)ifp->if_next) {
         if (kvm_read(kd, addr, (char *)ac, sizeof(*ac)) != sizeof(*ac)) {
-            syslog(LOG_ERR, "error reading ifnet");
+            error("error reading ifnet");
             return 0;
         }
 
@@ -1143,17 +1169,17 @@ get_ether_addr(ipaddr, hwaddr)
         for (ifap = ifp->if_addrlist; ifap; ifap=ifaddr.ifa.ifa_next) {
             if (kvm_read(kd, (u_long)ifap, (char *)&ifaddr,
                      sizeof(ifaddr)) != sizeof(ifaddr)) {
-                syslog(LOG_ERR, "error reading ifaddr");
+                error("error reading ifaddr");
                 return 0;
             }
             if (kvm_read(kd, (u_long)ifaddr.ifa.ifa_addr, &ifaddrsaddr,
                 sizeof(struct sockaddr_in)) != sizeof(struct sockaddr_in)) {
-                syslog(LOG_ERR, "error reading ifaddrsaddr");
+                error("error reading ifaddrsaddr");
                 return(0);
             }
             if (kvm_read(kd, (u_long)ifaddr.ifa.ifa_netmask, &ifmasksaddr,
                 sizeof(struct sockaddr_in)) != sizeof(struct sockaddr_in)) {
-                syslog(LOG_ERR, "error reading ifmasksaddr");
+                error("error reading ifmasksaddr");
                 return(0);
             }
     /* Check if this interface on the right subnet */
@@ -1168,7 +1194,7 @@ get_ether_addr(ipaddr, hwaddr)
                 ifip = ifinetptr->sin_addr.s_addr;
                 if (kvm_read(kd, (u_long)ifaddr.ifa.ifa_netmask, &ifmasksaddr,
                     sizeof(struct sockaddr_in)) != sizeof(struct sockaddr_in)) {
-                    syslog(LOG_ERR, "error reading ifmasksaddr");
+                    error("error reading ifmasksaddr");
                     return(0);
                 }
                 mask = ifmasksaddr.sin_addr.s_addr;
@@ -1219,7 +1245,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);
@@ -1235,7 +1261,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))
@@ -1254,9 +1280,9 @@ GetMask(addr)
 
 #define        WTMPFILE        "/var/adm/wtmp"
 
-int
+void
 logwtmp(line, name, host)
-    char *line, *name, *host;
+    const char *line, *name, *host;
 {
     int fd;
     struct stat buf;
@@ -1265,9 +1291,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);
@@ -1275,6 +1301,15 @@ logwtmp(line, name, host)
     close(fd);
 }
 
+/*
+ * Use the hostid as part of the random number seed.
+ */
+int
+get_host_seed()
+{
+    return gethostid();
+}
+
 /*
  * Code for locking/unlocking the serial device.
  */
@@ -1292,8 +1327,7 @@ int lock(char *device)
        devname = device;
 
     if ((rc = ttylock(devname)) == 0) {
-       devlocked = (char *) malloc(strlen(devname) + 1);
-       sprintf(devlocked,"%s",devname);
+       devlocked = strdup(devname);
     } else
        devlocked = (char *) 0;