]> git.ozlabs.org Git - ppp.git/blobdiff - pppd/auth.c
get crypt decl; mods to shadow password stuff
[ppp.git] / pppd / auth.c
index 6685ea91d00636dec78059522f2949d71389d226..2faaf2d190afb27f45503c13a0f42c9f69a10809 100644 (file)
  */
 
 #ifndef lint
-static char rcsid[] = "$Id: auth.c,v 1.4 1994/05/18 05:59:43 paulus Exp $";
+static char rcsid[] = "$Id: auth.c,v 1.23 1996/05/28 00:47:01 paulus Exp $";
 #endif
 
 #include <stdio.h>
 #include <stddef.h>
+#include <stdlib.h>
+#include <unistd.h>
 #include <syslog.h>
 #include <pwd.h>
 #include <string.h>
 #include <sys/types.h>
 #include <sys/stat.h>
+#include <sys/socket.h>
 
 #include <netdb.h>
 #include <netinet/in.h>
 #include <arpa/inet.h>
 
-#include "ppp.h"
+#ifdef SVR4
+#include <crypt.h>
+#else
+#ifdef SUNOS4
+extern char *crypt();
+#endif
+#endif
+
+#ifdef HAS_SHADOW
+#include <shadow.h>
+#include <shadow/pwauth.h>
+#ifndef PW_PPP
+#define PW_PPP PW_LOGIN
+#endif
+#endif
+
 #include "pppd.h"
 #include "fsm.h"
 #include "lcp.h"
+#include "ipcp.h"
 #include "upap.h"
 #include "chap.h"
-#include "ipcp.h"
 #include "pathnames.h"
 
-#ifdef sparc
+#if defined(sun) && defined(sparc)
 #include <alloca.h>
-#ifndef __GNUC__
-/* why alloca.h doesn't define what alloca() returns is a mystery */
-/* char *alloca __ARGS((int)); */
-#endif /*__GNUC__*/
 #endif /*sparc*/
 
 /* Used for storing a sequence of words.  Usually malloced. */
