djm@openbsd.org | 56912de | 2017-04-30 23:10:43 +0000 | [diff] [blame] | 1 | /* $OpenBSD: authfile.c,v 1.124 2017/04/30 23:10:43 djm Exp $ */ |
Damien Miller | d4a8b7e | 1999-10-27 13:42:43 +1000 | [diff] [blame] | 2 | /* |
Damien Miller | bcd00ab | 2013-12-07 10:41:55 +1100 | [diff] [blame] | 3 | * Copyright (c) 2000, 2013 Markus Friedl. All rights reserved. |
Damien Miller | e4340be | 2000-09-16 13:29:08 +1100 | [diff] [blame] | 4 | * |
| 5 | * Redistribution and use in source and binary forms, with or without |
| 6 | * modification, are permitted provided that the following conditions |
| 7 | * are met: |
| 8 | * 1. Redistributions of source code must retain the above copyright |
| 9 | * notice, this list of conditions and the following disclaimer. |
| 10 | * 2. Redistributions in binary form must reproduce the above copyright |
| 11 | * notice, this list of conditions and the following disclaimer in the |
| 12 | * documentation and/or other materials provided with the distribution. |
| 13 | * |
| 14 | * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR |
| 15 | * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES |
| 16 | * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. |
| 17 | * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, |
| 18 | * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT |
| 19 | * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, |
| 20 | * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY |
| 21 | * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
| 22 | * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF |
| 23 | * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
Damien Miller | 95def09 | 1999-11-25 00:26:21 +1100 | [diff] [blame] | 24 | */ |
Damien Miller | d4a8b7e | 1999-10-27 13:42:43 +1000 | [diff] [blame] | 25 | |
| 26 | #include "includes.h" |
Damien Miller | f17883e | 2006-03-15 11:45:54 +1100 | [diff] [blame] | 27 | |
| 28 | #include <sys/types.h> |
| 29 | #include <sys/stat.h> |
Damien Miller | d783435 | 2006-08-05 12:39:39 +1000 | [diff] [blame] | 30 | #include <sys/uio.h> |
Damien Miller | d4a8b7e | 1999-10-27 13:42:43 +1000 | [diff] [blame] | 31 | |
Darren Tucker | ba72405 | 2006-07-12 22:24:22 +1000 | [diff] [blame] | 32 | #include <errno.h> |
Damien Miller | 57cf638 | 2006-07-10 21:13:46 +1000 | [diff] [blame] | 33 | #include <fcntl.h> |
Damien Miller | a7a73ee | 2006-08-05 11:37:59 +1000 | [diff] [blame] | 34 | #include <stdio.h> |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 35 | #include <stdarg.h> |
Damien Miller | e7a1e5c | 2006-08-05 11:34:19 +1000 | [diff] [blame] | 36 | #include <stdlib.h> |
Damien Miller | e3476ed | 2006-07-24 14:13:33 +1000 | [diff] [blame] | 37 | #include <string.h> |
Damien Miller | e6b3b61 | 2006-07-24 14:01:23 +1000 | [diff] [blame] | 38 | #include <unistd.h> |
deraadt@openbsd.org | 087266e | 2015-01-20 23:14:00 +0000 | [diff] [blame] | 39 | #include <limits.h> |
Damien Miller | 57cf638 | 2006-07-10 21:13:46 +1000 | [diff] [blame] | 40 | |
Damien Miller | d783435 | 2006-08-05 12:39:39 +1000 | [diff] [blame] | 41 | #include "cipher.h" |
Ben Lindstrom | 226cfa0 | 2001-01-22 05:34:40 +0000 | [diff] [blame] | 42 | #include "ssh.h" |
| 43 | #include "log.h" |
Ben Lindstrom | 31ca54a | 2001-02-09 02:11:24 +0000 | [diff] [blame] | 44 | #include "authfile.h" |
Damien Miller | 040b64f | 2002-01-22 23:10:04 +1100 | [diff] [blame] | 45 | #include "rsa.h" |
Darren Tucker | f0f9098 | 2004-12-11 13:39:50 +1100 | [diff] [blame] | 46 | #include "misc.h" |
Damien Miller | eccb9de | 2005-06-17 12:59:34 +1000 | [diff] [blame] | 47 | #include "atomicio.h" |
markus@openbsd.org | 16db0a7 | 2015-07-09 09:49:46 +0000 | [diff] [blame] | 48 | #include "sshkey.h" |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 49 | #include "sshbuf.h" |
| 50 | #include "ssherr.h" |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 51 | #include "krl.h" |
Damien Miller | d4a8b7e | 1999-10-27 13:42:43 +1000 | [diff] [blame] | 52 | |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 53 | #define MAX_KEY_FILE_SIZE (1024 * 1024) |
| 54 | |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 55 | /* Save a key blob to a file */ |
| 56 | static int |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 57 | sshkey_save_private_blob(struct sshbuf *keybuf, const char *filename) |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 58 | { |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 59 | int fd, oerrno; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 60 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 61 | if ((fd = open(filename, O_WRONLY | O_CREAT | O_TRUNC, 0600)) < 0) |
| 62 | return SSH_ERR_SYSTEM_ERROR; |
| 63 | if (atomicio(vwrite, fd, (u_char *)sshbuf_ptr(keybuf), |
| 64 | sshbuf_len(keybuf)) != sshbuf_len(keybuf)) { |
| 65 | oerrno = errno; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 66 | close(fd); |
| 67 | unlink(filename); |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 68 | errno = oerrno; |
| 69 | return SSH_ERR_SYSTEM_ERROR; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 70 | } |
| 71 | close(fd); |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 72 | return 0; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 73 | } |
| 74 | |
Damien Miller | eba71ba | 2000-04-29 23:57:08 +1000 | [diff] [blame] | 75 | int |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 76 | sshkey_save_private(struct sshkey *key, const char *filename, |
| 77 | const char *passphrase, const char *comment, |
| 78 | int force_new_format, const char *new_format_cipher, int new_format_rounds) |
Damien Miller | eba71ba | 2000-04-29 23:57:08 +1000 | [diff] [blame] | 79 | { |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 80 | struct sshbuf *keyblob = NULL; |
| 81 | int r; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 82 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 83 | if ((keyblob = sshbuf_new()) == NULL) |
| 84 | return SSH_ERR_ALLOC_FAIL; |
| 85 | if ((r = sshkey_private_to_fileblob(key, keyblob, passphrase, comment, |
| 86 | force_new_format, new_format_cipher, new_format_rounds)) != 0) |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 87 | goto out; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 88 | if ((r = sshkey_save_private_blob(keyblob, filename)) != 0) |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 89 | goto out; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 90 | r = 0; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 91 | out: |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 92 | sshbuf_free(keyblob); |
| 93 | return r; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 94 | } |
| 95 | |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 96 | /* Load a key from a fd into a buffer */ |
| 97 | int |
djm@openbsd.org | 1195f4c | 2015-01-08 10:14:08 +0000 | [diff] [blame] | 98 | sshkey_load_file(int fd, struct sshbuf *blob) |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 99 | { |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 100 | u_char buf[1024]; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 101 | size_t len; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 102 | struct stat st; |
djm@openbsd.org | 54d0220 | 2016-11-25 23:24:45 +0000 | [diff] [blame] | 103 | int r, dontmax = 0; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 104 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 105 | if (fstat(fd, &st) < 0) |
| 106 | return SSH_ERR_SYSTEM_ERROR; |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 107 | if ((st.st_mode & (S_IFSOCK|S_IFCHR|S_IFIFO)) == 0 && |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 108 | st.st_size > MAX_KEY_FILE_SIZE) |
| 109 | return SSH_ERR_INVALID_FORMAT; |
djm@openbsd.org | 54d0220 | 2016-11-25 23:24:45 +0000 | [diff] [blame] | 110 | /* |
| 111 | * Pre-allocate the buffer used for the key contents and clamp its |
| 112 | * maximum size. This ensures that key contents are never leaked via |
| 113 | * implicit realloc() in the sshbuf code. |
| 114 | */ |
| 115 | if ((st.st_mode & S_IFREG) == 0 || st.st_size <= 0) { |
deraadt@openbsd.org | ef47843 | 2017-03-26 00:18:52 +0000 | [diff] [blame] | 116 | st.st_size = 64*1024; /* 64k ought to be enough for anybody. :) */ |
djm@openbsd.org | 54d0220 | 2016-11-25 23:24:45 +0000 | [diff] [blame] | 117 | dontmax = 1; |
| 118 | } |
| 119 | if ((r = sshbuf_allocate(blob, st.st_size)) != 0 || |
| 120 | (dontmax && (r = sshbuf_set_max_size(blob, st.st_size)) != 0)) |
| 121 | return r; |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 122 | for (;;) { |
| 123 | if ((len = atomicio(read, fd, buf, sizeof(buf))) == 0) { |
| 124 | if (errno == EPIPE) |
| 125 | break; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 126 | r = SSH_ERR_SYSTEM_ERROR; |
| 127 | goto out; |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 128 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 129 | if ((r = sshbuf_put(blob, buf, len)) != 0) |
| 130 | goto out; |
| 131 | if (sshbuf_len(blob) > MAX_KEY_FILE_SIZE) { |
| 132 | r = SSH_ERR_INVALID_FORMAT; |
| 133 | goto out; |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 134 | } |
| 135 | } |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 136 | if ((st.st_mode & (S_IFSOCK|S_IFCHR|S_IFIFO)) == 0 && |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 137 | st.st_size != (off_t)sshbuf_len(blob)) { |
| 138 | r = SSH_ERR_FILE_CHANGED; |
| 139 | goto out; |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 140 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 141 | r = 0; |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 142 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 143 | out: |
| 144 | explicit_bzero(buf, sizeof(buf)); |
| 145 | if (r != 0) |
| 146 | sshbuf_reset(blob); |
| 147 | return r; |
Damien Miller | eba71ba | 2000-04-29 23:57:08 +1000 | [diff] [blame] | 148 | } |
| 149 | |
Damien Miller | eba71ba | 2000-04-29 23:57:08 +1000 | [diff] [blame] | 150 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 151 | /* XXX remove error() calls from here? */ |
| 152 | int |
| 153 | sshkey_perm_ok(int fd, const char *filename) |
Damien Miller | eba71ba | 2000-04-29 23:57:08 +1000 | [diff] [blame] | 154 | { |
Damien Miller | eba71ba | 2000-04-29 23:57:08 +1000 | [diff] [blame] | 155 | struct stat st; |
| 156 | |
Ben Lindstrom | 7aff261 | 2001-09-23 13:53:22 +0000 | [diff] [blame] | 157 | if (fstat(fd, &st) < 0) |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 158 | return SSH_ERR_SYSTEM_ERROR; |
Ben Lindstrom | 7aff261 | 2001-09-23 13:53:22 +0000 | [diff] [blame] | 159 | /* |
| 160 | * if a key owned by the user is accessed, then we check the |
| 161 | * permissions of the file. if the key owned by a different user, |
| 162 | * then we don't care. |
| 163 | */ |
Damien Miller | b70b61f | 2000-09-16 16:25:12 +1100 | [diff] [blame] | 164 | #ifdef HAVE_CYGWIN |
Damien Miller | cb5e44a | 2000-09-29 12:12:36 +1100 | [diff] [blame] | 165 | if (check_ntsec(filename)) |
Damien Miller | b70b61f | 2000-09-16 16:25:12 +1100 | [diff] [blame] | 166 | #endif |
Ben Lindstrom | 7aff261 | 2001-09-23 13:53:22 +0000 | [diff] [blame] | 167 | if ((st.st_uid == getuid()) && (st.st_mode & 077) != 0) { |
Damien Miller | eba71ba | 2000-04-29 23:57:08 +1000 | [diff] [blame] | 168 | error("@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"); |
| 169 | error("@ WARNING: UNPROTECTED PRIVATE KEY FILE! @"); |
| 170 | error("@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"); |
Ben Lindstrom | 7aff261 | 2001-09-23 13:53:22 +0000 | [diff] [blame] | 171 | error("Permissions 0%3.3o for '%s' are too open.", |
Damien Miller | 04bd8b0 | 2003-05-25 14:38:33 +1000 | [diff] [blame] | 172 | (u_int)st.st_mode & 0777, filename); |
djm@openbsd.org | 6b0d576 | 2015-04-17 13:32:09 +0000 | [diff] [blame] | 173 | error("It is required that your private key files are NOT accessible by others."); |
Ben Lindstrom | d0fca42 | 2001-03-26 13:44:06 +0000 | [diff] [blame] | 174 | error("This private key will be ignored."); |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 175 | return SSH_ERR_KEY_BAD_PERMISSIONS; |
Damien Miller | e4340be | 2000-09-16 13:29:08 +1100 | [diff] [blame] | 176 | } |
| 177 | return 0; |
| 178 | } |
| 179 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 180 | /* XXX kill perm_ok now that we have SSH_ERR_KEY_BAD_PERMISSIONS? */ |
| 181 | int |
| 182 | sshkey_load_private_type(int type, const char *filename, const char *passphrase, |
| 183 | struct sshkey **keyp, char **commentp, int *perm_ok) |
Damien Miller | e4340be | 2000-09-16 13:29:08 +1100 | [diff] [blame] | 184 | { |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 185 | int fd, r; |
Damien Miller | e4340be | 2000-09-16 13:29:08 +1100 | [diff] [blame] | 186 | |
djm@openbsd.org | dce19bf | 2016-04-09 12:39:30 +0000 | [diff] [blame] | 187 | if (keyp != NULL) |
| 188 | *keyp = NULL; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 189 | if (commentp != NULL) |
| 190 | *commentp = NULL; |
| 191 | |
| 192 | if ((fd = open(filename, O_RDONLY)) < 0) { |
| 193 | if (perm_ok != NULL) |
| 194 | *perm_ok = 0; |
| 195 | return SSH_ERR_SYSTEM_ERROR; |
| 196 | } |
| 197 | if (sshkey_perm_ok(fd, filename) != 0) { |
| 198 | if (perm_ok != NULL) |
| 199 | *perm_ok = 0; |
| 200 | r = SSH_ERR_KEY_BAD_PERMISSIONS; |
| 201 | goto out; |
| 202 | } |
| 203 | if (perm_ok != NULL) |
| 204 | *perm_ok = 1; |
| 205 | |
djm@openbsd.org | 1195f4c | 2015-01-08 10:14:08 +0000 | [diff] [blame] | 206 | r = sshkey_load_private_type_fd(fd, type, passphrase, keyp, commentp); |
| 207 | out: |
| 208 | close(fd); |
| 209 | return r; |
| 210 | } |
| 211 | |
| 212 | int |
| 213 | sshkey_load_private_type_fd(int fd, int type, const char *passphrase, |
| 214 | struct sshkey **keyp, char **commentp) |
| 215 | { |
| 216 | struct sshbuf *buffer = NULL; |
| 217 | int r; |
| 218 | |
djm@openbsd.org | dce19bf | 2016-04-09 12:39:30 +0000 | [diff] [blame] | 219 | if (keyp != NULL) |
| 220 | *keyp = NULL; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 221 | if ((buffer = sshbuf_new()) == NULL) { |
| 222 | r = SSH_ERR_ALLOC_FAIL; |
| 223 | goto out; |
| 224 | } |
djm@openbsd.org | 1195f4c | 2015-01-08 10:14:08 +0000 | [diff] [blame] | 225 | if ((r = sshkey_load_file(fd, buffer)) != 0 || |
| 226 | (r = sshkey_parse_private_fileblob_type(buffer, type, |
| 227 | passphrase, keyp, commentp)) != 0) |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 228 | goto out; |
djm@openbsd.org | 1195f4c | 2015-01-08 10:14:08 +0000 | [diff] [blame] | 229 | |
| 230 | /* success */ |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 231 | r = 0; |
| 232 | out: |
mmcc@openbsd.org | 52d7078 | 2015-12-11 04:21:11 +0000 | [diff] [blame] | 233 | sshbuf_free(buffer); |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 234 | return r; |
| 235 | } |
| 236 | |
| 237 | /* XXX this is almost identical to sshkey_load_private_type() */ |
| 238 | int |
| 239 | sshkey_load_private(const char *filename, const char *passphrase, |
| 240 | struct sshkey **keyp, char **commentp) |
| 241 | { |
| 242 | struct sshbuf *buffer = NULL; |
| 243 | int r, fd; |
| 244 | |
djm@openbsd.org | dce19bf | 2016-04-09 12:39:30 +0000 | [diff] [blame] | 245 | if (keyp != NULL) |
| 246 | *keyp = NULL; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 247 | if (commentp != NULL) |
| 248 | *commentp = NULL; |
| 249 | |
| 250 | if ((fd = open(filename, O_RDONLY)) < 0) |
| 251 | return SSH_ERR_SYSTEM_ERROR; |
| 252 | if (sshkey_perm_ok(fd, filename) != 0) { |
| 253 | r = SSH_ERR_KEY_BAD_PERMISSIONS; |
| 254 | goto out; |
| 255 | } |
| 256 | |
| 257 | if ((buffer = sshbuf_new()) == NULL) { |
| 258 | r = SSH_ERR_ALLOC_FAIL; |
| 259 | goto out; |
| 260 | } |
djm@openbsd.org | 1195f4c | 2015-01-08 10:14:08 +0000 | [diff] [blame] | 261 | if ((r = sshkey_load_file(fd, buffer)) != 0 || |
tim@openbsd.org | 3c019a9 | 2015-09-13 14:39:16 +0000 | [diff] [blame] | 262 | (r = sshkey_parse_private_fileblob(buffer, passphrase, keyp, |
| 263 | commentp)) != 0) |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 264 | goto out; |
| 265 | r = 0; |
| 266 | out: |
| 267 | close(fd); |
mmcc@openbsd.org | 52d7078 | 2015-12-11 04:21:11 +0000 | [diff] [blame] | 268 | sshbuf_free(buffer); |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 269 | return r; |
| 270 | } |
| 271 | |
| 272 | static int |
| 273 | sshkey_try_load_public(struct sshkey *k, const char *filename, char **commentp) |
| 274 | { |
| 275 | FILE *f; |
| 276 | char line[SSH_MAX_PUBKEY_BYTES]; |
| 277 | char *cp; |
| 278 | u_long linenum = 0; |
| 279 | int r; |
| 280 | |
| 281 | if (commentp != NULL) |
| 282 | *commentp = NULL; |
| 283 | if ((f = fopen(filename, "r")) == NULL) |
| 284 | return SSH_ERR_SYSTEM_ERROR; |
| 285 | while (read_keyfile_line(f, filename, line, sizeof(line), |
| 286 | &linenum) != -1) { |
| 287 | cp = line; |
| 288 | switch (*cp) { |
| 289 | case '#': |
| 290 | case '\n': |
| 291 | case '\0': |
| 292 | continue; |
| 293 | } |
| 294 | /* Abort loading if this looks like a private key */ |
| 295 | if (strncmp(cp, "-----BEGIN", 10) == 0 || |
| 296 | strcmp(cp, "SSH PRIVATE KEY FILE") == 0) |
| 297 | break; |
| 298 | /* Skip leading whitespace. */ |
| 299 | for (; *cp && (*cp == ' ' || *cp == '\t'); cp++) |
| 300 | ; |
| 301 | if (*cp) { |
| 302 | if ((r = sshkey_read(k, &cp)) == 0) { |
| 303 | cp[strcspn(cp, "\r\n")] = '\0'; |
| 304 | if (commentp) { |
| 305 | *commentp = strdup(*cp ? |
| 306 | cp : filename); |
| 307 | if (*commentp == NULL) |
| 308 | r = SSH_ERR_ALLOC_FAIL; |
| 309 | } |
| 310 | fclose(f); |
| 311 | return r; |
| 312 | } |
| 313 | } |
| 314 | } |
| 315 | fclose(f); |
| 316 | return SSH_ERR_INVALID_FORMAT; |
| 317 | } |
| 318 | |
| 319 | /* load public key from ssh v1 private or any pubkey file */ |
| 320 | int |
| 321 | sshkey_load_public(const char *filename, struct sshkey **keyp, char **commentp) |
| 322 | { |
| 323 | struct sshkey *pub = NULL; |
deraadt@openbsd.org | 087266e | 2015-01-20 23:14:00 +0000 | [diff] [blame] | 324 | char file[PATH_MAX]; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 325 | int r, fd; |
| 326 | |
| 327 | if (keyp != NULL) |
| 328 | *keyp = NULL; |
| 329 | if (commentp != NULL) |
| 330 | *commentp = NULL; |
| 331 | |
djm@openbsd.org | 5248429 | 2015-02-23 16:55:51 +0000 | [diff] [blame] | 332 | /* XXX should load file once and attempt to parse each format */ |
| 333 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 334 | if ((fd = open(filename, O_RDONLY)) < 0) |
| 335 | goto skip; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 336 | close(fd); |
Damien Miller | db27472 | 2003-05-14 13:45:22 +1000 | [diff] [blame] | 337 | |
| 338 | /* try ssh2 public key */ |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 339 | if ((pub = sshkey_new(KEY_UNSPEC)) == NULL) |
| 340 | return SSH_ERR_ALLOC_FAIL; |
| 341 | if ((r = sshkey_try_load_public(pub, filename, commentp)) == 0) { |
| 342 | if (keyp != NULL) |
| 343 | *keyp = pub; |
| 344 | return 0; |
| 345 | } |
| 346 | sshkey_free(pub); |
| 347 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 348 | |
| 349 | skip: |
| 350 | /* try .pub suffix */ |
| 351 | if ((pub = sshkey_new(KEY_UNSPEC)) == NULL) |
| 352 | return SSH_ERR_ALLOC_FAIL; |
| 353 | r = SSH_ERR_ALLOC_FAIL; /* in case strlcpy or strlcat fail */ |
Ben Lindstrom | d0fca42 | 2001-03-26 13:44:06 +0000 | [diff] [blame] | 354 | if ((strlcpy(file, filename, sizeof file) < sizeof(file)) && |
| 355 | (strlcat(file, ".pub", sizeof file) < sizeof(file)) && |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 356 | (r = sshkey_try_load_public(pub, file, commentp)) == 0) { |
| 357 | if (keyp != NULL) |
| 358 | *keyp = pub; |
| 359 | return 0; |
| 360 | } |
| 361 | sshkey_free(pub); |
djm@openbsd.org | 5248429 | 2015-02-23 16:55:51 +0000 | [diff] [blame] | 362 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 363 | return r; |
Damien Miller | e4340be | 2000-09-16 13:29:08 +1100 | [diff] [blame] | 364 | } |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 365 | |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 366 | /* Load the certificate associated with the named private key */ |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 367 | int |
| 368 | sshkey_load_cert(const char *filename, struct sshkey **keyp) |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 369 | { |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 370 | struct sshkey *pub = NULL; |
| 371 | char *file = NULL; |
| 372 | int r = SSH_ERR_INTERNAL_ERROR; |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 373 | |
djm@openbsd.org | dce19bf | 2016-04-09 12:39:30 +0000 | [diff] [blame] | 374 | if (keyp != NULL) |
| 375 | *keyp = NULL; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 376 | |
| 377 | if (asprintf(&file, "%s-cert.pub", filename) == -1) |
| 378 | return SSH_ERR_ALLOC_FAIL; |
| 379 | |
| 380 | if ((pub = sshkey_new(KEY_UNSPEC)) == NULL) { |
| 381 | goto out; |
Damien Miller | 5458c4d | 2010-08-05 13:05:15 +1000 | [diff] [blame] | 382 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 383 | if ((r = sshkey_try_load_public(pub, file, NULL)) != 0) |
| 384 | goto out; |
djm@openbsd.org | dce19bf | 2016-04-09 12:39:30 +0000 | [diff] [blame] | 385 | /* success */ |
| 386 | if (keyp != NULL) { |
| 387 | *keyp = pub; |
| 388 | pub = NULL; |
| 389 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 390 | r = 0; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 391 | out: |
mmcc@openbsd.org | d59ce08 | 2015-12-10 17:08:40 +0000 | [diff] [blame] | 392 | free(file); |
mmcc@openbsd.org | 89540b6 | 2015-12-11 02:31:47 +0000 | [diff] [blame] | 393 | sshkey_free(pub); |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 394 | return r; |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 395 | } |
| 396 | |
| 397 | /* Load private key and certificate */ |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 398 | int |
| 399 | sshkey_load_private_cert(int type, const char *filename, const char *passphrase, |
| 400 | struct sshkey **keyp, int *perm_ok) |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 401 | { |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 402 | struct sshkey *key = NULL, *cert = NULL; |
| 403 | int r; |
| 404 | |
djm@openbsd.org | dce19bf | 2016-04-09 12:39:30 +0000 | [diff] [blame] | 405 | if (keyp != NULL) |
| 406 | *keyp = NULL; |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 407 | |
| 408 | switch (type) { |
Damien Miller | 1f0311c | 2014-05-15 14:24:09 +1000 | [diff] [blame] | 409 | #ifdef WITH_OPENSSL |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 410 | case KEY_RSA: |
| 411 | case KEY_DSA: |
Damien Miller | eb8b60e | 2010-08-31 22:41:14 +1000 | [diff] [blame] | 412 | case KEY_ECDSA: |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 413 | #endif /* WITH_OPENSSL */ |
markus@openbsd.org | 16db0a7 | 2015-07-09 09:49:46 +0000 | [diff] [blame] | 414 | case KEY_ED25519: |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 415 | case KEY_UNSPEC: |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 416 | break; |
| 417 | default: |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 418 | return SSH_ERR_KEY_TYPE_UNKNOWN; |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 419 | } |
| 420 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 421 | if ((r = sshkey_load_private_type(type, filename, |
| 422 | passphrase, &key, NULL, perm_ok)) != 0 || |
| 423 | (r = sshkey_load_cert(filename, &cert)) != 0) |
| 424 | goto out; |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 425 | |
| 426 | /* Make sure the private key matches the certificate */ |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 427 | if (sshkey_equal_public(key, cert) == 0) { |
| 428 | r = SSH_ERR_KEY_CERT_MISMATCH; |
| 429 | goto out; |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 430 | } |
| 431 | |
djm@openbsd.org | c28fc62 | 2015-07-03 03:43:18 +0000 | [diff] [blame] | 432 | if ((r = sshkey_to_certified(key)) != 0 || |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 433 | (r = sshkey_cert_copy(cert, key)) != 0) |
| 434 | goto out; |
| 435 | r = 0; |
djm@openbsd.org | dce19bf | 2016-04-09 12:39:30 +0000 | [diff] [blame] | 436 | if (keyp != NULL) { |
| 437 | *keyp = key; |
| 438 | key = NULL; |
| 439 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 440 | out: |
mmcc@openbsd.org | 89540b6 | 2015-12-11 02:31:47 +0000 | [diff] [blame] | 441 | sshkey_free(key); |
| 442 | sshkey_free(cert); |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 443 | return r; |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 444 | } |
| 445 | |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 446 | /* |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 447 | * Returns success if the specified "key" is listed in the file "filename", |
| 448 | * SSH_ERR_KEY_NOT_FOUND: if the key is not listed or another error. |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 449 | * If "strict_type" is set then the key type must match exactly, |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 450 | * otherwise a comparison that ignores certficiate data is performed. |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 451 | * If "check_ca" is set and "key" is a certificate, then its CA key is |
| 452 | * also checked and sshkey_in_file() will return success if either is found. |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 453 | */ |
| 454 | int |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 455 | sshkey_in_file(struct sshkey *key, const char *filename, int strict_type, |
| 456 | int check_ca) |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 457 | { |
| 458 | FILE *f; |
| 459 | char line[SSH_MAX_PUBKEY_BYTES]; |
| 460 | char *cp; |
| 461 | u_long linenum = 0; |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 462 | int r = 0; |
| 463 | struct sshkey *pub = NULL; |
| 464 | int (*sshkey_compare)(const struct sshkey *, const struct sshkey *) = |
| 465 | strict_type ? sshkey_equal : sshkey_equal_public; |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 466 | |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 467 | if ((f = fopen(filename, "r")) == NULL) |
| 468 | return SSH_ERR_SYSTEM_ERROR; |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 469 | |
| 470 | while (read_keyfile_line(f, filename, line, sizeof(line), |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 471 | &linenum) != -1) { |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 472 | cp = line; |
| 473 | |
| 474 | /* Skip leading whitespace. */ |
| 475 | for (; *cp && (*cp == ' ' || *cp == '\t'); cp++) |
| 476 | ; |
| 477 | |
| 478 | /* Skip comments and empty lines */ |
| 479 | switch (*cp) { |
| 480 | case '#': |
| 481 | case '\n': |
| 482 | case '\0': |
| 483 | continue; |
| 484 | } |
| 485 | |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 486 | if ((pub = sshkey_new(KEY_UNSPEC)) == NULL) { |
| 487 | r = SSH_ERR_ALLOC_FAIL; |
| 488 | goto out; |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 489 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 490 | if ((r = sshkey_read(pub, &cp)) != 0) |
| 491 | goto out; |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 492 | if (sshkey_compare(key, pub) || |
| 493 | (check_ca && sshkey_is_cert(key) && |
| 494 | sshkey_compare(key->cert->signature_key, pub))) { |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 495 | r = 0; |
| 496 | goto out; |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 497 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 498 | sshkey_free(pub); |
| 499 | pub = NULL; |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 500 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 501 | r = SSH_ERR_KEY_NOT_FOUND; |
| 502 | out: |
mmcc@openbsd.org | 89540b6 | 2015-12-11 02:31:47 +0000 | [diff] [blame] | 503 | sshkey_free(pub); |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 504 | fclose(f); |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 505 | return r; |
Damien Miller | 1aed65e | 2010-03-04 21:53:35 +1100 | [diff] [blame] | 506 | } |
Damien Miller | 8668706 | 2014-07-02 15:28:02 +1000 | [diff] [blame] | 507 | |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 508 | /* |
| 509 | * Checks whether the specified key is revoked, returning 0 if not, |
| 510 | * SSH_ERR_KEY_REVOKED if it is or another error code if something |
| 511 | * unexpected happened. |
| 512 | * This will check both the key and, if it is a certificate, its CA key too. |
| 513 | * "revoked_keys_file" may be a KRL or a one-per-line list of public keys. |
| 514 | */ |
| 515 | int |
| 516 | sshkey_check_revoked(struct sshkey *key, const char *revoked_keys_file) |
| 517 | { |
| 518 | int r; |
| 519 | |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 520 | r = ssh_krl_file_contains_key(revoked_keys_file, key); |
| 521 | /* If this was not a KRL to begin with then continue below */ |
| 522 | if (r != SSH_ERR_KRL_BAD_MAGIC) |
| 523 | return r; |
djm@openbsd.org | 5e39a49 | 2014-12-04 02:24:32 +0000 | [diff] [blame] | 524 | |
| 525 | /* |
| 526 | * If the file is not a KRL or we can't handle KRLs then attempt to |
| 527 | * parse the file as a flat list of keys. |
| 528 | */ |
| 529 | switch ((r = sshkey_in_file(key, revoked_keys_file, 0, 1))) { |
| 530 | case 0: |
| 531 | /* Key found => revoked */ |
| 532 | return SSH_ERR_KEY_REVOKED; |
| 533 | case SSH_ERR_KEY_NOT_FOUND: |
| 534 | /* Key not found => not revoked */ |
| 535 | return 0; |
| 536 | default: |
| 537 | /* Some other error occurred */ |
| 538 | return r; |
| 539 | } |
| 540 | } |
| 541 | |