]> andersk Git - openssh.git/blobdiff - auth-options.c
- markus@cvs.openbsd.org 2002/03/19 10:35:39
[openssh.git] / auth-options.c
index 83ef02c42447ef8ebaf27a7c18167903e48f07d8..3408b3d8f78b9dc113f4a292596f40eb86386fc0 100644 (file)
@@ -10,7 +10,7 @@
  */
 
 #include "includes.h"
-RCSID("$OpenBSD: auth-options.c,v 1.19 2001/06/24 05:25:09 markus Exp $");
+RCSID("$OpenBSD: auth-options.c,v 1.23 2002/03/19 10:35:39 markus Exp $");
 
 #include "packet.h"
 #include "xmalloc.h"
@@ -20,6 +20,13 @@ RCSID("$OpenBSD: auth-options.c,v 1.19 2001/06/24 05:25:09 markus Exp $");
 #include "channels.h"
 #include "auth-options.h"
 #include "servconf.h"
+#include "bufaux.h"
+#include "misc.h"
+#include "monitor_wrap.h"
+
+/* Debugging messages */
+Buffer auth_debug;
+int auth_debug_init;
 
 /* Flags set authorized_keys flags */
 int no_port_forwarding_flag = 0;
@@ -35,9 +42,28 @@ struct envstring *custom_environment = NULL;
 
 extern ServerOptions options;
 
