]> git.ozlabs.org Git - ppp.git/blobdiff - pppd/auth.c
Add plugin support plus an initial selection of hooks.
[ppp.git] / pppd / auth.c
index 716d849ae8546a51a0cefc12e5bac2fb44873d78..ac2eee93a654066c15a2c3553d102081fc79b426 100644 (file)
@@ -32,7 +32,7 @@
  * WARRANTIES OF MERCHANTIBILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
-#define RCSID  "$Id: auth.c,v 1.57 1999/08/13 06:46:10 paulus Exp $"
+#define RCSID  "$Id: auth.c,v 1.58 1999/09/11 12:08:56 paulus Exp $"
 
 #include <stdio.h>
 #include <stddef.h>
@@ -96,6 +96,10 @@ static int logged_in;
 /* List of addresses which the peer may use. */
 static struct permitted_ip *addresses[NUM_PPP];
 
+/* Wordlist giving addresses which the peer may use
+   without authenticating itself. */
+static struct wordlist *noauth_addrs;
+
 /* Extra options to apply, from the secrets file entry for the peer. */
 static struct wordlist *extra_options;
 
@@ -108,6 +112,20 @@ static int num_np_up;
 /* Set if we got the contents of passwd[] from the pap-secrets file. */
 static int passwd_from_file;
 