@@ -80,20 +94,20 @@ struct wordlist {
 #define FALSE  0
 #define TRUE   1
 
-extern char user[];
-extern char passwd[];
-extern char devname[];
-extern char our_name[];
-extern char remote_name[];
-extern char hostname[];
-extern int uselogin;
-extern int usehostname;
-extern int auth_required;
-
 /* Records which authentication operations haven't completed yet. */
-static int auth_pending[NPPP];
+static int auth_pending[NUM_PPP];
+
+/* Set if we have successfully called login() */
 static int logged_in;
-static struct wordlist *addresses[NPPP];
+
+/* List of addresses which the peer may use. */
+static struct wordlist *addresses[NUM_PPP];
+
+/* Number of network protocols which we have opened. */
+static int num_np_open;
+
+/* Number of network protocols which have come up. */
+static int num_np_up;
 
 /* Bits in auth_pending[] */
 #define UPAP_WITHPEER  1
@@ -101,20 +115,20 @@ static struct wordlist *addresses[NPPP];
 #define CHAP_WITHPEER  4
 #define CHAP_PEER      8
 
-/* Prototypes */
-void check_access __ARGS((FILE *, char *));
-
-static int  login __ARGS((char *, char *, char **, int *));
-static void logout __ARGS((void));
-static int  null_login __ARGS((int));
-static int  get_upap_passwd __ARGS((void));
-static int  have_upap_secret __ARGS((void));
-static int  have_chap_secret __ARGS((char *, char *));
-static int  scan_authfile __ARGS((FILE *, char *, char *, char *,
-                                 struct wordlist **, char *));
-static void free_wordlist __ARGS((struct wordlist *));
-
-extern char *crypt __ARGS((char *, char *));
+/* Prototypes for procedures local to this file. */
+
+static void network_phase __P((int));
+static void check_idle __P((caddr_t));
+static int  login __P((char *, char *, char **, int *));
+static void logout __P((void));
+static int  null_login __P((int));
+static int  get_upap_passwd __P((void));
+static int  have_upap_secret __P((void));
+static int  have_chap_secret __P((char *, char *, u_int32_t));
+static int  ip_addr_check __P((u_int32_t, struct wordlist *));
+static int  scan_authfile __P((FILE *, char *, char *, u_int32_t, char *,
+                              struct wordlist **, char *));
+static void free_wordlist __P((struct wordlist *));
 
 /*
  * An Open on LCP has requested a change from Dead to Establish phase.
@@ -134,6 +148,8 @@ void
 link_terminated(unit)
     int unit;
 {
+    if (phase == PHASE_DEAD)
+       return;
     if (logged_in)
        logout();
     phase = PHASE_DEAD;
@@ -147,6 +163,19 @@ void
 link_down(unit)
     int unit;
 {
+    int i;
+    struct protent *protp;
+
+    for (i = 0; (protp = protocols[i]) != NULL; ++i) {
+       if (!protp->enabled_flag)
+           continue;
+        if (protp->protocol != PPP_LCP && protp->lowerdown != NULL)
+           (*protp->lowerdown)(unit);
+        if (protp->protocol < 0xC000 && protp->close != NULL)
+           (*protp->close)(unit, "LCP down");
+    }
+    num_np_open = 0;
+    num_np_up = 0;
     phase = PHASE_TERMINATE;
 }
 
@@ -162,6 +191,16 @@ link_established(unit)
     lcp_options *wo = &lcp_wantoptions[unit];
     lcp_options *go = &lcp_gotoptions[unit];
     lcp_options *ho = &lcp_hisoptions[unit];
+    int i;
+    struct protent *protp;
+
+    /*
+     * Tell higher-level protocols that LCP is up.
+     */
+    for (i = 0; (protp = protocols[i]) != NULL; ++i)
+        if (protp->protocol != PPP_LCP && protp->enabled_flag
+           && protp->lowerup != NULL)
+           (*protp->lowerup)(unit);
 
     if (auth_required && !(go->neg_chap || go->neg_upap)) {
        /*
@@ -169,9 +208,9 @@ link_established(unit)
         * treat it as though it authenticated with PAP using a username
         * of "" and a password of "".  If that's not OK, boot it out.
         */
-       if (wo->neg_upap && !null_login(unit)) {
+       if (!wo->neg_upap || !null_login(unit)) {
            syslog(LOG_WARNING, "peer refused to authenticate");
-           lcp_close(unit);
+           lcp_close(unit, "peer refused to authenticate");
            phase = PHASE_TERMINATE;
            return;
        }
@@ -195,10 +234,32 @@ link_established(unit)
     }
     auth_pending[unit] = auth;
 
-    if (!auth) {
-       phase = PHASE_NETWORK;
-       ipcp_open(unit);
-    }
+    if (!auth)
+       network_phase(unit);
+}
+
+/*
+ * Proceed to the network phase.
+ */
+static void
+network_phase(unit)
+    int unit;
+{
+    int i;
+    struct protent *protp;
+
+    phase = PHASE_NETWORK;
+#if 0
+    if (!demand)
+       set_filters(&pass_filter, &active_filter);
+#endif
+    for (i = 0; (protp = protocols[i]) != NULL; ++i)
+        if (protp->protocol < 0xC000 && protp->enabled_flag
+           && protp->open != NULL) {
+           (*protp->open)(unit);
+           if (protp->protocol != PPP_CCP)
+               ++num_np_open;
+       }
 }
 
 /*
@@ -211,7 +272,7 @@ auth_peer_fail(unit, protocol)
     /*
      * Authentication failure: take the link down
      */
-    lcp_close(unit);
+    lcp_close(unit, "Authentication failed");
     phase = PHASE_TERMINATE;
 }
 