+static void
+auth_send_debug(Buffer *m)
+{
+       char *msg;
+
+       while (buffer_len(m)) {
+               msg = buffer_get_string(m, NULL);
+               packet_send_debug("%s", msg);
+               xfree(msg);
+       }
+}
+
 void
 auth_clear_options(void)
 {
+       if (auth_debug_init)
+               buffer_clear(&auth_debug);
+       else {
+               buffer_init(&auth_debug);
+               auth_debug_init = 1;
+       }
+
        no_agent_forwarding_flag = 0;
        no_port_forwarding_flag = 0;
        no_pty_flag = 0;
@@ -62,6 +88,7 @@ auth_clear_options(void)
 int
 auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
 {
+       char tmp[1024];
        const char *cp;
        int i;
 
@@ -74,28 +101,32 @@ auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
        while (*opts && *opts != ' ' && *opts != '\t') {
                cp = "no-port-forwarding";
                if (strncasecmp(opts, cp, strlen(cp)) == 0) {
-                       packet_send_debug("Port forwarding disabled.");
+                       snprintf(tmp, sizeof(tmp), "Port forwarding disabled.");
+                       buffer_put_cstring(&auth_debug, tmp);
                        no_port_forwarding_flag = 1;
                        opts += strlen(cp);
                        goto next_option;
                }
                cp = "no-agent-forwarding";
                if (strncasecmp(opts, cp, strlen(cp)) == 0) {
-                       packet_send_debug("Agent forwarding disabled.");
+                       snprintf(tmp, sizeof(tmp), "Agent forwarding disabled.");
+                       buffer_put_cstring(&auth_debug, tmp);
                        no_agent_forwarding_flag = 1;
                        opts += strlen(cp);
                        goto next_option;
                }
                cp = "no-X11-forwarding";
                if (strncasecmp(opts, cp, strlen(cp)) == 0) {
-                       packet_send_debug("X11 forwarding disabled.");
+                       snprintf(tmp, sizeof(tmp), "X11 forwarding disabled.");
+                       buffer_put_cstring(&auth_debug, tmp);
                        no_x11_forwarding_flag = 1;
                        opts += strlen(cp);
                        goto next_option;
                }
                cp = "no-pty";
                if (strncasecmp(opts, cp, strlen(cp)) == 0) {
-                       packet_send_debug("Pty allocation disabled.");
+                       snprintf(tmp, sizeof(tmp), "Pty allocation disabled.");
+                       buffer_put_cstring(&auth_debug, tmp);
                        no_pty_flag = 1;
                        opts += strlen(cp);
                        goto next_option;
@@ -118,14 +149,16 @@ auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
                        if (!*opts) {
                                debug("%.100s, line %lu: missing end quote",
                                    file, linenum);
-                               packet_send_debug("%.100s, line %lu: missing end quote",
+                               snprintf(tmp, sizeof(tmp), "%.100s, line %lu: missing end quote",
                                    file, linenum);
+                               buffer_put_cstring(&auth_debug, tmp);
                                xfree(forced_command);
                                forced_command = NULL;
                                goto bad_option;
                        }
                        forced_command[i] = 0;
-                       packet_send_debug("Forced command: %.900s", forced_command);
+                       snprintf(tmp, sizeof(tmp), "Forced command: %.900s", forced_command);
+                       buffer_put_cstring(&auth_debug, tmp);
                        opts++;
                        goto next_option;
                }
@@ -150,13 +183,15 @@ auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
                        if (!*opts) {
                                debug("%.100s, line %lu: missing end quote",
                                    file, linenum);
-                               packet_send_debug("%.100s, line %lu: missing end quote",
+                               snprintf(tmp, sizeof(tmp), "%.100s, line %lu: missing end quote",
                                    file, linenum);
+                               buffer_put_cstring(&auth_debug, tmp);
                                xfree(s);
                                goto bad_option;
                        }
                        s[i] = 0;
-                       packet_send_debug("Adding to environment: %.900s", s);
+                       snprintf(tmp, sizeof(tmp), "Adding to environment: %.900s", s);
+                       buffer_put_cstring(&auth_debug, tmp);
                        debug("Adding to environment: %.900s", s);
                        opts++;
                        new_envstring = xmalloc(sizeof(struct envstring));
@@ -169,7 +204,7 @@ auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
                if (strncasecmp(opts, cp, strlen(cp)) == 0) {
                        const char *remote_ip = get_remote_ipaddr();
                        const char *remote_host = get_canonical_hostname(
-                           options.reverse_mapping_check);
+                           options.verify_reverse_mapping);
                        char *patterns = xmalloc(strlen(opts) + 1);
 
                        opts += strlen(cp);
@@ -187,8 +222,9 @@ auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
                        if (!*opts) {
                                debug("%.100s, line %lu: missing end quote",
                                    file, linenum);
-                               packet_send_debug("%.100s, line %lu: missing end quote",
+                               snprintf(tmp, sizeof(tmp), "%.100s, line %lu: missing end quote",
                                    file, linenum);
+                               buffer_put_cstring(&auth_debug, tmp);
                                xfree(patterns);
                                goto bad_option;
                        }
@@ -201,9 +237,11 @@ auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
                                    "correct key but not from a permitted "
                                    "host (host=%.200s, ip=%.200s).",
                                    pw->pw_name, remote_host, remote_ip);
-                               packet_send_debug("Your host '%.200s' is not "
+                               snprintf(tmp, sizeof(tmp),
+                                   "Your host '%.200s' is not "
                                    "permitted to use this key for login.",
                                    remote_host);
+                               buffer_put_cstring(&auth_debug, tmp);
                                /* deny access */
                                return 0;
                        }
@@ -213,8 +251,8 @@ auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
                }
                cp = "permitopen=\"";
                if (strncasecmp(opts, cp, strlen(cp)) == 0) {
+                       char host[256], sport[6];
                        u_short port;
-                       char *c, *ep;
                        char *patterns = xmalloc(strlen(opts) + 1);
 
                        opts += strlen(cp);
@@ -232,35 +270,35 @@ auth_parse_options(struct passwd *pw, char *opts, char *file, u_long linenum)
                        if (!*opts) {
                                debug("%.100s, line %lu: missing end quote",
                                    file, linenum);
-                               packet_send_debug("%.100s, line %lu: missing end quote",
+                               snprintf(tmp, sizeof(tmp), "%.100s, line %lu: missing end quote",
                                    file, linenum);
+                               buffer_put_cstring(&auth_debug, tmp);
                                xfree(patterns);
                                goto bad_option;
                        }
                        patterns[i] = 0;
                        opts++;
-                       c = strchr(patterns, ':');
-                       if (c == NULL) {
-                               debug("%.100s, line %lu: permitopen: missing colon <%.100s>",
-                                   file, linenum, patterns);
-                               packet_send_debug("%.100s, line %lu: missing colon",
-                                   file, linenum);
+                       if (sscanf(patterns, "%255[^:]:%5[0-9]", host, sport) != 2 &&
+                           sscanf(patterns, "%255[^/]/%5[0-9]", host, sport) != 2) {
+                               debug("%.100s, line %lu: Bad permitopen specification "
+                                   "<%.100s>", file, linenum, patterns);
+                               snprintf(tmp, sizeof(tmp), "%.100s, line %lu: "
+                                   "Bad permitopen specification", file, linenum);
+                               buffer_put_cstring(&auth_debug, tmp);
                                xfree(patterns);
                                goto bad_option;
                        }
-                       *c = 0;
-                       c++;
-                       port = strtol(c, &ep, 0);
-                       if (c == ep) {
-                               debug("%.100s, line %lu: permitopen: missing port <%.100s>",
-                                   file, linenum, patterns);
-                               packet_send_debug("%.100s, line %lu: missing port",
-                                   file, linenum);
+                       if ((port = a2port(sport)) == 0) {
+                               debug("%.100s, line %lu: Bad permitopen port <%.100s>",
+                                   file, linenum, sport);
+                               snprintf(tmp, sizeof(tmp), "%.100s, line %lu: "
+                                   "Bad permitopen port", file, linenum);
+                               buffer_put_cstring(&auth_debug, tmp);
                                xfree(patterns);
                                goto bad_option;
                        }
                        if (options.allow_tcp_forwarding)
-                               channel_add_permitted_opens(patterns, port);
+                               channel_add_permitted_opens(host, port);
                        xfree(patterns);
                        goto next_option;
                }
@@ -278,14 +316,24 @@ next_option:
                opts++;
                /* Process the next option. */
        }
+
+       if (!use_privsep)
+               auth_send_debug(&auth_debug);
+
        /* grant access */
        return 1;
 
 bad_option:
        log("Bad options in %.100s file, line %lu: %.50s",
            file, linenum, opts);
-       packet_send_debug("Bad options in %.100s file, line %lu: %.50s",
+       snprintf(tmp, sizeof(tmp),
+           "Bad options in %.100s file, line %lu: %.50s",
            file, linenum, opts);
+       buffer_put_cstring(&auth_debug, tmp);
+
+       if (!use_privsep)
+               auth_send_debug(&auth_debug);
+
        /* deny access */
        return 0;
 }
This page took 0.050529 seconds and 4 git commands to generate.