+/* Hook to enable a plugin to control the idle time limit */
+int (*idle_time_hook) __P((struct ppp_idle *)) = NULL;
+
+/* Hook for a plugin to say whether we can possibly authenticate any peer */
+int (*pap_check_hook) __P((void)) = NULL;
+
+/* Hook for a plugin to check the PAP user and password */
+int (*pap_auth_hook) __P((char *user, char *passwd, char **msgp,
+                         struct wordlist **paddrs,
+                         struct wordlist **popts)) = NULL;
+
+/* Hook for a plugin to get the PAP password for authenticating us */
+int (*pap_passwd_hook) __P((char *user, char *passwd)) = NULL;
+
 /*
  * This is used to ensure that we don't start an auth-up/down
  * script while one is already running.
@@ -147,7 +165,7 @@ extern char *crypt __P((const char *, const char *));
 static void network_phase __P((int));
 static void check_idle __P((void *));
 static void connect_time_expired __P((void *));
-static int  plogin __P((char *, char *, char **, int *));
+static int  plogin __P((char *, char *, char **));
 static void plogout __P((void));
 static int  null_login __P((int));
 static int  get_pap_passwd __P((char *));
@@ -155,14 +173,16 @@ static int  have_pap_secret __P((int *));
 static int  have_chap_secret __P((char *, char *, int, int *));
 static int  ip_addr_check __P((u_int32_t, struct permitted_ip *));
 static int  scan_authfile __P((FILE *, char *, char *, char *,
-                              struct wordlist **, char *));
+                              struct wordlist **, struct wordlist **,
+                              char *));
 static void free_wordlist __P((struct wordlist *));
 static void auth_script __P((char *));
 static void auth_script_done __P((void *));
-static void set_allowed_addrs __P((int, struct wordlist *));
+static void set_allowed_addrs __P((int, struct wordlist *, struct wordlist *));
 static int  some_ip_ok __P((struct wordlist *));
 static int  setupapfile __P((char **));
 static int  privgroup __P((char **));
+static int  set_noauth_addr __P((char **));
 static void check_access __P((FILE *, char *));
 
 /*
@@ -210,6 +230,9 @@ option_t auth_options[] = {
       NULL, MAXSECRETLEN },
     { "privgroup", o_special, privgroup,
       "Allow group members to use privileged options", OPT_PRIV },
+    { "allow-ip", o_special, set_noauth_addr,
+      "Set IP address(es) which can be used without authentication",
+      OPT_PRIV },
     { NULL }
 };
 
@@ -280,6 +303,29 @@ privgroup(argv)
 }
 
 
+/*
+ * set_noauth_addr - set address(es) that can be used without authentication.
+ * Equivalent to specifying an entry like `"" * "" addr' in pap-secrets.
+ */
+static int
+set_noauth_addr(argv)
+    char **argv;
+{
+    char *addr = *argv;
+    int l = strlen(addr);
+    struct wordlist *wp;
+
+    wp = (struct wordlist *) malloc(sizeof(struct wordlist) + l + 1);
+    if (wp == NULL)
+       novm("allow-ip argument");
+    wp->word = (char *) (wp + 1);
+    wp->next = noauth_addrs;
+    BCOPY(addr, wp->word, l);
+    noauth_addrs = wp;
+    return 1;
+}
+
+
 /*
  * An Open on LCP has requested a change from Dead to Establish phase.
  * Do what's necessary to bring the physical layer up.
@@ -302,7 +348,7 @@ link_terminated(unit)
        return;
     if (logged_in)
        plogout();
-    phase = PHASE_DEAD;
+    new_phase(PHASE_DEAD);
     notice("Connection terminated.");
 }
 
@@ -318,6 +364,7 @@ link_down(unit)
 
     auth_state = s_down;
     if (auth_script_state == s_up && auth_script_pid == 0) {
+       update_link_stats(unit);
        auth_script_state = s_down;
        auth_script(_PATH_AUTHDOWN);
     }
@@ -332,7 +379,7 @@ link_down(unit)
     num_np_open = 0;
     num_np_up = 0;
     if (phase != PHASE_DEAD)
-       phase = PHASE_TERMINATE;
+       new_phase(PHASE_TERMINATE);
 }
 
 /*
@@ -361,10 +408,14 @@ link_established(unit)
     if (auth_required && !(go->neg_chap || go->neg_upap)) {
        /*
         * We wanted the peer to authenticate itself, and it refused:
-        * 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 we have some address(es) it can use without auth, fine,
+        * otherwise 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 (noauth_addrs != NULL) {
+           set_allowed_addrs(unit, noauth_addrs, NULL);
+       } else if (!wo->neg_upap || !null_login(unit)) {
            warn("peer refused to authenticate: terminating link");
            lcp_close(unit, "peer refused to authenticate");
            status = EXIT_PEER_AUTH_FAILED;
@@ -372,7 +423,7 @@ link_established(unit)
        }
     }
 
-    phase = PHASE_AUTHENTICATE;
+    new_phase(PHASE_AUTHENTICATE);
     auth = 0;
     if (go->neg_chap) {
        ChapAuthPeer(unit, our_name, go->chap_mdtype);
@@ -424,7 +475,7 @@ network_phase(unit)
      * If we negotiated callback, do it now.
      */
     if (go->neg_cbcp) {
-       phase = PHASE_CALLBACK;
+       new_phase(PHASE_CALLBACK);
        (*cbcp_protent.open)(unit);
        return;
     }
@@ -447,7 +498,7 @@ start_networks()
     int i;
     struct protent *protp;
 
-    phase = PHASE_NETWORK;
+    new_phase(PHASE_NETWORK);
 #if 0
     if (!demand)
        set_filters(&pass_filter, &active_filter);
