]> andersk Git - openssh.git/blobdiff - auth-rhosts.c
- djm@cvs.openbsd.org 2010/01/30 02:54:53
[openssh.git] / auth-rhosts.c
index bd15261f705946054ca1a9def56e1051792dc511..5c12967016b4d62720e2aadfa10505c9c3f310a2 100644 (file)
@@ -1,3 +1,4 @@
+/* $OpenBSD: auth-rhosts.c,v 1.43 2008/06/13 14:18:51 dtucker Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
  */
 
 #include "includes.h"
-RCSID("$OpenBSD: auth-rhosts.c,v 1.27 2002/03/04 12:43:06 markus Exp $");
+
+#include <sys/types.h>
+#include <sys/stat.h>
+
+#ifdef HAVE_NETGROUP_H
+# include <netgroup.h>
+#endif
+#include <pwd.h>
+#include <stdio.h>
+#include <string.h>
+#include <stdarg.h>
+#include <fcntl.h>
+#include <unistd.h>
 
 #include "packet.h"
+#include "buffer.h"
 #include "uidswap.h"
 #include "pathnames.h"
 #include "log.h"
 #include "servconf.h"
 #include "canohost.h"
+#include "key.h"
+#include "hostfile.h"
 #include "auth.h"
+#include "misc.h"
 
 /* import */
 extern ServerOptions options;
+extern int use_privsep;
 
 /*
  * This function processes an rhosts-style file (.rhosts, .shosts, or
@@ -40,12 +58,27 @@ check_rhosts_file(const char *filename, const char *hostname,
 {
        FILE *f;
        char buf[1024]; /* Must not be larger than host, user, dummy below. */
+       int fd;
+       struct stat st;
 
        /* Open the .rhosts file, deny if unreadable */
-       f = fopen(filename, "r");
-       if (!f)
+       if ((fd = open(filename, O_RDONLY|O_NONBLOCK)) == -1)
                return 0;
