djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 1 | /* $OpenBSD: auth-options.c,v 1.90 2019/11/25 00:54:23 djm Exp $ */ |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 2 | /* |
| 3 | * Copyright (c) 2018 Damien Miller <djm@mindrot.org> |
| 4 | * |
| 5 | * Permission to use, copy, modify, and distribute this software for any |
| 6 | * purpose with or without fee is hereby granted, provided that the above |
| 7 | * copyright notice and this permission notice appear in all copies. |
| 8 | * |
| 9 | * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES |
| 10 | * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF |
| 11 | * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR |
| 12 | * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES |
| 13 | * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN |
| 14 | * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF |
| 15 | * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. |
| 16 | */ |
Damien Miller | e4340be | 2000-09-16 13:29:08 +1100 | [diff] [blame] | 17 | |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 18 | #include "includes.h" |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 19 | |
Damien Miller | 9f2abc4 | 2006-07-10 20:53:08 +1000 | [diff] [blame] | 20 | #include <sys/types.h> |
| 21 | |
djm@openbsd.org | be02d7c | 2019-09-06 04:53:27 +0000 | [diff] [blame] | 22 | #include <stdlib.h> |
Damien Miller | b8fe89c | 2006-07-24 14:51:00 +1000 | [diff] [blame] | 23 | #include <netdb.h> |
Damien Miller | 9f2abc4 | 2006-07-10 20:53:08 +1000 | [diff] [blame] | 24 | #include <pwd.h> |
Damien Miller | e3476ed | 2006-07-24 14:13:33 +1000 | [diff] [blame] | 25 | #include <string.h> |
Damien Miller | d783435 | 2006-08-05 12:39:39 +1000 | [diff] [blame] | 26 | #include <stdio.h> |
| 27 | #include <stdarg.h> |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 28 | #include <ctype.h> |
| 29 | #include <limits.h> |
Damien Miller | 9f2abc4 | 2006-07-10 20:53:08 +1000 | [diff] [blame] | 30 | |
Damien Miller | b84886b | 2008-05-19 15:05:07 +1000 | [diff] [blame] | 31 | #include "openbsd-compat/sys-queue.h" |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 32 | |
Damien Miller | f6d9e22 | 2000-06-18 14:50:44 +1000 | [diff] [blame] | 33 | #include "xmalloc.h" |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 34 | #include "ssherr.h" |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 35 | #include "log.h" |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 36 | #include "sshbuf.h" |
Damien Miller | 7acefbb | 2014-07-18 14:11:24 +1000 | [diff] [blame] | 37 | #include "misc.h" |
markus@openbsd.org | ae8b463 | 2015-01-14 10:30:34 +0000 | [diff] [blame] | 38 | #include "sshkey.h" |
djm@openbsd.org | 7c85685 | 2018-03-03 03:15:51 +0000 | [diff] [blame] | 39 | #include "match.h" |
| 40 | #include "ssh2.h" |
Damien Miller | 4e270b0 | 2010-04-16 15:56:21 +1000 | [diff] [blame] | 41 | #include "auth-options.h" |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 42 | |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 43 | static int |
| 44 | dup_strings(char ***dstp, size_t *ndstp, char **src, size_t nsrc) |
| 45 | { |
| 46 | char **dst; |
| 47 | size_t i, j; |
| 48 | |
| 49 | *dstp = NULL; |
| 50 | *ndstp = 0; |
| 51 | if (nsrc == 0) |
| 52 | return 0; |
| 53 | |
| 54 | if ((dst = calloc(nsrc, sizeof(*src))) == NULL) |
| 55 | return -1; |
| 56 | for (i = 0; i < nsrc; i++) { |
| 57 | if ((dst[i] = strdup(src[i])) == NULL) { |
| 58 | for (j = 0; j < i; j++) |
| 59 | free(dst[j]); |
| 60 | free(dst); |
| 61 | return -1; |
| 62 | } |
| 63 | } |
| 64 | /* success */ |
| 65 | *dstp = dst; |
| 66 | *ndstp = nsrc; |
| 67 | return 0; |
| 68 | } |
| 69 | |
| 70 | #define OPTIONS_CRITICAL 1 |
| 71 | #define OPTIONS_EXTENSIONS 2 |
| 72 | static int |
| 73 | cert_option_list(struct sshauthopt *opts, struct sshbuf *oblob, |
| 74 | u_int which, int crit) |
| 75 | { |
| 76 | char *command, *allowed; |
| 77 | char *name = NULL; |
| 78 | struct sshbuf *c = NULL, *data = NULL; |
| 79 | int r, ret = -1, found; |
| 80 | |
| 81 | if ((c = sshbuf_fromb(oblob)) == NULL) { |
| 82 | error("%s: sshbuf_fromb failed", __func__); |
| 83 | goto out; |
| 84 | } |
| 85 | |
| 86 | while (sshbuf_len(c) > 0) { |
| 87 | sshbuf_free(data); |
| 88 | data = NULL; |
| 89 | if ((r = sshbuf_get_cstring(c, &name, NULL)) != 0 || |
| 90 | (r = sshbuf_froms(c, &data)) != 0) { |
| 91 | error("Unable to parse certificate options: %s", |
| 92 | ssh_err(r)); |
| 93 | goto out; |
| 94 | } |
| 95 | debug3("found certificate option \"%.100s\" len %zu", |
| 96 | name, sshbuf_len(data)); |
| 97 | found = 0; |
| 98 | if ((which & OPTIONS_EXTENSIONS) != 0) { |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 99 | if (strcmp(name, "no-touch-required") == 0) { |
| 100 | opts->no_require_user_presence = 1; |
| 101 | found = 1; |
| 102 | } else if (strcmp(name, "permit-X11-forwarding") == 0) { |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 103 | opts->permit_x11_forwarding_flag = 1; |
| 104 | found = 1; |
| 105 | } else if (strcmp(name, |
| 106 | "permit-agent-forwarding") == 0) { |
| 107 | opts->permit_agent_forwarding_flag = 1; |
| 108 | found = 1; |
| 109 | } else if (strcmp(name, |
| 110 | "permit-port-forwarding") == 0) { |
| 111 | opts->permit_port_forwarding_flag = 1; |
| 112 | found = 1; |
| 113 | } else if (strcmp(name, "permit-pty") == 0) { |
| 114 | opts->permit_pty_flag = 1; |
| 115 | found = 1; |
| 116 | } else if (strcmp(name, "permit-user-rc") == 0) { |
| 117 | opts->permit_user_rc = 1; |
| 118 | found = 1; |
| 119 | } |
| 120 | } |
| 121 | if (!found && (which & OPTIONS_CRITICAL) != 0) { |
| 122 | if (strcmp(name, "force-command") == 0) { |
| 123 | if ((r = sshbuf_get_cstring(data, &command, |
| 124 | NULL)) != 0) { |
| 125 | error("Unable to parse \"%s\" " |
| 126 | "section: %s", name, ssh_err(r)); |
| 127 | goto out; |
| 128 | } |
| 129 | if (opts->force_command != NULL) { |
| 130 | error("Certificate has multiple " |
| 131 | "force-command options"); |
| 132 | free(command); |
| 133 | goto out; |
| 134 | } |
| 135 | opts->force_command = command; |
| 136 | found = 1; |
| 137 | } |
| 138 | if (strcmp(name, "source-address") == 0) { |
| 139 | if ((r = sshbuf_get_cstring(data, &allowed, |
| 140 | NULL)) != 0) { |
| 141 | error("Unable to parse \"%s\" " |
| 142 | "section: %s", name, ssh_err(r)); |
| 143 | goto out; |
| 144 | } |
| 145 | if (opts->required_from_host_cert != NULL) { |
| 146 | error("Certificate has multiple " |
| 147 | "source-address options"); |
| 148 | free(allowed); |
| 149 | goto out; |
| 150 | } |
| 151 | /* Check syntax */ |
| 152 | if (addr_match_cidr_list(NULL, allowed) == -1) { |
| 153 | error("Certificate source-address " |
| 154 | "contents invalid"); |
| 155 | goto out; |
| 156 | } |
| 157 | opts->required_from_host_cert = allowed; |
| 158 | found = 1; |
| 159 | } |
| 160 | } |
| 161 | |
| 162 | if (!found) { |
| 163 | if (crit) { |
| 164 | error("Certificate critical option \"%s\" " |
| 165 | "is not supported", name); |
| 166 | goto out; |
| 167 | } else { |
| 168 | logit("Certificate extension \"%s\" " |
| 169 | "is not supported", name); |
| 170 | } |
| 171 | } else if (sshbuf_len(data) != 0) { |
| 172 | error("Certificate option \"%s\" corrupt " |
| 173 | "(extra data)", name); |
| 174 | goto out; |
| 175 | } |
| 176 | free(name); |
| 177 | name = NULL; |
| 178 | } |
| 179 | /* successfully parsed all options */ |
| 180 | ret = 0; |
| 181 | |
| 182 | out: |
| 183 | free(name); |
| 184 | sshbuf_free(data); |
| 185 | sshbuf_free(c); |
| 186 | return ret; |
| 187 | } |
| 188 | |
| 189 | struct sshauthopt * |
| 190 | sshauthopt_new(void) |
| 191 | { |
| 192 | struct sshauthopt *ret; |
| 193 | |
| 194 | if ((ret = calloc(1, sizeof(*ret))) == NULL) |
| 195 | return NULL; |
| 196 | ret->force_tun_device = -1; |
| 197 | return ret; |
| 198 | } |
| 199 | |
| 200 | void |
| 201 | sshauthopt_free(struct sshauthopt *opts) |
| 202 | { |
| 203 | size_t i; |
| 204 | |
| 205 | if (opts == NULL) |
| 206 | return; |
| 207 | |
| 208 | free(opts->cert_principals); |
| 209 | free(opts->force_command); |
| 210 | free(opts->required_from_host_cert); |
| 211 | free(opts->required_from_host_keys); |
| 212 | |
| 213 | for (i = 0; i < opts->nenv; i++) |
| 214 | free(opts->env[i]); |
| 215 | free(opts->env); |
| 216 | |
| 217 | for (i = 0; i < opts->npermitopen; i++) |
| 218 | free(opts->permitopen[i]); |
| 219 | free(opts->permitopen); |
| 220 | |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 221 | for (i = 0; i < opts->npermitlisten; i++) |
| 222 | free(opts->permitlisten[i]); |
| 223 | free(opts->permitlisten); |
| 224 | |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 225 | explicit_bzero(opts, sizeof(*opts)); |
| 226 | free(opts); |
| 227 | } |
| 228 | |
| 229 | struct sshauthopt * |
| 230 | sshauthopt_new_with_keys_defaults(void) |
| 231 | { |
| 232 | struct sshauthopt *ret = NULL; |
| 233 | |
| 234 | if ((ret = sshauthopt_new()) == NULL) |
| 235 | return NULL; |
| 236 | |
| 237 | /* Defaults for authorized_keys flags */ |
| 238 | ret->permit_port_forwarding_flag = 1; |
| 239 | ret->permit_agent_forwarding_flag = 1; |
| 240 | ret->permit_x11_forwarding_flag = 1; |
| 241 | ret->permit_pty_flag = 1; |
| 242 | ret->permit_user_rc = 1; |
| 243 | return ret; |
| 244 | } |
| 245 | |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 246 | /* |
| 247 | * Parse and record a permitopen/permitlisten directive. |
| 248 | * Return 0 on success. Return -1 on failure and sets *errstrp to error reason. |
| 249 | */ |
| 250 | static int |
djm@openbsd.org | 87ddd67 | 2018-06-19 02:59:41 +0000 | [diff] [blame] | 251 | handle_permit(const char **optsp, int allow_bare_port, |
| 252 | char ***permitsp, size_t *npermitsp, const char **errstrp) |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 253 | { |
| 254 | char *opt, *tmp, *cp, *host, **permits = *permitsp; |
| 255 | size_t npermits = *npermitsp; |
| 256 | const char *errstr = "unknown error"; |
| 257 | |
djm@openbsd.org | 5b2b79f | 2019-07-09 04:15:00 +0000 | [diff] [blame] | 258 | if (npermits > SSH_AUTHOPT_PERMIT_MAX) { |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 259 | *errstrp = "too many permission directives"; |
| 260 | return -1; |
| 261 | } |
djm@openbsd.org | 6d41815 | 2018-06-07 09:26:42 +0000 | [diff] [blame] | 262 | if ((opt = opt_dequote(optsp, &errstr)) == NULL) { |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 263 | return -1; |
| 264 | } |
djm@openbsd.org | 87ddd67 | 2018-06-19 02:59:41 +0000 | [diff] [blame] | 265 | if (allow_bare_port && strchr(opt, ':') == NULL) { |
| 266 | /* |
| 267 | * Allow a bare port number in permitlisten to indicate a |
| 268 | * listen_host wildcard. |
| 269 | */ |
deraadt@openbsd.org | 5cdbaa7 | 2019-06-27 18:03:37 +0000 | [diff] [blame] | 270 | if (asprintf(&tmp, "*:%s", opt) == -1) { |
dtucker@openbsd.org | b36ee3f | 2019-09-13 04:36:43 +0000 | [diff] [blame] | 271 | free(opt); |
djm@openbsd.org | 87ddd67 | 2018-06-19 02:59:41 +0000 | [diff] [blame] | 272 | *errstrp = "memory allocation failed"; |
| 273 | return -1; |
| 274 | } |
| 275 | free(opt); |
| 276 | opt = tmp; |
| 277 | } |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 278 | if ((tmp = strdup(opt)) == NULL) { |
| 279 | free(opt); |
| 280 | *errstrp = "memory allocation failed"; |
| 281 | return -1; |
| 282 | } |
| 283 | cp = tmp; |
| 284 | /* validate syntax before recording it. */ |
| 285 | host = hpdelim(&cp); |
| 286 | if (host == NULL || strlen(host) >= NI_MAXHOST) { |
| 287 | free(tmp); |
| 288 | free(opt); |
| 289 | *errstrp = "invalid permission hostname"; |
| 290 | return -1; |
| 291 | } |
| 292 | /* |
| 293 | * don't want to use permitopen_port to avoid |
| 294 | * dependency on channels.[ch] here. |
| 295 | */ |
| 296 | if (cp == NULL || |
| 297 | (strcmp(cp, "*") != 0 && a2port(cp) <= 0)) { |
| 298 | free(tmp); |
| 299 | free(opt); |
| 300 | *errstrp = "invalid permission port"; |
| 301 | return -1; |
| 302 | } |
| 303 | /* XXX - add streamlocal support */ |
| 304 | free(tmp); |
| 305 | /* Record it */ |
| 306 | if ((permits = recallocarray(permits, npermits, npermits + 1, |
| 307 | sizeof(*permits))) == NULL) { |
| 308 | free(opt); |
| 309 | /* NB. don't update *permitsp if alloc fails */ |
| 310 | *errstrp = "memory allocation failed"; |
| 311 | return -1; |
| 312 | } |
| 313 | permits[npermits++] = opt; |
| 314 | *permitsp = permits; |
| 315 | *npermitsp = npermits; |
| 316 | return 0; |
| 317 | } |
| 318 | |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 319 | struct sshauthopt * |
| 320 | sshauthopt_parse(const char *opts, const char **errstrp) |
| 321 | { |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 322 | char **oarray, *opt, *cp, *tmp; |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 323 | int r; |
| 324 | struct sshauthopt *ret = NULL; |
| 325 | const char *errstr = "unknown error"; |
djm@openbsd.org | bf0fbf2 | 2018-03-12 00:52:01 +0000 | [diff] [blame] | 326 | uint64_t valid_before; |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 327 | |
| 328 | if (errstrp != NULL) |
| 329 | *errstrp = NULL; |
| 330 | if ((ret = sshauthopt_new_with_keys_defaults()) == NULL) |
| 331 | goto alloc_fail; |
| 332 | |
| 333 | if (opts == NULL) |
| 334 | return ret; |
| 335 | |
| 336 | while (*opts && *opts != ' ' && *opts != '\t') { |
| 337 | /* flag options */ |
| 338 | if ((r = opt_flag("restrict", 0, &opts)) != -1) { |
| 339 | ret->restricted = 1; |
| 340 | ret->permit_port_forwarding_flag = 0; |
| 341 | ret->permit_agent_forwarding_flag = 0; |
| 342 | ret->permit_x11_forwarding_flag = 0; |
| 343 | ret->permit_pty_flag = 0; |
| 344 | ret->permit_user_rc = 0; |
| 345 | } else if ((r = opt_flag("cert-authority", 0, &opts)) != -1) { |
| 346 | ret->cert_authority = r; |
| 347 | } else if ((r = opt_flag("port-forwarding", 1, &opts)) != -1) { |
| 348 | ret->permit_port_forwarding_flag = r == 1; |
| 349 | } else if ((r = opt_flag("agent-forwarding", 1, &opts)) != -1) { |
| 350 | ret->permit_agent_forwarding_flag = r == 1; |
| 351 | } else if ((r = opt_flag("x11-forwarding", 1, &opts)) != -1) { |
| 352 | ret->permit_x11_forwarding_flag = r == 1; |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 353 | } else if ((r = opt_flag("touch-required", 1, &opts)) != -1) { |
| 354 | ret->no_require_user_presence = r != 1; /* NB. flip */ |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 355 | } else if ((r = opt_flag("pty", 1, &opts)) != -1) { |
| 356 | ret->permit_pty_flag = r == 1; |
| 357 | } else if ((r = opt_flag("user-rc", 1, &opts)) != -1) { |
| 358 | ret->permit_user_rc = r == 1; |
| 359 | } else if (opt_match(&opts, "command")) { |
| 360 | if (ret->force_command != NULL) { |
| 361 | errstr = "multiple \"command\" clauses"; |
| 362 | goto fail; |
| 363 | } |
| 364 | ret->force_command = opt_dequote(&opts, &errstr); |
| 365 | if (ret->force_command == NULL) |
| 366 | goto fail; |
| 367 | } else if (opt_match(&opts, "principals")) { |
| 368 | if (ret->cert_principals != NULL) { |
| 369 | errstr = "multiple \"principals\" clauses"; |
| 370 | goto fail; |
| 371 | } |
| 372 | ret->cert_principals = opt_dequote(&opts, &errstr); |
| 373 | if (ret->cert_principals == NULL) |
| 374 | goto fail; |
| 375 | } else if (opt_match(&opts, "from")) { |
| 376 | if (ret->required_from_host_keys != NULL) { |
| 377 | errstr = "multiple \"from\" clauses"; |
| 378 | goto fail; |
| 379 | } |
| 380 | ret->required_from_host_keys = opt_dequote(&opts, |
| 381 | &errstr); |
| 382 | if (ret->required_from_host_keys == NULL) |
| 383 | goto fail; |
djm@openbsd.org | abc0fa3 | 2018-03-14 05:35:40 +0000 | [diff] [blame] | 384 | } else if (opt_match(&opts, "expiry-time")) { |
djm@openbsd.org | bf0fbf2 | 2018-03-12 00:52:01 +0000 | [diff] [blame] | 385 | if ((opt = opt_dequote(&opts, &errstr)) == NULL) |
| 386 | goto fail; |
| 387 | if (parse_absolute_time(opt, &valid_before) != 0 || |
| 388 | valid_before == 0) { |
| 389 | free(opt); |
| 390 | errstr = "invalid expires time"; |
| 391 | goto fail; |
| 392 | } |
| 393 | free(opt); |
| 394 | if (ret->valid_before == 0 || |
| 395 | valid_before < ret->valid_before) |
| 396 | ret->valid_before = valid_before; |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 397 | } else if (opt_match(&opts, "environment")) { |
| 398 | if (ret->nenv > INT_MAX) { |
| 399 | errstr = "too many environment strings"; |
| 400 | goto fail; |
| 401 | } |
| 402 | if ((opt = opt_dequote(&opts, &errstr)) == NULL) |
| 403 | goto fail; |
| 404 | /* env name must be alphanumeric and followed by '=' */ |
| 405 | if ((tmp = strchr(opt, '=')) == NULL) { |
| 406 | free(opt); |
| 407 | errstr = "invalid environment string"; |
| 408 | goto fail; |
| 409 | } |
djm@openbsd.org | 5eff5b8 | 2018-10-03 06:38:35 +0000 | [diff] [blame] | 410 | if ((cp = strdup(opt)) == NULL) |
| 411 | goto alloc_fail; |
| 412 | cp[tmp - opt] = '\0'; /* truncate at '=' */ |
| 413 | if (!valid_env_name(cp)) { |
| 414 | free(cp); |
| 415 | free(opt); |
| 416 | errstr = "invalid environment string"; |
| 417 | goto fail; |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 418 | } |
djm@openbsd.org | 5eff5b8 | 2018-10-03 06:38:35 +0000 | [diff] [blame] | 419 | free(cp); |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 420 | /* Append it. */ |
| 421 | oarray = ret->env; |
| 422 | if ((ret->env = recallocarray(ret->env, ret->nenv, |
| 423 | ret->nenv + 1, sizeof(*ret->env))) == NULL) { |
| 424 | free(opt); |
| 425 | ret->env = oarray; /* put it back for cleanup */ |
| 426 | goto alloc_fail; |
| 427 | } |
| 428 | ret->env[ret->nenv++] = opt; |
| 429 | } else if (opt_match(&opts, "permitopen")) { |
djm@openbsd.org | 87ddd67 | 2018-06-19 02:59:41 +0000 | [diff] [blame] | 430 | if (handle_permit(&opts, 0, &ret->permitopen, |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 431 | &ret->npermitopen, &errstr) != 0) |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 432 | goto fail; |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 433 | } else if (opt_match(&opts, "permitlisten")) { |
djm@openbsd.org | 87ddd67 | 2018-06-19 02:59:41 +0000 | [diff] [blame] | 434 | if (handle_permit(&opts, 1, &ret->permitlisten, |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 435 | &ret->npermitlisten, &errstr) != 0) |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 436 | goto fail; |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 437 | } else if (opt_match(&opts, "tunnel")) { |
| 438 | if ((opt = opt_dequote(&opts, &errstr)) == NULL) |
| 439 | goto fail; |
| 440 | ret->force_tun_device = a2tun(opt, NULL); |
| 441 | free(opt); |
| 442 | if (ret->force_tun_device == SSH_TUNID_ERR) { |
| 443 | errstr = "invalid tun device"; |
| 444 | goto fail; |
| 445 | } |
| 446 | } |
| 447 | /* |
| 448 | * Skip the comma, and move to the next option |
| 449 | * (or break out if there are no more). |
| 450 | */ |
| 451 | if (*opts == '\0' || *opts == ' ' || *opts == '\t') |
| 452 | break; /* End of options. */ |
| 453 | /* Anything other than a comma is an unknown option */ |
| 454 | if (*opts != ',') { |
| 455 | errstr = "unknown key option"; |
| 456 | goto fail; |
| 457 | } |
| 458 | opts++; |
| 459 | if (*opts == '\0') { |
| 460 | errstr = "unexpected end-of-options"; |
| 461 | goto fail; |
| 462 | } |
| 463 | } |
| 464 | |
| 465 | /* success */ |
| 466 | if (errstrp != NULL) |
| 467 | *errstrp = NULL; |
| 468 | return ret; |
| 469 | |
| 470 | alloc_fail: |
| 471 | errstr = "memory allocation failed"; |
| 472 | fail: |
| 473 | sshauthopt_free(ret); |
| 474 | if (errstrp != NULL) |
| 475 | *errstrp = errstr; |
| 476 | return NULL; |
| 477 | } |
| 478 | |
| 479 | struct sshauthopt * |
| 480 | sshauthopt_from_cert(struct sshkey *k) |
| 481 | { |
| 482 | struct sshauthopt *ret; |
| 483 | |
| 484 | if (k == NULL || !sshkey_type_is_cert(k->type) || k->cert == NULL || |
| 485 | k->cert->type != SSH2_CERT_TYPE_USER) |
| 486 | return NULL; |
| 487 | |
| 488 | if ((ret = sshauthopt_new()) == NULL) |
| 489 | return NULL; |
| 490 | |
| 491 | /* Handle options and critical extensions separately */ |
| 492 | if (cert_option_list(ret, k->cert->critical, |
| 493 | OPTIONS_CRITICAL, 1) == -1) { |
| 494 | sshauthopt_free(ret); |
| 495 | return NULL; |
| 496 | } |
| 497 | if (cert_option_list(ret, k->cert->extensions, |
| 498 | OPTIONS_EXTENSIONS, 0) == -1) { |
| 499 | sshauthopt_free(ret); |
| 500 | return NULL; |
| 501 | } |
| 502 | /* success */ |
| 503 | return ret; |
| 504 | } |
| 505 | |
| 506 | /* |
| 507 | * Merges "additional" options to "primary" and returns the result. |
| 508 | * NB. Some options from primary have primacy. |
| 509 | */ |
| 510 | struct sshauthopt * |
| 511 | sshauthopt_merge(const struct sshauthopt *primary, |
| 512 | const struct sshauthopt *additional, const char **errstrp) |
| 513 | { |
| 514 | struct sshauthopt *ret; |
| 515 | const char *errstr = "internal error"; |
| 516 | const char *tmp; |
| 517 | |
| 518 | if (errstrp != NULL) |
| 519 | *errstrp = NULL; |
| 520 | |
| 521 | if ((ret = sshauthopt_new()) == NULL) |
| 522 | goto alloc_fail; |
| 523 | |
| 524 | /* cert_authority and cert_principals are cleared in result */ |
| 525 | |
| 526 | /* Prefer access lists from primary. */ |
| 527 | /* XXX err is both set and mismatch? */ |
| 528 | tmp = primary->required_from_host_cert; |
| 529 | if (tmp == NULL) |
| 530 | tmp = additional->required_from_host_cert; |
| 531 | if (tmp != NULL && (ret->required_from_host_cert = strdup(tmp)) == NULL) |
| 532 | goto alloc_fail; |
| 533 | tmp = primary->required_from_host_keys; |
| 534 | if (tmp == NULL) |
| 535 | tmp = additional->required_from_host_keys; |
| 536 | if (tmp != NULL && (ret->required_from_host_keys = strdup(tmp)) == NULL) |
| 537 | goto alloc_fail; |
| 538 | |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 539 | /* |
| 540 | * force_tun_device, permitopen/permitlisten and environment all |
| 541 | * prefer the primary. |
| 542 | */ |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 543 | ret->force_tun_device = primary->force_tun_device; |
| 544 | if (ret->force_tun_device == -1) |
| 545 | ret->force_tun_device = additional->force_tun_device; |
| 546 | if (primary->nenv > 0) { |
| 547 | if (dup_strings(&ret->env, &ret->nenv, |
| 548 | primary->env, primary->nenv) != 0) |
| 549 | goto alloc_fail; |
| 550 | } else if (additional->nenv) { |
| 551 | if (dup_strings(&ret->env, &ret->nenv, |
| 552 | additional->env, additional->nenv) != 0) |
| 553 | goto alloc_fail; |
| 554 | } |
| 555 | if (primary->npermitopen > 0) { |
| 556 | if (dup_strings(&ret->permitopen, &ret->npermitopen, |
| 557 | primary->permitopen, primary->npermitopen) != 0) |
| 558 | goto alloc_fail; |
| 559 | } else if (additional->npermitopen > 0) { |
| 560 | if (dup_strings(&ret->permitopen, &ret->npermitopen, |
| 561 | additional->permitopen, additional->npermitopen) != 0) |
| 562 | goto alloc_fail; |
| 563 | } |
| 564 | |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 565 | if (primary->npermitlisten > 0) { |
| 566 | if (dup_strings(&ret->permitlisten, &ret->npermitlisten, |
| 567 | primary->permitlisten, primary->npermitlisten) != 0) |
| 568 | goto alloc_fail; |
| 569 | } else if (additional->npermitlisten > 0) { |
| 570 | if (dup_strings(&ret->permitlisten, &ret->npermitlisten, |
| 571 | additional->permitlisten, additional->npermitlisten) != 0) |
| 572 | goto alloc_fail; |
| 573 | } |
| 574 | |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 575 | #define OPTFLAG_AND(x) ret->x = (primary->x == 1) && (additional->x == 1) |
| 576 | /* Permissive flags are logical-AND (i.e. must be set in both) */ |
| 577 | OPTFLAG_AND(permit_port_forwarding_flag); |
| 578 | OPTFLAG_AND(permit_agent_forwarding_flag); |
| 579 | OPTFLAG_AND(permit_x11_forwarding_flag); |
| 580 | OPTFLAG_AND(permit_pty_flag); |
| 581 | OPTFLAG_AND(permit_user_rc); |
| 582 | OPTFLAG_AND(no_require_user_presence); |
| 583 | #undef OPTFLAG_AND |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 584 | |
djm@openbsd.org | bf0fbf2 | 2018-03-12 00:52:01 +0000 | [diff] [blame] | 585 | /* Earliest expiry time should win */ |
| 586 | if (primary->valid_before != 0) |
| 587 | ret->valid_before = primary->valid_before; |
| 588 | if (additional->valid_before != 0 && |
| 589 | additional->valid_before < ret->valid_before) |
| 590 | ret->valid_before = additional->valid_before; |
| 591 | |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 592 | /* |
| 593 | * When both multiple forced-command are specified, only |
| 594 | * proceed if they are identical, otherwise fail. |
| 595 | */ |
| 596 | if (primary->force_command != NULL && |
| 597 | additional->force_command != NULL) { |
| 598 | if (strcmp(primary->force_command, |
| 599 | additional->force_command) == 0) { |
| 600 | /* ok */ |
| 601 | ret->force_command = strdup(primary->force_command); |
| 602 | if (ret->force_command == NULL) |
| 603 | goto alloc_fail; |
| 604 | } else { |
| 605 | errstr = "forced command options do not match"; |
| 606 | goto fail; |
| 607 | } |
| 608 | } else if (primary->force_command != NULL) { |
| 609 | if ((ret->force_command = strdup( |
| 610 | primary->force_command)) == NULL) |
| 611 | goto alloc_fail; |
| 612 | } else if (additional->force_command != NULL) { |
| 613 | if ((ret->force_command = strdup( |
| 614 | additional->force_command)) == NULL) |
| 615 | goto alloc_fail; |
| 616 | } |
| 617 | /* success */ |
| 618 | if (errstrp != NULL) |
| 619 | *errstrp = NULL; |
| 620 | return ret; |
| 621 | |
| 622 | alloc_fail: |
| 623 | errstr = "memory allocation failed"; |
| 624 | fail: |
| 625 | if (errstrp != NULL) |
| 626 | *errstrp = errstr; |
| 627 | sshauthopt_free(ret); |
| 628 | return NULL; |
| 629 | } |
| 630 | |
| 631 | /* |
| 632 | * Copy options |
| 633 | */ |
| 634 | struct sshauthopt * |
| 635 | sshauthopt_copy(const struct sshauthopt *orig) |
| 636 | { |
| 637 | struct sshauthopt *ret; |
| 638 | |
| 639 | if ((ret = sshauthopt_new()) == NULL) |
| 640 | return NULL; |
| 641 | |
| 642 | #define OPTSCALAR(x) ret->x = orig->x |
| 643 | OPTSCALAR(permit_port_forwarding_flag); |
| 644 | OPTSCALAR(permit_agent_forwarding_flag); |
| 645 | OPTSCALAR(permit_x11_forwarding_flag); |
| 646 | OPTSCALAR(permit_pty_flag); |
| 647 | OPTSCALAR(permit_user_rc); |
| 648 | OPTSCALAR(restricted); |
| 649 | OPTSCALAR(cert_authority); |
| 650 | OPTSCALAR(force_tun_device); |
djm@openbsd.org | bf0fbf2 | 2018-03-12 00:52:01 +0000 | [diff] [blame] | 651 | OPTSCALAR(valid_before); |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 652 | OPTSCALAR(no_require_user_presence); |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 653 | #undef OPTSCALAR |
| 654 | #define OPTSTRING(x) \ |
| 655 | do { \ |
| 656 | if (orig->x != NULL && (ret->x = strdup(orig->x)) == NULL) { \ |
| 657 | sshauthopt_free(ret); \ |
| 658 | return NULL; \ |
| 659 | } \ |
| 660 | } while (0) |
| 661 | OPTSTRING(cert_principals); |
| 662 | OPTSTRING(force_command); |
| 663 | OPTSTRING(required_from_host_cert); |
| 664 | OPTSTRING(required_from_host_keys); |
| 665 | #undef OPTSTRING |
| 666 | |
| 667 | if (dup_strings(&ret->env, &ret->nenv, orig->env, orig->nenv) != 0 || |
| 668 | dup_strings(&ret->permitopen, &ret->npermitopen, |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 669 | orig->permitopen, orig->npermitopen) != 0 || |
| 670 | dup_strings(&ret->permitlisten, &ret->npermitlisten, |
| 671 | orig->permitlisten, orig->npermitlisten) != 0) { |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 672 | sshauthopt_free(ret); |
| 673 | return NULL; |
| 674 | } |
| 675 | return ret; |
| 676 | } |
| 677 | |
| 678 | static int |
| 679 | serialise_array(struct sshbuf *m, char **a, size_t n) |
| 680 | { |
| 681 | struct sshbuf *b; |
| 682 | size_t i; |
| 683 | int r; |
| 684 | |
| 685 | if (n > INT_MAX) |
| 686 | return SSH_ERR_INTERNAL_ERROR; |
| 687 | |
| 688 | if ((b = sshbuf_new()) == NULL) { |
| 689 | return SSH_ERR_ALLOC_FAIL; |
| 690 | } |
| 691 | for (i = 0; i < n; i++) { |
| 692 | if ((r = sshbuf_put_cstring(b, a[i])) != 0) { |
| 693 | sshbuf_free(b); |
| 694 | return r; |
| 695 | } |
| 696 | } |
| 697 | if ((r = sshbuf_put_u32(m, n)) != 0 || |
| 698 | (r = sshbuf_put_stringb(m, b)) != 0) { |
| 699 | sshbuf_free(b); |
| 700 | return r; |
| 701 | } |
| 702 | /* success */ |
| 703 | return 0; |
| 704 | } |
| 705 | |
| 706 | static int |
| 707 | deserialise_array(struct sshbuf *m, char ***ap, size_t *np) |
| 708 | { |
| 709 | char **a = NULL; |
| 710 | size_t i, n = 0; |
| 711 | struct sshbuf *b = NULL; |
| 712 | u_int tmp; |
| 713 | int r = SSH_ERR_INTERNAL_ERROR; |
| 714 | |
| 715 | if ((r = sshbuf_get_u32(m, &tmp)) != 0 || |
| 716 | (r = sshbuf_froms(m, &b)) != 0) |
| 717 | goto out; |
| 718 | if (tmp > INT_MAX) { |
| 719 | r = SSH_ERR_INVALID_FORMAT; |
| 720 | goto out; |
| 721 | } |
| 722 | n = tmp; |
| 723 | if (n > 0 && (a = calloc(n, sizeof(*a))) == NULL) { |
| 724 | r = SSH_ERR_ALLOC_FAIL; |
| 725 | goto out; |
| 726 | } |
| 727 | for (i = 0; i < n; i++) { |
| 728 | if ((r = sshbuf_get_cstring(b, &a[i], NULL)) != 0) |
| 729 | goto out; |
| 730 | } |
| 731 | /* success */ |
| 732 | r = 0; |
| 733 | *ap = a; |
| 734 | a = NULL; |
| 735 | *np = n; |
| 736 | n = 0; |
| 737 | out: |
| 738 | for (i = 0; i < n; i++) |
| 739 | free(a[i]); |
| 740 | free(a); |
| 741 | sshbuf_free(b); |
| 742 | return r; |
| 743 | } |
| 744 | |
| 745 | static int |
| 746 | serialise_nullable_string(struct sshbuf *m, const char *s) |
| 747 | { |
| 748 | int r; |
| 749 | |
| 750 | if ((r = sshbuf_put_u8(m, s == NULL)) != 0 || |
| 751 | (r = sshbuf_put_cstring(m, s)) != 0) |
| 752 | return r; |
| 753 | return 0; |
| 754 | } |
| 755 | |
| 756 | static int |
| 757 | deserialise_nullable_string(struct sshbuf *m, char **sp) |
| 758 | { |
| 759 | int r; |
| 760 | u_char flag; |
| 761 | |
| 762 | *sp = NULL; |
| 763 | if ((r = sshbuf_get_u8(m, &flag)) != 0 || |
| 764 | (r = sshbuf_get_cstring(m, flag ? NULL : sp, NULL)) != 0) |
| 765 | return r; |
| 766 | return 0; |
| 767 | } |
| 768 | |
| 769 | int |
| 770 | sshauthopt_serialise(const struct sshauthopt *opts, struct sshbuf *m, |
| 771 | int untrusted) |
| 772 | { |
| 773 | int r = SSH_ERR_INTERNAL_ERROR; |
| 774 | |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 775 | /* Flag options */ |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 776 | if ((r = sshbuf_put_u8(m, opts->permit_port_forwarding_flag)) != 0 || |
| 777 | (r = sshbuf_put_u8(m, opts->permit_agent_forwarding_flag)) != 0 || |
| 778 | (r = sshbuf_put_u8(m, opts->permit_x11_forwarding_flag)) != 0 || |
| 779 | (r = sshbuf_put_u8(m, opts->permit_pty_flag)) != 0 || |
| 780 | (r = sshbuf_put_u8(m, opts->permit_user_rc)) != 0 || |
| 781 | (r = sshbuf_put_u8(m, opts->restricted)) != 0 || |
djm@openbsd.org | bf0fbf2 | 2018-03-12 00:52:01 +0000 | [diff] [blame] | 782 | (r = sshbuf_put_u8(m, opts->cert_authority)) != 0 || |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 783 | (r = sshbuf_put_u8(m, opts->no_require_user_presence)) != 0) |
| 784 | return r; |
| 785 | |
| 786 | /* Simple integer options */ |
| 787 | if ((r = sshbuf_put_u64(m, opts->valid_before)) != 0) |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 788 | return r; |
| 789 | |
| 790 | /* tunnel number can be negative to indicate "unset" */ |
| 791 | if ((r = sshbuf_put_u8(m, opts->force_tun_device == -1)) != 0 || |
| 792 | (r = sshbuf_put_u32(m, (opts->force_tun_device < 0) ? |
| 793 | 0 : (u_int)opts->force_tun_device)) != 0) |
| 794 | return r; |
| 795 | |
| 796 | /* String options; these may be NULL */ |
| 797 | if ((r = serialise_nullable_string(m, |
| 798 | untrusted ? "yes" : opts->cert_principals)) != 0 || |
| 799 | (r = serialise_nullable_string(m, |
| 800 | untrusted ? "true" : opts->force_command)) != 0 || |
| 801 | (r = serialise_nullable_string(m, |
| 802 | untrusted ? NULL : opts->required_from_host_cert)) != 0 || |
| 803 | (r = serialise_nullable_string(m, |
| 804 | untrusted ? NULL : opts->required_from_host_keys)) != 0) |
| 805 | return r; |
| 806 | |
| 807 | /* Array options */ |
| 808 | if ((r = serialise_array(m, opts->env, |
| 809 | untrusted ? 0 : opts->nenv)) != 0 || |
| 810 | (r = serialise_array(m, opts->permitopen, |
djm@openbsd.org | 6d41815 | 2018-06-07 09:26:42 +0000 | [diff] [blame] | 811 | untrusted ? 0 : opts->npermitopen)) != 0 || |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 812 | (r = serialise_array(m, opts->permitlisten, |
| 813 | untrusted ? 0 : opts->npermitlisten)) != 0) |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 814 | return r; |
| 815 | |
| 816 | /* success */ |
| 817 | return 0; |
| 818 | } |
| 819 | |
| 820 | int |
| 821 | sshauthopt_deserialise(struct sshbuf *m, struct sshauthopt **optsp) |
| 822 | { |
| 823 | struct sshauthopt *opts = NULL; |
| 824 | int r = SSH_ERR_INTERNAL_ERROR; |
| 825 | u_char f; |
| 826 | u_int tmp; |
| 827 | |
| 828 | if ((opts = calloc(1, sizeof(*opts))) == NULL) |
| 829 | return SSH_ERR_ALLOC_FAIL; |
| 830 | |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 831 | /* Flag options */ |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 832 | #define OPT_FLAG(x) \ |
| 833 | do { \ |
| 834 | if ((r = sshbuf_get_u8(m, &f)) != 0) \ |
| 835 | goto out; \ |
| 836 | opts->x = f; \ |
| 837 | } while (0) |
| 838 | OPT_FLAG(permit_port_forwarding_flag); |
| 839 | OPT_FLAG(permit_agent_forwarding_flag); |
| 840 | OPT_FLAG(permit_x11_forwarding_flag); |
| 841 | OPT_FLAG(permit_pty_flag); |
| 842 | OPT_FLAG(permit_user_rc); |
| 843 | OPT_FLAG(restricted); |
| 844 | OPT_FLAG(cert_authority); |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 845 | OPT_FLAG(no_require_user_presence); |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 846 | #undef OPT_FLAG |
| 847 | |
djm@openbsd.org | 2e71263 | 2019-11-25 00:54:23 +0000 | [diff] [blame] | 848 | /* Simple integer options */ |
djm@openbsd.org | bf0fbf2 | 2018-03-12 00:52:01 +0000 | [diff] [blame] | 849 | if ((r = sshbuf_get_u64(m, &opts->valid_before)) != 0) |
| 850 | goto out; |
| 851 | |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 852 | /* tunnel number can be negative to indicate "unset" */ |
| 853 | if ((r = sshbuf_get_u8(m, &f)) != 0 || |
| 854 | (r = sshbuf_get_u32(m, &tmp)) != 0) |
| 855 | goto out; |
| 856 | opts->force_tun_device = f ? -1 : (int)tmp; |
| 857 | |
| 858 | /* String options may be NULL */ |
| 859 | if ((r = deserialise_nullable_string(m, &opts->cert_principals)) != 0 || |
| 860 | (r = deserialise_nullable_string(m, &opts->force_command)) != 0 || |
| 861 | (r = deserialise_nullable_string(m, |
| 862 | &opts->required_from_host_cert)) != 0 || |
| 863 | (r = deserialise_nullable_string(m, |
| 864 | &opts->required_from_host_keys)) != 0) |
| 865 | goto out; |
| 866 | |
| 867 | /* Array options */ |
| 868 | if ((r = deserialise_array(m, &opts->env, &opts->nenv)) != 0 || |
| 869 | (r = deserialise_array(m, |
djm@openbsd.org | 93c06ab | 2018-06-06 18:23:32 +0000 | [diff] [blame] | 870 | &opts->permitopen, &opts->npermitopen)) != 0 || |
| 871 | (r = deserialise_array(m, |
| 872 | &opts->permitlisten, &opts->npermitlisten)) != 0) |
djm@openbsd.org | 90c4bec | 2018-03-03 03:06:02 +0000 | [diff] [blame] | 873 | goto out; |
| 874 | |
| 875 | /* success */ |
| 876 | r = 0; |
| 877 | *optsp = opts; |
| 878 | opts = NULL; |
| 879 | out: |
| 880 | sshauthopt_free(opts); |
| 881 | return r; |
| 882 | } |