@@ -577,15 +628,22 @@ void
 np_up(unit, proto)
     int unit, proto;
 {
+    int tlim;
+
     if (num_np_up == 0) {
        /*
         * At this point we consider that the link has come up successfully.
         */
        status = EXIT_OK;
        unsuccess = 0;
+       new_phase(PHASE_RUNNING);
 
-       if (idle_time_limit > 0)
-           TIMEOUT(check_idle, NULL, idle_time_limit);
+       if (idle_time_hook != 0)
+           tlim = (*idle_time_hook)(NULL);
+       else
+           tlim = idle_time_limit;
+       if (tlim > 0)
+           TIMEOUT(check_idle, NULL, tlim);
 
        /*
         * Set a timeout to close the connection once the maximum
@@ -610,8 +668,9 @@ void
 np_down(unit, proto)
     int unit, proto;
 {
-    if (--num_np_up == 0 && idle_time_limit > 0) {
+    if (--num_np_up == 0) {
        UNTIMEOUT(check_idle, NULL);
+       new_phase(PHASE_NETWORK);
     }
 }
 
@@ -634,22 +693,28 @@ np_finished(unit, proto)
  */
 static void
 check_idle(arg)
-     void *arg;
+    void *arg;
 {
     struct ppp_idle idle;
     time_t itime;
+    int tlim;
 
     if (!get_idle_time(0, &idle))
        return;
-    itime = MIN(idle.xmit_idle, idle.recv_idle);
-    if (itime >= idle_time_limit) {
+    if (idle_time_hook != 0) {
+       tlim = idle_time_hook(&idle);
+    } else {
+       itime = MIN(idle.xmit_idle, idle.recv_idle);
+       tlim = idle_time_limit - itime;
+    }
+    if (tlim <= 0) {
        /* link is idle: shut it down. */
        notice("Terminating connection due to lack of activity.");
        lcp_close(0, "Link inactive");
        need_holdoff = 0;
        status = EXIT_IDLE_TIMEOUT;
     } else {
-       TIMEOUT(check_idle, NULL, idle_time_limit - itime);
+       TIMEOUT(check_idle, NULL, tlim);
     }
 }
 
@@ -710,7 +775,7 @@ auth_check_options()
                                    our_name, 1, &lacks_ip);
     }
 
-    if (auth_required && !can_auth) {
+    if (auth_required && !can_auth && noauth_addrs == NULL) {
        if (explicit_remote)
            option_error(
 "The remote system (%s) is required to authenticate itself but I",
@@ -769,38 +834,50 @@ auth_reset(unit)
  * In either case, msg points to an appropriate message.
  */
 int
-check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
+check_passwd(unit, auser, userlen, apasswd, passwdlen, msg)
     int unit;
     char *auser;
     int userlen;
     char *apasswd;
     int passwdlen;
     char **msg;
-    int *msglen;
 {
     int ret;
     char *filename;
     FILE *f;
-    struct wordlist *addrs;
+    struct wordlist *addrs = NULL, *opts = NULL;
     char passwd[256], user[256];
     char secret[MAXWORDLEN];
     static int attempts = 0;
 
     /*
      * Make copies of apasswd and auser, then null-terminate them.
+     * If there are unprintable characters in the password, make
+     * them visible.
      */
-    BCOPY(apasswd, passwd, passwdlen);
-    passwd[passwdlen] = '\0';
-    BCOPY(auser, user, userlen);
-    user[userlen] = '\0';
-    *msg = (char *) 0;
+    slprintf(passwd, sizeof(passwd), "%.*v", passwdlen, apasswd);
+    slprintf(user, sizeof(user), "%.*v", userlen, auser);
+    *msg = "";
+
+    /*
+     * Check if a plugin wants to handle this.
+     */
+    if (pap_auth_hook) {
+       ret = (*pap_auth_hook)(user, passwd, msg, &addrs, &opts);
+       if (ret >= 0) {
+           if (ret)
+               set_allowed_addrs(unit, addrs, opts);
+           BZERO(passwd, sizeof(passwd));
+           return ret? UPAP_AUTHACK: UPAP_AUTHNAK;
+       }
+    }
 
     /*
      * Open the file of pap secrets and scan for a suitable secret
      * for authenticating this user.
      */
     filename = _PATH_UPAPFILE;
-    addrs = NULL;
+    addrs = opts = NULL;
     ret = UPAP_AUTHNAK;
     f = fopen(filename, "r");
     if (f == NULL) {
@@ -808,7 +885,7 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
 
     } else {
        check_access(f, filename);
-       if (scan_authfile(f, user, our_name, secret, &addrs, filename) < 0) {
+       if (scan_authfile(f, user, our_name, secret, &addrs, &opts, filename) < 0) {
            warn("no PAP secret found for %s", user);
        } else if (secret[0] != 0) {
            /* password given in pap-secrets - must match */
@@ -819,7 +896,7 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
                warn("PAP authentication failure for %s", user);
        } else if (uselogin) {
            /* empty password in pap-secrets and login option */
-           ret = plogin(user, passwd, msg, msglen);
+           ret = plogin(user, passwd, msg);
            if (ret == UPAP_AUTHNAK)
                warn("PAP login failure for %s", user);
        } else {
@@ -830,9 +907,8 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
     }
 
     if (ret == UPAP_AUTHNAK) {
-        if (*msg == (char *) 0)
+        if (**msg == 0)
            *msg = "Login incorrect";
-       *msglen = strlen(*msg);
        /*
         * XXX can we ever get here more than once??
         * Frustrate passwd stealer programs.
@@ -847,13 +923,14 @@ check_passwd(unit, auser, userlen, apasswd, passwdlen, msg, msglen)
            sleep((u_int) (attempts - 3) * 5);
        if (addrs != NULL)
            free_wordlist(addrs);
+       if (opts != NULL)
+           free_wordlist(opts);
 
     } else {
        attempts = 0;                   /* Reset count */
-       if (*msg == (char *) 0)
+       if (**msg == 0)
            *msg = "Login ok";
-       *msglen = strlen(*msg);
-       set_allowed_addrs(unit, addrs);
+       set_allowed_addrs(unit, addrs, opts);
     }
 
     BZERO(passwd, sizeof(passwd));
@@ -938,11 +1015,10 @@ static struct pam_conv PAM_conversation = {
  */
 
 static int
-plogin(user, passwd, msg, msglen)
+plogin(user, passwd, msg)
     char *user;
     char *passwd;
     char **msg;
-    int *msglen;
 {
     char *tty;
 
@@ -1098,12 +1174,11 @@ null_login(unit)
     char *filename;
     FILE *f;
     int i, ret;
-    struct wordlist *addrs;
+    struct wordlist *addrs, *opts;
     char secret[MAXWORDLEN];
 
     /*
      * Open the file of pap secrets and scan for a suitable secret.
-     * We don't accept a wildcard client.
      */
     filename = _PATH_UPAPFILE;
     addrs = NULL;
@@ -1112,14 +1187,16 @@ null_login(unit)
        return 0;
     check_access(f, filename);
 
-    i = scan_authfile(f, "", our_name, secret, &addrs, filename);
-    ret = i >= 0 && (i & NONWILD_CLIENT) != 0 && secret[0] == 0;
+    i = scan_authfile(f, "", our_name, secret, &addrs, &opts, filename);
+    ret = i >= 0 && secret[0] == 0;
     BZERO(secret, sizeof(secret));
 
     if (ret)
-       set_allowed_addrs(unit, addrs);
-    else
+       set_allowed_addrs(unit, addrs, opts);
+    else {
        free_wordlist(addrs);
+       free_wordlist(opts);
+    }
 
     fclose(f);
     return ret;
@@ -1142,6 +1219,15 @@ get_pap_passwd(passwd)
     struct wordlist *addrs;
     char secret[MAXWORDLEN];
 
+    /*
+     * Check whether a plugin wants to supply this.
+     */
+    if (pap_passwd_hook) {
+       ret = (*pap_passwd_hook)(user, passwd);
+       if (ret >= 0)
+           return ret;
+    }
+
     filename = _PATH_UPAPFILE;
     addrs = NULL;
     f = fopen(filename, "r");
@@ -1150,7 +1236,7 @@ get_pap_passwd(passwd)
     check_access(f, filename);
     ret = scan_authfile(f, user,
                        (remote_name[0]? remote_name: NULL),
-                       secret, NULL, filename);
+                       secret, NULL, NULL, filename);
     fclose(f);
     if (ret < 0)
        return 0;
@@ -1174,13 +1260,20 @@ have_pap_secret(lacks_ipp)
     char *filename;
     struct wordlist *addrs;
 
+    /* let the plugin decide, if there is one */
+    if (pap_check_hook) {
+       ret = (*pap_check_hook)();
+       if (ret >= 0)
+           return ret;
+    }
+
     filename = _PATH_UPAPFILE;
     f = fopen(filename, "r");
     if (f == NULL)
        return 0;
 
     ret = scan_authfile(f, (explicit_remote? remote_name: NULL), our_name,
-                       NULL, &addrs, filename);
+                       NULL, &addrs, NULL, filename);
     fclose(f);
     if (ret >= 0 && !some_ip_ok(addrs)) {
        if (lacks_ipp != 0)
@@ -1222,7 +1315,7 @@ have_chap_secret(client, server, need_ip, lacks_ipp)
     else if (server != NULL && server[0] == 0)
        server = NULL;
 
-    ret = scan_authfile(f, client, server, NULL, &addrs, filename);
+    ret = scan_authfile(f, client, server, NULL, &addrs, NULL, filename);
     fclose(f);
     if (ret >= 0 && need_ip && !some_ip_ok(addrs)) {
        if (lacks_ipp != 0)
@@ -1253,7 +1346,7 @@ get_secret(unit, client, server, secret, secret_len, am_server)
     FILE *f;
     int ret, len;
     char *filename;
-    struct wordlist *addrs;
+    struct wordlist *addrs, *opts;
     char secbuf[MAXWORDLEN];
 
     if (!am_server && passwd[0] != 0) {
@@ -1270,13 +1363,17 @@ get_secret(unit, client, server, secret, secret_len, am_server)
        }
        check_access(f, filename);
 
-       ret = scan_authfile(f, client, server, secbuf, &addrs, filename);
+       ret = scan_authfile(f, client, server, secbuf, &addrs, &opts, filename);
        fclose(f);
        if (ret < 0)
            return 0;
 
        if (am_server)
-           set_allowed_addrs(unit, addrs);
+           set_allowed_addrs(unit, addrs, opts);
+       else {
+           free_wordlist(addrs);
+           free_wordlist(opts);
+       }
     }
 
     len = strlen(secbuf);
@@ -1297,9 +1394,10 @@ get_secret(unit, client, server, secret, secret_len, am_server)
  * and leaves the following words in extra_options.
  */
 static void
-set_allowed_addrs(unit, addrs)
+set_allowed_addrs(unit, addrs, opts)
     int unit;
     struct wordlist *addrs;
+    struct wordlist *opts;
 {
     int n;
     struct wordlist *ap, **pap;
@@ -1316,21 +1414,13 @@ set_allowed_addrs(unit, addrs)
     addresses[unit] = NULL;
     if (extra_options != NULL)
        free_wordlist(extra_options);
-    extra_options = NULL;
+    extra_options = opts;
 
     /*
-     * Count the number of IP addresses given, and chop off
-     * any extra options for this peer.
+     * Count the number of IP addresses given.
      */
-    for (n = 0, pap = &addrs; (ap = *pap) != NULL; pap = &ap->next, ++n) {
-       if (strcmp(ap->word, "--") == 0) {
-           /* rest are options */
-           *pap = 0;
-           extra_options = ap->next;
-           free(ap);
-           break;
-       }
-    }
+    for (n = 0, pap = &addrs; (ap = *pap) != NULL; pap = &ap->next)
+       ++n;
     if (n == 0)
        return;
     ip = (struct permitted_ip *) malloc((n + 1) * sizeof(struct permitted_ip));
@@ -1452,12 +1542,20 @@ auth_ip_addr(unit, addr)
     int unit;
     u_int32_t addr;
 {
-    if (addresses[unit] == NULL) {
-       if (auth_required)
-           return 0;           /* no addresses authorized */
-       return allow_any_ip || !have_route_to(addr);
+    int ok;
+
+    /* don't allow loopback or multicast address */
+    if (bad_ip_adrs(addr))
+       return 0;
+
+    if (addresses[unit] != NULL) {
+       ok = ip_addr_check(addr, addresses[unit]);
+       if (ok >= 0)
+           return ok;
     }
-    return ip_addr_check(addr, addresses[unit]);
+    if (auth_required)
+       return 0;               /* no addresses authorized */
+    return allow_any_ip || !have_route_to(addr);
 }
 
 static int
@@ -1465,10 +1563,6 @@ ip_addr_check(addr, addrs)
     u_int32_t addr;
     struct permitted_ip *addrs;
 {
-    /* don't allow loopback or multicast address */
-    if (bad_ip_adrs(addr))
-       return 0;
-
     for (; ; ++addrs)
        if ((addr & addrs->mask) == addrs->base)
            return addrs->permit;
@@ -1497,7 +1591,7 @@ some_ip_ok(addrs)
     struct wordlist *addrs;
 {
     for (; addrs != 0; addrs = addrs->next) {
-       if (strcmp(addrs->word, "-") == 0)
+       if (addrs->word[0] == '-')
            break;
        if (addrs->word[0] != '!')
            return 1;           /* some IP address is allowed */
@@ -1530,29 +1624,34 @@ check_access(f, filename)
  * if no secret is found, otherwise >= 0.  The return value has
  * NONWILD_CLIENT set if the secret didn't have "*" for the client, and
  * NONWILD_SERVER set if the secret didn't have "*" for the server.
- * Any following words on the line (i.e. address authorization
- * info) are placed in a wordlist and returned in *addrs.
+ * Any following words on the line up to a "--" (i.e. address authorization
+ * info) are placed in a wordlist and returned in *addrs.  Any
+ * following words (extra options) are placed in a wordlist and
+ * returned in *opts.
  * We assume secret is NULL or points to MAXWORDLEN bytes of space.
  */
 static int
-scan_authfile(f, client, server, secret, addrs, filename)
+scan_authfile(f, client, server, secret, addrs, opts, filename)
     FILE *f;
     char *client;
     char *server;
     char *secret;
     struct wordlist **addrs;
+    struct wordlist **opts;
     char *filename;
 {
     int newline, xxx;
     int got_flag, best_flag;
     FILE *sf;
-    struct wordlist *ap, *addr_list, *alist, *alast;
+    struct wordlist *ap, *addr_list, *alist, **app;
     char word[MAXWORDLEN];
     char atfile[MAXWORDLEN];
     char lsecret[MAXWORDLEN];
 
     if (addrs != NULL)
        *addrs = NULL;
+    if (opts != NULL)
+       *opts = NULL;
     addr_list = NULL;
     if (!getword(f, word, &newline, filename))
        return -1;              /* file is empty??? */
@@ -1629,23 +1728,20 @@ scan_authfile(f, client, server, secret, addrs, filename)
        /*
         * Now read address authorization info and make a wordlist.
         */
-       alist = alast = NULL;
+       app = &alist;
        for (;;) {
            if (!getword(f, word, &newline, filename) || newline)
                break;
            ap = (struct wordlist *) malloc(sizeof(struct wordlist));
            if (ap == NULL)
                novm("authorized addresses");
-           ap->next = NULL;
            ap->word = strdup(word);
            if (ap->word == NULL)
-               novm("authorized address");
-           if (alist == NULL)
-               alist = ap;
-           else
-               alast->next = ap;
-           alast = ap;
+               novm("authorized addresses");
+           *app = ap;
+           app = &ap->next;
        }
+       *app = NULL;
 
        /*
         * This is the best so far; remember it.
@@ -1661,6 +1757,20 @@ scan_authfile(f, client, server, secret, addrs, filename)
            break;
     }
 
+    /* scan for a -- word indicating the start of options */
+    for (app = &addr_list; (ap = *app) != NULL; app = &ap->next)
+       if (strcmp(ap->word, "--") == 0)
+           break;
+    /* ap = start of options */
+    if (ap != NULL) {
+       ap = ap->next;          /* first option */
+       free(*app);                     /* free the "--" word */
+       *app = NULL;            /* terminate addr list */
+    }
+    if (opts != NULL)
+       *opts = ap;
+    else if (ap != NULL)
+       free_wordlist(ap);
     if (addrs != NULL)
        *addrs = addr_list;
     else if (addr_list != NULL)