markus@openbsd.org | 75b8af8 | 2017-05-31 10:54:00 +0000 | [diff] [blame] | 1 | /* $OpenBSD: auth-options.c,v 1.73 2017/05/31 10:54:00 markus Exp $ */ |
Damien Miller | e4340be | 2000-09-16 13:29:08 +1100 | [diff] [blame] | 2 | /* |
| 3 | * Author: Tatu Ylonen <ylo@cs.hut.fi> |
| 4 | * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland |
| 5 | * All rights reserved |
Damien Miller | e4340be | 2000-09-16 13:29:08 +1100 | [diff] [blame] | 6 | * As far as I am concerned, the code I have written for this software |
| 7 | * can be used freely for any purpose. Any derived versions of this |
| 8 | * software must be clearly marked as such, and if the derived work is |
| 9 | * incompatible with the protocol description in the RFC file, it must be |
| 10 | * called by a name other than "ssh" or "Secure Shell". |
| 11 | */ |
| 12 | |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 13 | #include "includes.h" |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 14 | |
Damien Miller | 9f2abc4 | 2006-07-10 20:53:08 +1000 | [diff] [blame] | 15 | #include <sys/types.h> |
| 16 | |
Damien Miller | b8fe89c | 2006-07-24 14:51:00 +1000 | [diff] [blame] | 17 | #include <netdb.h> |
Damien Miller | 9f2abc4 | 2006-07-10 20:53:08 +1000 | [diff] [blame] | 18 | #include <pwd.h> |
Damien Miller | e3476ed | 2006-07-24 14:13:33 +1000 | [diff] [blame] | 19 | #include <string.h> |
Damien Miller | d783435 | 2006-08-05 12:39:39 +1000 | [diff] [blame] | 20 | #include <stdio.h> |
| 21 | #include <stdarg.h> |
Damien Miller | 9f2abc4 | 2006-07-10 20:53:08 +1000 | [diff] [blame] | 22 | |
Damien Miller | b84886b | 2008-05-19 15:05:07 +1000 | [diff] [blame] | 23 | #include "openbsd-compat/sys-queue.h" |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 24 | |
| 25 | #include "key.h" /* XXX for typedef */ |
| 26 | #include "buffer.h" /* XXX for typedef */ |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 27 | #include "xmalloc.h" |
| 28 | #include "match.h" |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 29 | #include "ssherr.h" |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 30 | #include "log.h" |
| 31 | #include "canohost.h" |
djm@openbsd.org | 9576726 | 2016-03-07 19:02:43 +0000 | [diff] [blame] | 32 | #include "packet.h" |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 33 | #include "sshbuf.h" |
Damien Miller | 7acefbb | 2014-07-18 14:11:24 +1000 | [diff] [blame] | 34 | #include "misc.h" |
Ben Lindstrom | c763767 | 2001-06-09 00:36:26 +0000 | [diff] [blame] | 35 | #include "channels.h" |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 36 | #include "servconf.h" |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 37 | #include "sshkey.h" |
Damien Miller | 4e270b0 | 2010-04-16 15:56:21 +1000 | [diff] [blame] | 38 | #include "auth-options.h" |
Damien Miller | d783435 | 2006-08-05 12:39:39 +1000 | [diff] [blame] | 39 | #include "hostfile.h" |
Ben Lindstrom | a574cda | 2002-05-15 16:16:14 +0000 | [diff] [blame] | 40 | #include "auth.h" |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 41 | |
| 42 | /* Flags set authorized_keys flags */ |
| 43 | int no_port_forwarding_flag = 0; |
| 44 | int no_agent_forwarding_flag = 0; |
| 45 | int no_x11_forwarding_flag = 0; |
| 46 | int no_pty_flag = 0; |
Damien Miller | 95e8095 | 2008-03-27 11:03:05 +1100 | [diff] [blame] | 47 | int no_user_rc = 0; |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 48 | int key_is_cert_authority = 0; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 49 | |
| 50 | /* "command=" option. */ |
| 51 | char *forced_command = NULL; |
| 52 | |
| 53 | /* "environment=" options. */ |
| 54 | struct envstring *custom_environment = NULL; |
| 55 | |
Damien Miller | d27b947 | 2005-12-13 19:29:02 +1100 | [diff] [blame] | 56 | /* "tunnel=" option. */ |
| 57 | int forced_tun_device = -1; |
| 58 | |
Damien Miller | 30da344 | 2010-05-10 11:58:03 +1000 | [diff] [blame] | 59 | /* "principals=" option. */ |
| 60 | char *authorized_principals = NULL; |
| 61 | |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 62 | extern ServerOptions options; |
| 63 | |
Ben Lindstrom | 7a2073c | 2002-03-22 02:30:41 +0000 | [diff] [blame] | 64 | void |
Damien Miller | 874d77b | 2000-10-14 16:23:11 +1100 | [diff] [blame] | 65 | auth_clear_options(void) |
| 66 | { |
| 67 | no_agent_forwarding_flag = 0; |
| 68 | no_port_forwarding_flag = 0; |
| 69 | no_pty_flag = 0; |
| 70 | no_x11_forwarding_flag = 0; |
Damien Miller | 95e8095 | 2008-03-27 11:03:05 +1100 | [diff] [blame] | 71 | no_user_rc = 0; |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 72 | key_is_cert_authority = 0; |
Damien Miller | 874d77b | 2000-10-14 16:23:11 +1100 | [diff] [blame] | 73 | while (custom_environment) { |
| 74 | struct envstring *ce = custom_environment; |
| 75 | custom_environment = ce->next; |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 76 | free(ce->s); |
| 77 | free(ce); |
Damien Miller | 874d77b | 2000-10-14 16:23:11 +1100 | [diff] [blame] | 78 | } |
mmcc@openbsd.org | d59ce08 | 2015-12-10 17:08:40 +0000 | [diff] [blame] | 79 | free(forced_command); |
| 80 | forced_command = NULL; |
| 81 | free(authorized_principals); |
| 82 | authorized_principals = NULL; |
Damien Miller | d27b947 | 2005-12-13 19:29:02 +1100 | [diff] [blame] | 83 | forced_tun_device = -1; |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 84 | channel_clear_permitted_opens(); |
Damien Miller | 874d77b | 2000-10-14 16:23:11 +1100 | [diff] [blame] | 85 | } |
| 86 | |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 87 | /* |
djm@openbsd.org | 383f10f | 2015-11-16 00:30:02 +0000 | [diff] [blame] | 88 | * Match flag 'opt' in *optsp, and if allow_negate is set then also match |
| 89 | * 'no-opt'. Returns -1 if option not matched, 1 if option matches or 0 |
| 90 | * if negated option matches. |
| 91 | * If the option or negated option matches, then *optsp is updated to |
| 92 | * point to the first character after the option and, if 'msg' is not NULL |
| 93 | * then a message based on it added via auth_debug_add(). |
| 94 | */ |
| 95 | static int |
| 96 | match_flag(const char *opt, int allow_negate, char **optsp, const char *msg) |
| 97 | { |
| 98 | size_t opt_len = strlen(opt); |
| 99 | char *opts = *optsp; |
| 100 | int negate = 0; |
| 101 | |
| 102 | if (allow_negate && strncasecmp(opts, "no-", 3) == 0) { |
| 103 | opts += 3; |
| 104 | negate = 1; |
| 105 | } |
| 106 | if (strncasecmp(opts, opt, opt_len) == 0) { |
| 107 | *optsp = opts + opt_len; |
| 108 | if (msg != NULL) { |
| 109 | auth_debug_add("%s %s.", msg, |
| 110 | negate ? "disabled" : "enabled"); |
| 111 | } |
| 112 | return negate ? 0 : 1; |
| 113 | } |
| 114 | return -1; |
| 115 | } |
| 116 | |
| 117 | /* |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 118 | * return 1 if access is granted, 0 if not. |
| 119 | * side effect: sets key option flags |
| 120 | */ |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 121 | int |
markus@openbsd.org | 75b8af8 | 2017-05-31 10:54:00 +0000 | [diff] [blame] | 122 | auth_parse_options(struct passwd *pw, char *opts, const char *file, |
| 123 | u_long linenum) |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 124 | { |
djm@openbsd.org | 9576726 | 2016-03-07 19:02:43 +0000 | [diff] [blame] | 125 | struct ssh *ssh = active_state; /* XXX */ |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 126 | const char *cp; |
djm@openbsd.org | 383f10f | 2015-11-16 00:30:02 +0000 | [diff] [blame] | 127 | int i, r; |
Damien Miller | 874d77b | 2000-10-14 16:23:11 +1100 | [diff] [blame] | 128 | |
| 129 | /* reset options */ |
| 130 | auth_clear_options(); |
| 131 | |
Ben Lindstrom | 36d7bd0 | 2001-02-10 22:27:19 +0000 | [diff] [blame] | 132 | if (!opts) |
| 133 | return 1; |
| 134 | |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 135 | while (*opts && *opts != ' ' && *opts != '\t') { |
djm@openbsd.org | 383f10f | 2015-11-16 00:30:02 +0000 | [diff] [blame] | 136 | if ((r = match_flag("cert-authority", 0, &opts, NULL)) != -1) { |
| 137 | key_is_cert_authority = r; |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 138 | goto next_option; |
| 139 | } |
djm@openbsd.org | 383f10f | 2015-11-16 00:30:02 +0000 | [diff] [blame] | 140 | if ((r = match_flag("restrict", 0, &opts, NULL)) != -1) { |
| 141 | auth_debug_add("Key is restricted."); |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 142 | no_port_forwarding_flag = 1; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 143 | no_agent_forwarding_flag = 1; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 144 | no_x11_forwarding_flag = 1; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 145 | no_pty_flag = 1; |
djm@openbsd.org | 383f10f | 2015-11-16 00:30:02 +0000 | [diff] [blame] | 146 | no_user_rc = 1; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 147 | goto next_option; |
| 148 | } |
djm@openbsd.org | 383f10f | 2015-11-16 00:30:02 +0000 | [diff] [blame] | 149 | if ((r = match_flag("port-forwarding", 1, &opts, |
| 150 | "Port forwarding")) != -1) { |
| 151 | no_port_forwarding_flag = r != 1; |
| 152 | goto next_option; |
| 153 | } |
| 154 | if ((r = match_flag("agent-forwarding", 1, &opts, |
| 155 | "Agent forwarding")) != -1) { |
| 156 | no_agent_forwarding_flag = r != 1; |
| 157 | goto next_option; |
| 158 | } |
| 159 | if ((r = match_flag("x11-forwarding", 1, &opts, |
| 160 | "X11 forwarding")) != -1) { |
| 161 | no_x11_forwarding_flag = r != 1; |
| 162 | goto next_option; |
| 163 | } |
| 164 | if ((r = match_flag("pty", 1, &opts, |
| 165 | "PTY allocation")) != -1) { |
| 166 | no_pty_flag = r != 1; |
| 167 | goto next_option; |
| 168 | } |
| 169 | if ((r = match_flag("user-rc", 1, &opts, |
| 170 | "User rc execution")) != -1) { |
| 171 | no_user_rc = r != 1; |
Damien Miller | 95e8095 | 2008-03-27 11:03:05 +1100 | [diff] [blame] | 172 | goto next_option; |
| 173 | } |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 174 | cp = "command=\""; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 175 | if (strncasecmp(opts, cp, strlen(cp)) == 0) { |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 176 | opts += strlen(cp); |
mmcc@openbsd.org | d59ce08 | 2015-12-10 17:08:40 +0000 | [diff] [blame] | 177 | free(forced_command); |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 178 | forced_command = xmalloc(strlen(opts) + 1); |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 179 | i = 0; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 180 | while (*opts) { |
| 181 | if (*opts == '"') |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 182 | break; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 183 | if (*opts == '\\' && opts[1] == '"') { |
| 184 | opts += 2; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 185 | forced_command[i++] = '"'; |
| 186 | continue; |
| 187 | } |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 188 | forced_command[i++] = *opts++; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 189 | } |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 190 | if (!*opts) { |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 191 | debug("%.100s, line %lu: missing end quote", |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 192 | file, linenum); |
Ben Lindstrom | a574cda | 2002-05-15 16:16:14 +0000 | [diff] [blame] | 193 | auth_debug_add("%.100s, line %lu: missing end quote", |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 194 | file, linenum); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 195 | free(forced_command); |
Damien Miller | 056ddf7 | 2001-03-14 10:15:20 +1100 | [diff] [blame] | 196 | forced_command = NULL; |
| 197 | goto bad_option; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 198 | } |
Damien Miller | 9829926 | 2006-07-24 14:01:43 +1000 | [diff] [blame] | 199 | forced_command[i] = '\0'; |
Damien Miller | de53fd0 | 2011-01-06 22:44:18 +1100 | [diff] [blame] | 200 | auth_debug_add("Forced command."); |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 201 | opts++; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 202 | goto next_option; |
| 203 | } |
Damien Miller | 30da344 | 2010-05-10 11:58:03 +1000 | [diff] [blame] | 204 | cp = "principals=\""; |
| 205 | if (strncasecmp(opts, cp, strlen(cp)) == 0) { |
| 206 | opts += strlen(cp); |
mmcc@openbsd.org | d59ce08 | 2015-12-10 17:08:40 +0000 | [diff] [blame] | 207 | free(authorized_principals); |
Damien Miller | 30da344 | 2010-05-10 11:58:03 +1000 | [diff] [blame] | 208 | authorized_principals = xmalloc(strlen(opts) + 1); |
| 209 | i = 0; |
| 210 | while (*opts) { |
| 211 | if (*opts == '"') |
| 212 | break; |
| 213 | if (*opts == '\\' && opts[1] == '"') { |
| 214 | opts += 2; |
| 215 | authorized_principals[i++] = '"'; |
| 216 | continue; |
| 217 | } |
| 218 | authorized_principals[i++] = *opts++; |
| 219 | } |
| 220 | if (!*opts) { |
| 221 | debug("%.100s, line %lu: missing end quote", |
| 222 | file, linenum); |
| 223 | auth_debug_add("%.100s, line %lu: missing end quote", |
| 224 | file, linenum); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 225 | free(authorized_principals); |
Damien Miller | 30da344 | 2010-05-10 11:58:03 +1000 | [diff] [blame] | 226 | authorized_principals = NULL; |
| 227 | goto bad_option; |
| 228 | } |
| 229 | authorized_principals[i] = '\0'; |
| 230 | auth_debug_add("principals: %.900s", |
| 231 | authorized_principals); |
| 232 | opts++; |
| 233 | goto next_option; |
| 234 | } |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 235 | cp = "environment=\""; |
djm@openbsd.org | a42d67b | 2015-05-01 03:20:54 +0000 | [diff] [blame] | 236 | if (strncasecmp(opts, cp, strlen(cp)) == 0) { |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 237 | char *s; |
| 238 | struct envstring *new_envstring; |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 239 | |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 240 | opts += strlen(cp); |
| 241 | s = xmalloc(strlen(opts) + 1); |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 242 | i = 0; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 243 | while (*opts) { |
| 244 | if (*opts == '"') |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 245 | break; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 246 | if (*opts == '\\' && opts[1] == '"') { |
| 247 | opts += 2; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 248 | s[i++] = '"'; |
| 249 | continue; |
| 250 | } |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 251 | s[i++] = *opts++; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 252 | } |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 253 | if (!*opts) { |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 254 | debug("%.100s, line %lu: missing end quote", |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 255 | file, linenum); |
Ben Lindstrom | a574cda | 2002-05-15 16:16:14 +0000 | [diff] [blame] | 256 | auth_debug_add("%.100s, line %lu: missing end quote", |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 257 | file, linenum); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 258 | free(s); |
Damien Miller | 056ddf7 | 2001-03-14 10:15:20 +1100 | [diff] [blame] | 259 | goto bad_option; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 260 | } |
Damien Miller | 9829926 | 2006-07-24 14:01:43 +1000 | [diff] [blame] | 261 | s[i] = '\0'; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 262 | opts++; |
djm@openbsd.org | a42d67b | 2015-05-01 03:20:54 +0000 | [diff] [blame] | 263 | if (options.permit_user_env) { |
| 264 | auth_debug_add("Adding to environment: " |
| 265 | "%.900s", s); |
| 266 | debug("Adding to environment: %.900s", s); |
| 267 | new_envstring = xcalloc(1, |
| 268 | sizeof(*new_envstring)); |
| 269 | new_envstring->s = s; |
| 270 | new_envstring->next = custom_environment; |
| 271 | custom_environment = new_envstring; |
| 272 | s = NULL; |
| 273 | } |
| 274 | free(s); |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 275 | goto next_option; |
| 276 | } |
| 277 | cp = "from=\""; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 278 | if (strncasecmp(opts, cp, strlen(cp)) == 0) { |
djm@openbsd.org | 9576726 | 2016-03-07 19:02:43 +0000 | [diff] [blame] | 279 | const char *remote_ip = ssh_remote_ipaddr(ssh); |
| 280 | const char *remote_host = auth_get_canonical_hostname( |
| 281 | ssh, options.use_dns); |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 282 | char *patterns = xmalloc(strlen(opts) + 1); |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 283 | |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 284 | opts += strlen(cp); |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 285 | i = 0; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 286 | while (*opts) { |
| 287 | if (*opts == '"') |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 288 | break; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 289 | if (*opts == '\\' && opts[1] == '"') { |
| 290 | opts += 2; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 291 | patterns[i++] = '"'; |
| 292 | continue; |
| 293 | } |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 294 | patterns[i++] = *opts++; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 295 | } |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 296 | if (!*opts) { |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 297 | debug("%.100s, line %lu: missing end quote", |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 298 | file, linenum); |
Ben Lindstrom | a574cda | 2002-05-15 16:16:14 +0000 | [diff] [blame] | 299 | auth_debug_add("%.100s, line %lu: missing end quote", |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 300 | file, linenum); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 301 | free(patterns); |
Damien Miller | 056ddf7 | 2001-03-14 10:15:20 +1100 | [diff] [blame] | 302 | goto bad_option; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 303 | } |
Damien Miller | 9829926 | 2006-07-24 14:01:43 +1000 | [diff] [blame] | 304 | patterns[i] = '\0'; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 305 | opts++; |
Darren Tucker | 896ad5a | 2008-06-11 09:34:46 +1000 | [diff] [blame] | 306 | switch (match_host_and_ip(remote_host, remote_ip, |
| 307 | patterns)) { |
| 308 | case 1: |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 309 | free(patterns); |
Darren Tucker | 896ad5a | 2008-06-11 09:34:46 +1000 | [diff] [blame] | 310 | /* Host name matches. */ |
| 311 | goto next_option; |
| 312 | case -1: |
| 313 | debug("%.100s, line %lu: invalid criteria", |
| 314 | file, linenum); |
| 315 | auth_debug_add("%.100s, line %lu: " |
| 316 | "invalid criteria", file, linenum); |
| 317 | /* FALLTHROUGH */ |
| 318 | case 0: |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 319 | free(patterns); |
Damien Miller | 996acd2 | 2003-04-09 20:59:48 +1000 | [diff] [blame] | 320 | logit("Authentication tried for %.100s with " |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 321 | "correct key but not from a permitted " |
| 322 | "host (host=%.200s, ip=%.200s).", |
| 323 | pw->pw_name, remote_host, remote_ip); |
Ben Lindstrom | a574cda | 2002-05-15 16:16:14 +0000 | [diff] [blame] | 324 | auth_debug_add("Your host '%.200s' is not " |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 325 | "permitted to use this key for login.", |
| 326 | remote_host); |
Darren Tucker | 896ad5a | 2008-06-11 09:34:46 +1000 | [diff] [blame] | 327 | break; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 328 | } |
Darren Tucker | 896ad5a | 2008-06-11 09:34:46 +1000 | [diff] [blame] | 329 | /* deny access */ |
| 330 | return 0; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 331 | } |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 332 | cp = "permitopen=\""; |
| 333 | if (strncasecmp(opts, cp, strlen(cp)) == 0) { |
Damien Miller | f91ee4c | 2005-03-01 21:24:33 +1100 | [diff] [blame] | 334 | char *host, *p; |
Damien Miller | e37dde0 | 2009-01-28 16:33:01 +1100 | [diff] [blame] | 335 | int port; |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 336 | char *patterns = xmalloc(strlen(opts) + 1); |
| 337 | |
| 338 | opts += strlen(cp); |
| 339 | i = 0; |
| 340 | while (*opts) { |
| 341 | if (*opts == '"') |
| 342 | break; |
| 343 | if (*opts == '\\' && opts[1] == '"') { |
| 344 | opts += 2; |
| 345 | patterns[i++] = '"'; |
| 346 | continue; |
| 347 | } |
| 348 | patterns[i++] = *opts++; |
| 349 | } |
| 350 | if (!*opts) { |
| 351 | debug("%.100s, line %lu: missing end quote", |
| 352 | file, linenum); |
Damien Miller | f91ee4c | 2005-03-01 21:24:33 +1100 | [diff] [blame] | 353 | auth_debug_add("%.100s, line %lu: missing " |
| 354 | "end quote", file, linenum); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 355 | free(patterns); |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 356 | goto bad_option; |
| 357 | } |
Damien Miller | 9829926 | 2006-07-24 14:01:43 +1000 | [diff] [blame] | 358 | patterns[i] = '\0'; |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 359 | opts++; |
Damien Miller | f91ee4c | 2005-03-01 21:24:33 +1100 | [diff] [blame] | 360 | p = patterns; |
Damien Miller | 7acefbb | 2014-07-18 14:11:24 +1000 | [diff] [blame] | 361 | /* XXX - add streamlocal support */ |
Damien Miller | f91ee4c | 2005-03-01 21:24:33 +1100 | [diff] [blame] | 362 | host = hpdelim(&p); |
| 363 | if (host == NULL || strlen(host) >= NI_MAXHOST) { |
| 364 | debug("%.100s, line %lu: Bad permitopen " |
Darren Tucker | 90b9e02 | 2005-03-14 23:08:50 +1100 | [diff] [blame] | 365 | "specification <%.100s>", file, linenum, |
Damien Miller | f91ee4c | 2005-03-01 21:24:33 +1100 | [diff] [blame] | 366 | patterns); |
Ben Lindstrom | a574cda | 2002-05-15 16:16:14 +0000 | [diff] [blame] | 367 | auth_debug_add("%.100s, line %lu: " |
Damien Miller | f91ee4c | 2005-03-01 21:24:33 +1100 | [diff] [blame] | 368 | "Bad permitopen specification", file, |
| 369 | linenum); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 370 | free(patterns); |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 371 | goto bad_option; |
| 372 | } |
Darren Tucker | 47eede7 | 2005-03-14 23:08:12 +1100 | [diff] [blame] | 373 | host = cleanhostname(host); |
Darren Tucker | 1338b9e | 2011-10-02 18:57:35 +1100 | [diff] [blame] | 374 | if (p == NULL || (port = permitopen_port(p)) < 0) { |
Damien Miller | f91ee4c | 2005-03-01 21:24:33 +1100 | [diff] [blame] | 375 | debug("%.100s, line %lu: Bad permitopen port " |
| 376 | "<%.100s>", file, linenum, p ? p : ""); |
Ben Lindstrom | a574cda | 2002-05-15 16:16:14 +0000 | [diff] [blame] | 377 | auth_debug_add("%.100s, line %lu: " |
Ben Lindstrom | d71ba57 | 2001-09-12 18:03:31 +0000 | [diff] [blame] | 378 | "Bad permitopen port", file, linenum); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 379 | free(patterns); |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 380 | goto bad_option; |
| 381 | } |
Damien Miller | aa5b3f8 | 2012-12-03 09:50:54 +1100 | [diff] [blame] | 382 | if ((options.allow_tcp_forwarding & FORWARD_LOCAL) != 0) |
Ben Lindstrom | d71ba57 | 2001-09-12 18:03:31 +0000 | [diff] [blame] | 383 | channel_add_permitted_opens(host, port); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 384 | free(patterns); |
Ben Lindstrom | 7bb8b49 | 2001-03-17 00:47:54 +0000 | [diff] [blame] | 385 | goto next_option; |
| 386 | } |
Damien Miller | d27b947 | 2005-12-13 19:29:02 +1100 | [diff] [blame] | 387 | cp = "tunnel=\""; |
| 388 | if (strncasecmp(opts, cp, strlen(cp)) == 0) { |
| 389 | char *tun = NULL; |
| 390 | opts += strlen(cp); |
| 391 | tun = xmalloc(strlen(opts) + 1); |
| 392 | i = 0; |
| 393 | while (*opts) { |
| 394 | if (*opts == '"') |
| 395 | break; |
| 396 | tun[i++] = *opts++; |
| 397 | } |
| 398 | if (!*opts) { |
| 399 | debug("%.100s, line %lu: missing end quote", |
| 400 | file, linenum); |
| 401 | auth_debug_add("%.100s, line %lu: missing end quote", |
| 402 | file, linenum); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 403 | free(tun); |
Damien Miller | d27b947 | 2005-12-13 19:29:02 +1100 | [diff] [blame] | 404 | forced_tun_device = -1; |
| 405 | goto bad_option; |
| 406 | } |
Damien Miller | 9829926 | 2006-07-24 14:01:43 +1000 | [diff] [blame] | 407 | tun[i] = '\0'; |
Damien Miller | d27b947 | 2005-12-13 19:29:02 +1100 | [diff] [blame] | 408 | forced_tun_device = a2tun(tun, NULL); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 409 | free(tun); |
Damien Miller | 7b58e80 | 2005-12-13 19:33:19 +1100 | [diff] [blame] | 410 | if (forced_tun_device == SSH_TUNID_ERR) { |
Damien Miller | d27b947 | 2005-12-13 19:29:02 +1100 | [diff] [blame] | 411 | debug("%.100s, line %lu: invalid tun device", |
| 412 | file, linenum); |
| 413 | auth_debug_add("%.100s, line %lu: invalid tun device", |
| 414 | file, linenum); |
| 415 | forced_tun_device = -1; |
| 416 | goto bad_option; |
| 417 | } |
| 418 | auth_debug_add("Forced tun device: %d", forced_tun_device); |
| 419 | opts++; |
| 420 | goto next_option; |
| 421 | } |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 422 | next_option: |
| 423 | /* |
| 424 | * Skip the comma, and move to the next option |
| 425 | * (or break out if there are no more). |
| 426 | */ |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 427 | if (!*opts) |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 428 | fatal("Bugs in auth-options.c option processing."); |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 429 | if (*opts == ' ' || *opts == '\t') |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 430 | break; /* End of options. */ |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 431 | if (*opts != ',') |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 432 | goto bad_option; |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 433 | opts++; |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 434 | /* Process the next option. */ |
| 435 | } |
Ben Lindstrom | 7a2073c | 2002-03-22 02:30:41 +0000 | [diff] [blame] | 436 | |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 437 | /* grant access */ |
| 438 | return 1; |
| 439 | |
| 440 | bad_option: |
Damien Miller | 996acd2 | 2003-04-09 20:59:48 +1000 | [diff] [blame] | 441 | logit("Bad options in %.100s file, line %lu: %.50s", |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 442 | file, linenum, opts); |
Ben Lindstrom | a574cda | 2002-05-15 16:16:14 +0000 | [diff] [blame] | 443 | auth_debug_add("Bad options in %.100s file, line %lu: %.50s", |
Damien Miller | 3380426 | 2001-02-04 23:20:18 +1100 | [diff] [blame] | 444 | file, linenum, opts); |
Ben Lindstrom | 7a2073c | 2002-03-22 02:30:41 +0000 | [diff] [blame] | 445 | |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 446 | /* deny access */ |
| 447 | return 0; |
| 448 | } |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 449 | |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 450 | #define OPTIONS_CRITICAL 1 |
| 451 | #define OPTIONS_EXTENSIONS 2 |
| 452 | static int |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 453 | parse_option_list(struct sshbuf *oblob, struct passwd *pw, |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 454 | u_int which, int crit, |
| 455 | int *cert_no_port_forwarding_flag, |
| 456 | int *cert_no_agent_forwarding_flag, |
| 457 | int *cert_no_x11_forwarding_flag, |
| 458 | int *cert_no_pty_flag, |
| 459 | int *cert_no_user_rc, |
| 460 | char **cert_forced_command, |
| 461 | int *cert_source_address_done) |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 462 | { |
djm@openbsd.org | 9576726 | 2016-03-07 19:02:43 +0000 | [diff] [blame] | 463 | struct ssh *ssh = active_state; /* XXX */ |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 464 | char *command, *allowed; |
| 465 | const char *remote_ip; |
Damien Miller | ce98654 | 2013-07-18 16:12:44 +1000 | [diff] [blame] | 466 | char *name = NULL; |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 467 | struct sshbuf *c = NULL, *data = NULL; |
| 468 | int r, ret = -1, result, found; |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 469 | |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 470 | if ((c = sshbuf_fromb(oblob)) == NULL) { |
| 471 | error("%s: sshbuf_fromb failed", __func__); |
| 472 | goto out; |
| 473 | } |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 474 | |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 475 | while (sshbuf_len(c) > 0) { |
| 476 | sshbuf_free(data); |
| 477 | data = NULL; |
| 478 | if ((r = sshbuf_get_cstring(c, &name, NULL)) != 0 || |
| 479 | (r = sshbuf_froms(c, &data)) != 0) { |
| 480 | error("Unable to parse certificate options: %s", |
| 481 | ssh_err(r)); |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 482 | goto out; |
| 483 | } |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 484 | debug3("found certificate option \"%.100s\" len %zu", |
| 485 | name, sshbuf_len(data)); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 486 | found = 0; |
| 487 | if ((which & OPTIONS_EXTENSIONS) != 0) { |
| 488 | if (strcmp(name, "permit-X11-forwarding") == 0) { |
| 489 | *cert_no_x11_forwarding_flag = 0; |
| 490 | found = 1; |
| 491 | } else if (strcmp(name, |
| 492 | "permit-agent-forwarding") == 0) { |
| 493 | *cert_no_agent_forwarding_flag = 0; |
| 494 | found = 1; |
| 495 | } else if (strcmp(name, |
| 496 | "permit-port-forwarding") == 0) { |
| 497 | *cert_no_port_forwarding_flag = 0; |
| 498 | found = 1; |
| 499 | } else if (strcmp(name, "permit-pty") == 0) { |
| 500 | *cert_no_pty_flag = 0; |
| 501 | found = 1; |
| 502 | } else if (strcmp(name, "permit-user-rc") == 0) { |
| 503 | *cert_no_user_rc = 0; |
| 504 | found = 1; |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 505 | } |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 506 | } |
| 507 | if (!found && (which & OPTIONS_CRITICAL) != 0) { |
| 508 | if (strcmp(name, "force-command") == 0) { |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 509 | if ((r = sshbuf_get_cstring(data, &command, |
| 510 | NULL)) != 0) { |
| 511 | error("Unable to parse \"%s\" " |
| 512 | "section: %s", name, ssh_err(r)); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 513 | goto out; |
| 514 | } |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 515 | if (*cert_forced_command != NULL) { |
| 516 | error("Certificate has multiple " |
| 517 | "force-command options"); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 518 | free(command); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 519 | goto out; |
| 520 | } |
| 521 | *cert_forced_command = command; |
| 522 | found = 1; |
Damien Miller | 4139657 | 2010-03-04 21:51:11 +1100 | [diff] [blame] | 523 | } |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 524 | if (strcmp(name, "source-address") == 0) { |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 525 | if ((r = sshbuf_get_cstring(data, &allowed, |
| 526 | NULL)) != 0) { |
| 527 | error("Unable to parse \"%s\" " |
| 528 | "section: %s", name, ssh_err(r)); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 529 | goto out; |
| 530 | } |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 531 | if ((*cert_source_address_done)++) { |
| 532 | error("Certificate has multiple " |
| 533 | "source-address options"); |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 534 | free(allowed); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 535 | goto out; |
| 536 | } |
djm@openbsd.org | 9576726 | 2016-03-07 19:02:43 +0000 | [diff] [blame] | 537 | remote_ip = ssh_remote_ipaddr(ssh); |
Damien Miller | bf25d11 | 2013-12-29 17:44:56 +1100 | [diff] [blame] | 538 | result = addr_match_cidr_list(remote_ip, |
| 539 | allowed); |
| 540 | free(allowed); |
| 541 | switch (result) { |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 542 | case 1: |
| 543 | /* accepted */ |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 544 | break; |
| 545 | case 0: |
| 546 | /* no match */ |
| 547 | logit("Authentication tried for %.100s " |
| 548 | "with valid certificate but not " |
| 549 | "from a permitted host " |
| 550 | "(ip=%.200s).", pw->pw_name, |
| 551 | remote_ip); |
| 552 | auth_debug_add("Your address '%.200s' " |
| 553 | "is not permitted to use this " |
| 554 | "certificate for login.", |
| 555 | remote_ip); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 556 | goto out; |
| 557 | case -1: |
Damien Miller | bf25d11 | 2013-12-29 17:44:56 +1100 | [diff] [blame] | 558 | default: |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 559 | error("Certificate source-address " |
| 560 | "contents invalid"); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 561 | goto out; |
| 562 | } |
| 563 | found = 1; |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 564 | } |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 565 | } |
| 566 | |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 567 | if (!found) { |
| 568 | if (crit) { |
| 569 | error("Certificate critical option \"%s\" " |
| 570 | "is not supported", name); |
| 571 | goto out; |
| 572 | } else { |
| 573 | logit("Certificate extension \"%s\" " |
| 574 | "is not supported", name); |
| 575 | } |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 576 | } else if (sshbuf_len(data) != 0) { |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 577 | error("Certificate option \"%s\" corrupt " |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 578 | "(extra data)", name); |
| 579 | goto out; |
| 580 | } |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 581 | free(name); |
Damien Miller | ce98654 | 2013-07-18 16:12:44 +1000 | [diff] [blame] | 582 | name = NULL; |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 583 | } |
Damien Miller | 4e270b0 | 2010-04-16 15:56:21 +1000 | [diff] [blame] | 584 | /* successfully parsed all options */ |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 585 | ret = 0; |
| 586 | |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 587 | out: |
| 588 | if (ret != 0 && |
| 589 | cert_forced_command != NULL && |
| 590 | *cert_forced_command != NULL) { |
Darren Tucker | a627d42 | 2013-06-02 07:31:17 +1000 | [diff] [blame] | 591 | free(*cert_forced_command); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 592 | *cert_forced_command = NULL; |
| 593 | } |
mmcc@openbsd.org | d59ce08 | 2015-12-10 17:08:40 +0000 | [diff] [blame] | 594 | free(name); |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 595 | sshbuf_free(data); |
| 596 | sshbuf_free(c); |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 597 | return ret; |
| 598 | } |
| 599 | |
| 600 | /* |
| 601 | * Set options from critical certificate options. These supersede user key |
| 602 | * options so this must be called after auth_parse_options(). |
| 603 | */ |
| 604 | int |
djm@openbsd.org | fd6dcef | 2016-11-30 02:57:40 +0000 | [diff] [blame] | 605 | auth_cert_options(struct sshkey *k, struct passwd *pw, const char **reason) |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 606 | { |
| 607 | int cert_no_port_forwarding_flag = 1; |
| 608 | int cert_no_agent_forwarding_flag = 1; |
| 609 | int cert_no_x11_forwarding_flag = 1; |
| 610 | int cert_no_pty_flag = 1; |
| 611 | int cert_no_user_rc = 1; |
| 612 | char *cert_forced_command = NULL; |
| 613 | int cert_source_address_done = 0; |
| 614 | |
djm@openbsd.org | fd6dcef | 2016-11-30 02:57:40 +0000 | [diff] [blame] | 615 | *reason = "invalid certificate options"; |
| 616 | |
djm@openbsd.org | c28fc62 | 2015-07-03 03:43:18 +0000 | [diff] [blame] | 617 | /* Separate options and extensions for v01 certs */ |
| 618 | if (parse_option_list(k->cert->critical, pw, |
| 619 | OPTIONS_CRITICAL, 1, NULL, NULL, NULL, NULL, NULL, |
| 620 | &cert_forced_command, |
| 621 | &cert_source_address_done) == -1) |
| 622 | return -1; |
| 623 | if (parse_option_list(k->cert->extensions, pw, |
| 624 | OPTIONS_EXTENSIONS, 0, |
| 625 | &cert_no_port_forwarding_flag, |
| 626 | &cert_no_agent_forwarding_flag, |
| 627 | &cert_no_x11_forwarding_flag, |
| 628 | &cert_no_pty_flag, |
| 629 | &cert_no_user_rc, |
| 630 | NULL, NULL) == -1) |
| 631 | return -1; |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 632 | |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 633 | no_port_forwarding_flag |= cert_no_port_forwarding_flag; |
| 634 | no_agent_forwarding_flag |= cert_no_agent_forwarding_flag; |
| 635 | no_x11_forwarding_flag |= cert_no_x11_forwarding_flag; |
| 636 | no_pty_flag |= cert_no_pty_flag; |
| 637 | no_user_rc |= cert_no_user_rc; |
djm@openbsd.org | fd6dcef | 2016-11-30 02:57:40 +0000 | [diff] [blame] | 638 | /* |
| 639 | * Only permit both CA and key option forced-command if they match. |
| 640 | * Otherwise refuse the certificate. |
| 641 | */ |
| 642 | if (cert_forced_command != NULL && forced_command != NULL) { |
| 643 | if (strcmp(forced_command, cert_forced_command) == 0) { |
| 644 | free(forced_command); |
| 645 | forced_command = cert_forced_command; |
| 646 | } else { |
| 647 | *reason = "certificate and key options forced command " |
| 648 | "do not match"; |
| 649 | free(cert_forced_command); |
| 650 | return -1; |
| 651 | } |
| 652 | } else if (cert_forced_command != NULL) |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 653 | forced_command = cert_forced_command; |
djm@openbsd.org | fd6dcef | 2016-11-30 02:57:40 +0000 | [diff] [blame] | 654 | /* success */ |
| 655 | *reason = NULL; |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 656 | return 0; |
Damien Miller | 0a80ca1 | 2010-02-27 07:55:05 +1100 | [diff] [blame] | 657 | } |
| 658 | |