-
+       if (fstat(fd, &st) == -1) {
+               close(fd);
+               return 0;
+       }
+       if (!S_ISREG(st.st_mode)) {
+               logit("User %s hosts file %s is not a regular file",
+                   server_user, filename);
+               close(fd);
+               return 0;
+       }
+       unset_nonblock(fd);
+       if ((f = fdopen(fd, "r")) == NULL) {
+               close(fd);
+               return 0;
+       }
        while (fgets(buf, sizeof(buf), f)) {
                /* All three must be at least as big as buf to avoid overflows. */
                char hostbuf[1024], userbuf[1024], dummy[1024], *host, *user, *cp;
@@ -67,9 +100,10 @@ check_rhosts_file(const char *filename, const char *hostname,
                 * This should be safe because each buffer is as big as the
                 * whole string, and thus cannot be overwritten.
                 */
-               switch (sscanf(buf, "%s %s %s", hostbuf, userbuf, dummy)) {
+               switch (sscanf(buf, "%1023s %1023s %1023s", hostbuf, userbuf,
+                   dummy)) {
                case 0:
-                       packet_send_debug("Found empty line in %.100s.", filename);
+                       auth_debug_add("Found empty line in %.100s.", filename);
                        continue;
                case 1:
                        /* Host name only. */
@@ -79,7 +113,7 @@ check_rhosts_file(const char *filename, const char *hostname,
                        /* Got both host and user name. */
                        break;
                case 3:
-                       packet_send_debug("Found garbage in %.100s.", filename);
+                       auth_debug_add("Found garbage in %.100s.", filename);
                        continue;
                default:
                        /* Weird... */
@@ -106,8 +140,8 @@ check_rhosts_file(const char *filename, const char *hostname,
                /* Check for empty host/user names (particularly '+'). */
                if (!host[0] || !user[0]) {
                        /* We come here if either was '+' or '-'. */
-                       packet_send_debug("Ignoring wild host/user names in %.100s.",
-                                         filename);
+                       auth_debug_add("Ignoring wild host/user names in %.100s.",
+                           filename);
                        continue;
                }
                /* Verify that host name matches. */
@@ -130,8 +164,8 @@ check_rhosts_file(const char *filename, const char *hostname,
 
                /* If the entry was negated, deny access. */
                if (negated) {
-                       packet_send_debug("Matched negative entry in %.100s.",
-                                         filename);
+                       auth_debug_add("Matched negative entry in %.100s.",
+                           filename);
                        return 0;
                }
                /* Accept authentication. */
@@ -153,16 +187,14 @@ int
 auth_rhosts(struct passwd *pw, const char *client_user)
 {
        const char *hostname, *ipaddr;
-       int ret;
 
-       hostname = get_canonical_hostname(options.verify_reverse_mapping);
+       hostname = get_canonical_hostname(options.use_dns);
        ipaddr = get_remote_ipaddr();
-       ret = auth_rhosts2(pw, client_user, hostname, ipaddr);
-       return ret;
+       return auth_rhosts2(pw, client_user, hostname, ipaddr);
 }
 
-int
-auth_rhosts2(struct passwd *pw, const char *client_user, const char *hostname,
+static int
+auth_rhosts2_raw(struct passwd *pw, const char *client_user, const char *hostname,
     const char *ipaddr)
 {
        char buf[1024];
@@ -173,10 +205,6 @@ auth_rhosts2(struct passwd *pw, const char *client_user, const char *hostname,
        debug2("auth_rhosts2: clientuser %s hostname %s ipaddr %s",
            client_user, hostname, ipaddr);
 
-       /* no user given */
-       if (pw == NULL)
-               return 0;
-
        /* Switch to the user's uid. */
        temporarily_use_uid(pw);
        /*
@@ -205,13 +233,13 @@ auth_rhosts2(struct passwd *pw, const char *client_user, const char *hostname,
        if (pw->pw_uid != 0) {
                if (check_rhosts_file(_PATH_RHOSTS_EQUIV, hostname, ipaddr,
                    client_user, pw->pw_name)) {
-                       packet_send_debug("Accepted for %.100s [%.100s] by /etc/hosts.equiv.",
+                       auth_debug_add("Accepted for %.100s [%.100s] by /etc/hosts.equiv.",
                            hostname, ipaddr);
                        return 1;
                }
                if (check_rhosts_file(_PATH_SSH_HOSTS_EQUIV, hostname, ipaddr,
                    client_user, pw->pw_name)) {
-                       packet_send_debug("Accepted for %.100s [%.100s] by %.100s.",
+                       auth_debug_add("Accepted for %.100s [%.100s] by %.100s.",
                            hostname, ipaddr, _PATH_SSH_HOSTS_EQUIV);
                        return 1;
                }
@@ -221,19 +249,19 @@ auth_rhosts2(struct passwd *pw, const char *client_user, const char *hostname,
         * not group or world writable.
         */
        if (stat(pw->pw_dir, &st) < 0) {
-               log("Rhosts authentication refused for %.100s: no home directory %.200s",
-                   pw->pw_name, pw->pw_dir);
-               packet_send_debug("Rhosts authentication refused for %.100s: no home directory %.200s",
-                                 pw->pw_name, pw->pw_dir);
+               logit("Rhosts authentication refused for %.100s: "
+                   "no home directory %.200s", pw->pw_name, pw->pw_dir);
+               auth_debug_add("Rhosts authentication refused for %.100s: "
+                   "no home directory %.200s", pw->pw_name, pw->pw_dir);
                return 0;
        }
        if (options.strict_modes &&
            ((st.st_uid != 0 && st.st_uid != pw->pw_uid) ||
            (st.st_mode & 022) != 0)) {
-               log("Rhosts authentication refused for %.100s: bad ownership or modes for home directory.",
-                   pw->pw_name);
-               packet_send_debug("Rhosts authentication refused for %.100s: bad ownership or modes for home directory.",
-                                 pw->pw_name);
+               logit("Rhosts authentication refused for %.100s: "
+                   "bad ownership or modes for home directory.", pw->pw_name);
+               auth_debug_add("Rhosts authentication refused for %.100s: "
+                   "bad ownership or modes for home directory.", pw->pw_name);
                return 0;
        }
        /* Temporarily use the user's uid. */
@@ -257,23 +285,25 @@ auth_rhosts2(struct passwd *pw, const char *client_user, const char *hostname,
                if (options.strict_modes &&
                    ((st.st_uid != 0 && st.st_uid != pw->pw_uid) ||
                    (st.st_mode & 022) != 0)) {
-                       log("Rhosts authentication refused for %.100s: bad modes for %.200s",
+                       logit("Rhosts authentication refused for %.100s: bad modes for %.200s",
                            pw->pw_name, buf);
-                       packet_send_debug("Bad file modes for %.200s", buf);
+                       auth_debug_add("Bad file modes for %.200s", buf);
                        continue;
                }
                /* Check if we have been configured to ignore .rhosts and .shosts files. */
                if (options.ignore_rhosts) {
-                       packet_send_debug("Server has been configured to ignore %.100s.",
-                                         rhosts_files[rhosts_file_index]);
+                       auth_debug_add("Server has been configured to ignore %.100s.",
+                           rhosts_files[rhosts_file_index]);
                        continue;
                }
                /* Check if authentication is permitted by the file. */
                if (check_rhosts_file(buf, hostname, ipaddr, client_user, pw->pw_name)) {
-                       packet_send_debug("Accepted by %.100s.",
-                                         rhosts_files[rhosts_file_index]);
+                       auth_debug_add("Accepted by %.100s.",
+                           rhosts_files[rhosts_file_index]);
                        /* Restore the privileged uid. */
                        restore_uid();
+                       auth_debug_add("Accepted host %s ip %s client_user %s server_user %s",
+                               hostname, ipaddr, client_user, pw->pw_name);
                        return 1;
                }
        }
@@ -282,3 +312,16 @@ auth_rhosts2(struct passwd *pw, const char *client_user, const char *hostname,
        restore_uid();
        return 0;
 }
+
+int
+auth_rhosts2(struct passwd *pw, const char *client_user, const char *hostname,
+    const char *ipaddr)
+{
+       int ret;
+
+       auth_debug_reset();
+       ret = auth_rhosts2_raw(pw, client_user, hostname, ipaddr);
+       if (!use_privsep)
+               auth_debug_send();
+       return ret;
+}
This page took 0.083674 seconds and 4 git commands to generate.