@@ -225,10 +286,10 @@ auth_peer_success(unit, protocol)
     int bit;
 
     switch (protocol) {
-    case CHAP:
+    case PPP_CHAP:
        bit = CHAP_PEER;
        break;
-    case UPAP:
+    case PPP_PAP:
        bit = UPAP_PEER;
        break;
     default:
@@ -241,10 +302,8 @@ auth_peer_success(unit, protocol)
      * If there is no more authentication still to be done,
      * proceed to the network phase.
      */
-    if ((auth_pending[unit] &= ~bit) == 0) {
-       phase = PHASE_NETWORK;
-       ipcp_open(unit);
-    }
+    if ((auth_pending[unit] &= ~bit) == 0)
+        network_phase(unit);
 }
 
 /*
@@ -271,36 +330,98 @@ auth_withpeer_success(unit, protocol)
     int bit;
 
     switch (protocol) {
-    case CHAP:
+    case PPP_CHAP:
        bit = CHAP_WITHPEER;
        break;
-    case UPAP:
+    case PPP_PAP:
        bit = UPAP_WITHPEER;
        break;
     default:
        syslog(LOG_WARNING, "auth_peer_success: unknown protocol %x",
               protocol);
+       bit = 0;
     }
 
     /*
      * If there is no more authentication still being done,
      * proceed to the network phase.
      */
-    if ((auth_pending[unit] &= ~bit) == 0) {
-       phase = PHASE_NETWORK;
-       ipcp_open(unit);
+    if ((auth_pending[unit] &= ~bit) == 0)
+       network_phase(unit);
+}
+
+
+/*
+ * np_up - a network protocol has come up.
+ */
+void
+np_up(unit, proto)
+    int unit, proto;
+{
+    if (num_np_up == 0 && idle_time_limit > 0) {
+       TIMEOUT(check_idle, NULL, idle_time_limit);
+    }
+    ++num_np_up;
+}
+
+/*
+ * np_down - a network protocol has gone down.
+ */
+void
+np_down(unit, proto)
+    int unit, proto;
+{
+    if (--num_np_up == 0 && idle_time_limit > 0) {
+       UNTIMEOUT(check_idle, NULL);
+    }
+}
+
+/*
+ * np_finished - a network protocol has finished using the link.
+ */
+void
+np_finished(unit, proto)
+    int unit, proto;
+{
+    if (--num_np_open <= 0) {
+       /* no further use for the link: shut up shop. */
+       lcp_close(0, "No network protocols running");
     }
 }
 
+/*
+ * check_idle - check whether the link has been idle for long
+ * enough that we can shut it down.
+ */
+static void
+check_idle(arg)
+    caddr_t arg;
+{
+    struct ppp_idle idle;
+    time_t itime;
+
+    if (!get_idle_time(0, &idle))
+       return;
+    itime = MIN(idle.xmit_idle, idle.recv_idle);
+    if (itime >= idle_time_limit) {
+       /* link is idle: shut it down. */
+       syslog(LOG_INFO, "Terminating connection due to lack of activity.");
+       lcp_close(0, "Link inactive");
+    } else {
+       TIMEOUT(check_idle, NULL, idle_time_limit - itime);
+    }
+}
 
 /*
- * check_auth_options - called to check authentication options.
+ * auth_check_options - called to check authentication options.
  */
 void
-check_auth_options()
+auth_check_options()
 {
     lcp_options *wo = &lcp_wantoptions[0];
     lcp_options *ao = &lcp_allowoptions[0];
+    ipcp_options *ipwo = &ipcp_wantoptions[0];
+    u_int32_t remote;
 
     /* Default our_name to hostname, and user to our_name */
     if (our_name[0] == 0 || usehostname)
@@ -322,14 +443,17 @@ check_auth_options()
        ao->neg_upap = 0;
     if (wo->neg_upap && !uselogin && !have_upap_secret())
        wo->neg_upap = 0;
-    if (ao->neg_chap && !have_chap_secret(our_name, remote_name))
+    if (ao->neg_chap && !have_chap_secret(our_name, remote_name, (u_int32_t)0))
        ao->neg_chap = 0;
-    if (wo->neg_chap && !have_chap_secret(remote_name, our_name))
-       wo->neg_chap = 0;
+    if (wo->neg_chap) {
+       remote = ipwo->accept_remote? 0: ipwo->hisaddr;
+       if (!have_chap_secret(remote_name, our_name, remote))
+           wo->neg_chap = 0;
+    }
 
     if (auth_required && !wo->neg_chap && !wo->neg_upap) {
        fprintf(stderr, "\
-pppd: peer authentication required but no authentication files accessible\n");
+pppd: peer authentication required but no suitable secret(s) found\n");
        exit(1);
     }
 
@@ -360,6 +484,8 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
     char *filename;
     FILE *f;
     struct wordlist *addrs;
+    u_int32_t remote;
+    ipcp_options *ipwo = &ipcp_wantoptions[unit];
     char passwd[256], user[256];
     char secret[MAXWORDLEN];
     static int attempts = 0;
@@ -382,16 +508,18 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
     f = fopen(filename, "r");
     if (f == NULL) {
        if (!uselogin) {
-           syslog(LOG_ERR, "Can't open upap password file %s: %m", filename);
+           syslog(LOG_ERR, "Can't open PAP password file %s: %m", filename);
            ret = UPAP_AUTHNAK;
        }
 
     } else {
        check_access(f, filename);
-       if (scan_authfile(f, user, our_name, secret, &addrs, filename) < 0
-           || (secret[0] != 0 && strcmp(passwd, secret) != 0
+       remote = ipwo->accept_remote? 0: ipwo->hisaddr;
+       if (scan_authfile(f, user, our_name, remote,
+                         secret, &addrs, filename) < 0
+           || (secret[0] != 0 && (cryptpap || strcmp(passwd, secret) != 0)
                && strcmp(crypt(passwd, secret), secret) != 0)) {
-           syslog(LOG_WARNING, "upap authentication failure for %s", user);
+           syslog(LOG_WARNING, "PAP authentication failure for %s", user);
            ret = UPAP_AUTHNAK;
        }
        fclose(f);
@@ -400,7 +528,7 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
     if (uselogin && ret == UPAP_AUTHACK) {
        ret = login(user, passwd, msg, msglen);
        if (ret == UPAP_AUTHNAK) {
-           syslog(LOG_WARNING, "upap login failure for %s", user);
+           syslog(LOG_WARNING, "PAP login failure for %s", user);
        }
     }
 
@@ -414,7 +542,7 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
         */
        if (attempts++ >= 10) {
            syslog(LOG_WARNING, "%d LOGIN FAILURES ON %s, %s",
-                  attempts, devname, user);
+                  attempts, devnam, user);
            quit();
        }
        if (attempts > 3)
@@ -434,6 +562,66 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
     return ret;
 }
 
+#ifdef HAS_SHADOW
+/**************
+ * This function was lifted from the shadow-3.3.2 version by John Haugh II.
+ * It is included because the function was not in the standard libshadow
+ * library. If it is included in the library then I can remove it from here.
+ */
+
+#define        DAY     (24L*3600L)
+/*
+ * isexpired - determine if account is expired yet
+ *
+ *     isexpired calculates the expiration date based on the
+ *     password expiration criteria.
+ */
+
+/*ARGSUSED*/
+int
+isexpired (pw, sp)
+struct passwd  *pw;
+struct spwd    *sp;
+{
+       long    clock;
+
+       clock = time ((time_t *) 0) / DAY;
+
+       /*
+        * Quick and easy - there is an expired account field
+        * along with an inactive account field.  Do the expired
+        * one first since it is worse.
+        */
+
+       if (sp->sp_expire > 0 && sp->sp_expire < clock)
+               return 3;
+
+       if (sp->sp_inact > 0 && sp->sp_lstchg > 0 && sp->sp_max > 0 &&
+                       sp->sp_inact + sp->sp_lstchg + sp->sp_max < clock)
+               return 2;
+
+       /*
+        * The last and max fields must be present for an account
+        * to have an expired password.  A maximum of >10000 days
+        * is considered to be infinite.
+        */
+
+       if (sp->sp_lstchg == -1 ||
+                       sp->sp_max == -1 || sp->sp_max >= 10000L)
+               return 0;
+
+       /*
+        * Calculate today's day and the day on which the password
+        * is going to expire.  If that date has already passed,
+        * the password has expired.
+        */
+
+       if (sp->sp_lstchg + sp->sp_max < clock)
+               return 1;
+
+       return 0;
+}
+#endif
 
 /*
  * login - Check the user name and password against the system
@@ -455,10 +643,23 @@ login(user, passwd, msg, msglen)
     char *epasswd;
     char *tty;
 
+#ifdef HAS_SHADOW
+    struct spwd *spwd;
+    struct spwd *getspnam();
+#endif
+
     if ((pw = getpwnam(user)) == NULL) {
        return (UPAP_AUTHNAK);
     }
 
+#ifdef HAS_SHADOW
+    if ((spwd = getspnam(user)) == NULL) {
+        pw->pw_passwd = "";
+    } else {
+       pw->pw_passwd = spwd->sp_pwdp;
+    }
+#endif
+
     /*
      * XXX If no passwd, let them login without one.
      */
@@ -466,22 +667,39 @@ login(user, passwd, msg, msglen)
        return (UPAP_AUTHACK);
     }
 
+#ifdef HAS_SHADOW
+    if (pw->pw_passwd) {
+       if (pw->pw_passwd[0] == '@') {
+           if (pw_auth (pw->pw_passwd+1, pw->pw_name, PW_PPP, NULL)) {
+               return (UPAP_AUTHNAK);
+           }
+       } else {
+           epasswd = pw_encrypt(passwd, pw->pw_passwd);
+           if (strcmp(epasswd, pw->pw_passwd)) {
+               return (UPAP_AUTHNAK);
+           }
+       }
+       /* check the age of the password entry */
+       if (spwd && (isexpired (pw, spwd) != 0)) {
+           return (UPAP_AUTHNAK);
+       }
+    }
+#else
     epasswd = crypt(passwd, pw->pw_passwd);
     if (strcmp(epasswd, pw->pw_passwd)) {
        return (UPAP_AUTHNAK);
     }
+#endif
 
     syslog(LOG_INFO, "user %s logged in", user);
 
     /*
      * Write a wtmp entry for this user.
      */
-    tty = strrchr(devname, '/');
-    if (tty == NULL)
-       tty = devname;
-    else
-       tty++;
-    logwtmp(tty, user, "");            /* Add wtmp login entry */
+    tty = devnam;
+    if (strncmp(tty, "/dev/", 5) == 0)
+       tty += 5;
+    logwtmp(tty, user, remote_name);           /* Add wtmp login entry */
     logged_in = TRUE;
 
     return (UPAP_AUTHACK);
@@ -495,11 +713,9 @@ logout()
 {
     char *tty;
 
-    tty = strrchr(devname, '/');
-    if (tty == NULL)
-       tty = devname;
-    else
-       tty++;
+    tty = devnam;
+    if (strncmp(tty, "/dev/", 5) == 0)
+       tty += 5;
     logwtmp(tty, "", "");              /* Wipe out wtmp logout entry */
     logged_in = FALSE;
 }
@@ -531,7 +747,7 @@ null_login(unit)
        return 0;
     check_access(f, filename);
 
-    i = scan_authfile(f, "", our_name, secret, &addrs, filename);
+    i = scan_authfile(f, "", our_name, (u_int32_t)0, secret, &addrs, filename);
     ret = i >= 0 && (i & NONWILD_CLIENT) != 0 && secret[0] == 0;
 
     if (ret) {
@@ -564,7 +780,8 @@ get_upap_passwd()
     if (f == NULL)
        return 0;
     check_access(f, filename);
-    if (scan_authfile(f, user, remote_name, secret, NULL, filename) < 0)
+    if (scan_authfile(f, user, remote_name, (u_int32_t)0,
+                     secret, NULL, filename) < 0)
        return 0;
     strncpy(passwd, secret, MAXSECRETLEN);
     passwd[MAXSECRETLEN-1] = 0;
@@ -582,13 +799,16 @@ have_upap_secret()
     FILE *f;
     int ret;
     char *filename;
+    ipcp_options *ipwo = &ipcp_wantoptions[0];
+    u_int32_t remote;
 
     filename = _PATH_UPAPFILE;
     f = fopen(filename, "r");
     if (f == NULL)
        return 0;
 
-    ret = scan_authfile(f, NULL, our_name, NULL, NULL, filename);
+    remote = ipwo->accept_remote? 0: ipwo->hisaddr;
+    ret = scan_authfile(f, NULL, our_name, remote, NULL, NULL, filename);
     fclose(f);
     if (ret < 0)
        return 0;
@@ -604,9 +824,10 @@ have_upap_secret()
  * know the identity yet.
  */
 static int
-have_chap_secret(client, server)
+have_chap_secret(client, server, remote)
     char *client;
     char *server;
+    u_int32_t remote;
 {
     FILE *f;
     int ret;
@@ -622,7 +843,7 @@ have_chap_secret(client, server)
     else if (server[0] == 0)
        server = NULL;
 
-    ret = scan_authfile(f, client, server, NULL, NULL, filename);
+    ret = scan_authfile(f, client, server, remote, NULL, NULL, filename);
     fclose(f);
     if (ret < 0)
        return 0;
@@ -643,6 +864,7 @@ get_secret(unit, client, server, secret, secret_len, save_addrs)
     char *server;
     char *secret;
     int *secret_len;
+    int save_addrs;
 {
     FILE *f;
     int ret, len;
@@ -661,7 +883,8 @@ get_secret(unit, client, server, secret, secret_len, save_addrs)
     }
     check_access(f, filename);
 
-    ret = scan_authfile(f, client, server, secbuf, &addrs, filename);
+    ret = scan_authfile(f, client, server, (u_int32_t)0,
+                       secbuf, &addrs, filename);
     fclose(f);
     if (ret < 0)
        return 0;
@@ -690,33 +913,89 @@ get_secret(unit, client, server, secret, secret_len, save_addrs)
 int
 auth_ip_addr(unit, addr)
     int unit;
-    u_long addr;
+    u_int32_t addr;
 {
-    u_long a;
-    struct hostent *hp;
+    return ip_addr_check(addr, addresses[unit]);
+}
+
+static int
+ip_addr_check(addr, addrs)
+    u_int32_t addr;
     struct wordlist *addrs;
+{
+    u_int32_t a, mask, ah;
+    int accept;
+    char *ptr_word, *ptr_mask;
+    struct hostent *hp;
+    struct netent *np;
 
     /* don't allow loopback or multicast address */
     if (bad_ip_adrs(addr))
        return 0;
 
-    if ((addrs = addresses[unit]) == NULL)
+    if (addrs == NULL)
        return 1;               /* no restriction */
 
     for (; addrs != NULL; addrs = addrs->next) {
        /* "-" means no addresses authorized */
-       if (strcmp(addrs->word, "-") == 0)
+       ptr_word = addrs->word;
+       if (strcmp(ptr_word, "-") == 0)
            break;
-       if ((a = inet_addr(addrs->word)) == -1) {
-           if ((hp = gethostbyname(addrs->word)) == NULL) {
-               syslog(LOG_WARNING, "unknown host %s in auth. address list",
-                      addrs->word);
+
+       accept = 1;
+       if (*ptr_word == '!') {
+           accept = 0;
+           ++ptr_word;
+       }
+
+       mask = ~ (u_int32_t) 0;
+       ptr_mask = strchr (ptr_word, '/');
+       if (ptr_mask != NULL) {
+           int bit_count;
+
+           bit_count = (int) strtol (ptr_mask+1, (char **) 0, 10);
+           if (bit_count <= 0 || bit_count > 32) {
+               syslog (LOG_WARNING,
+                       "invalid address length %s in auth. address list",
+                       ptr_mask);
                continue;
-           } else
-               a = *(u_long *)hp->h_addr;
+           }
+           *ptr_mask = '\0';
+           mask <<= 32 - bit_count;
+       }
+
+       hp = gethostbyname(ptr_word);
+       if (hp != NULL && hp->h_addrtype == AF_INET) {
+           a    = *(u_int32_t *)hp->h_addr;
+           mask = ~ (u_int32_t) 0;     /* are we sure we want this? */
+       } else {
+           np = getnetbyname (ptr_word);
+           if (np != NULL && np->n_addrtype == AF_INET)
+               a = htonl (*(u_int32_t *)np->n_net);
+           else
+               a = inet_addr (ptr_word);
+           if (ptr_mask == NULL) {
+               /* calculate appropriate mask for net */
+               ah = ntohl(a);
+               if (IN_CLASSA(ah))
+                   mask = IN_CLASSA_NET;
+               else if (IN_CLASSB(ah))
+                   mask = IN_CLASSB_NET;
+               else if (IN_CLASSC(ah))
+                   mask = IN_CLASSC_NET;
+           }
        }
-       if (addr == a)
-           return 1;
+
+       if (ptr_mask != NULL)
+           *ptr_mask = '/';
+
+       if (a == -1L)
+           syslog (LOG_WARNING,
+                   "unknown host %s in auth. address list",
+                   addrs->word);
+       else
+           if (((addr ^ a) & htonl(mask)) == 0)
+               return accept;
     }
     return 0;                  /* not in list => can't have it */
 }
@@ -728,7 +1007,7 @@ auth_ip_addr(unit, addr)
  */
 int
 bad_ip_adrs(addr)
-    u_long addr;
+    u_int32_t addr;
 {
     addr = ntohl(addr);
     return (addr >> IN_CLASSA_NSHIFT) == IN_LOOPBACKNET
@@ -763,10 +1042,11 @@ check_access(f, filename)
  * info) are placed in a wordlist and returned in *addrs.  
  */
 static int
-scan_authfile(f, client, server, secret, addrs, filename)
+scan_authfile(f, client, server, ipaddr, secret, addrs, filename)
     FILE *f;
     char *client;
     char *server;
+    u_int32_t ipaddr;
     char *secret;
     struct wordlist **addrs;
     char *filename;
@@ -774,9 +1054,10 @@ scan_authfile(f, client, server, secret, addrs, filename)
     int newline, xxx;
     int got_flag, best_flag;
     FILE *sf;
-    struct wordlist *ap, *addr_list, *addr_last;
+    struct wordlist *ap, *addr_list, *alist, *alast;
     char word[MAXWORDLEN];
     char atfile[MAXWORDLEN];
+    char lsecret[MAXWORDLEN];
 
     if (addrs != NULL)
        *addrs = NULL;
@@ -852,16 +1133,12 @@ scan_authfile(f, client, server, secret, addrs, filename)
            fclose(sf);
        }
        if (secret != NULL)
-           strcpy(secret, word);
-               
-       best_flag = got_flag;
+           strcpy(lsecret, word);
 
        /*
         * Now read address authorization info and make a wordlist.
         */
-       if (addr_list)
-           free_wordlist(addr_list);
-       addr_list = NULL;
+       alist = alast = NULL;
        for (;;) {
            if (!getword(f, word, &newline, filename) || newline)
                break;
@@ -871,12 +1148,31 @@ scan_authfile(f, client, server, secret, addrs, filename)
                novm("authorized addresses");
            ap->next = NULL;
            strcpy(ap->word, word);
-           if (addr_list == NULL)
-               addr_list = ap;
+           if (alist == NULL)
+               alist = ap;
            else
-               addr_last->next = ap;
-           addr_last = ap;
+               alast->next = ap;
+           alast = ap;
+       }
+
+       /*
+        * Check if the given IP address is allowed by the wordlist.
+        */
+       if (ipaddr != 0 && !ip_addr_check(ipaddr, alist)) {
+           free_wordlist(alist);
+           continue;
        }
+
+       /*
+        * This is the best so far; remember it.
+        */
+       best_flag = got_flag;
+       if (addr_list)
+           free_wordlist(addr_list);
+       addr_list = alist;
+       if (secret != NULL)
+           strcpy(secret, lsecret);
+
        if (!newline)
            break;
     }