blob: e56f884f6f67942e46e8a61d22ab6bbf467a30ef [file] [log] [blame]
Damien Millerf3747bf2013-01-18 11:44:04 +11001/*
2 * Copyright (c) 2012 Damien Miller <djm@mindrot.org>
3 *
4 * Permission to use, copy, modify, and distribute this software for any
5 * purpose with or without fee is hereby granted, provided that the above
6 * copyright notice and this permission notice appear in all copies.
7 *
8 * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
9 * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
10 * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
11 * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
12 * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
13 * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
14 * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
15 */
16
djm@openbsd.orgb6de5ac2014-11-21 01:00:38 +000017/* $OpenBSD: krl.c,v 1.19 2014/11/21 01:00:38 djm Exp $ */
Damien Millerf3747bf2013-01-18 11:44:04 +110018
19#include "includes.h"
20
21#include <sys/types.h>
22#include <sys/param.h>
Damien Millerd60b2102013-01-20 22:49:58 +110023#include <openbsd-compat/sys-tree.h>
24#include <openbsd-compat/sys-queue.h>
Damien Millerf3747bf2013-01-18 11:44:04 +110025
26#include <errno.h>
27#include <fcntl.h>
28#include <limits.h>
29#include <string.h>
30#include <time.h>
31#include <unistd.h>
32
33#include "buffer.h"
34#include "key.h"
35#include "authfile.h"
Damien Millerf3747bf2013-01-18 11:44:04 +110036#include "misc.h"
37#include "log.h"
38#include "xmalloc.h"
39
40#include "krl.h"
41
42/* #define DEBUG_KRL */
43#ifdef DEBUG_KRL
44# define KRL_DBG(x) debug3 x
45#else
46# define KRL_DBG(x)
47#endif
48
49/*
50 * Trees of revoked serial numbers, key IDs and keys. This allows
51 * quick searching, querying and producing lists in canonical order.
52 */
53
54/* Tree of serial numbers. XXX make smarter: really need a real sparse bitmap */
55struct revoked_serial {
56 u_int64_t lo, hi;
57 RB_ENTRY(revoked_serial) tree_entry;
58};
59static int serial_cmp(struct revoked_serial *a, struct revoked_serial *b);
60RB_HEAD(revoked_serial_tree, revoked_serial);
61RB_GENERATE_STATIC(revoked_serial_tree, revoked_serial, tree_entry, serial_cmp);
62
63/* Tree of key IDs */
64struct revoked_key_id {
65 char *key_id;
66 RB_ENTRY(revoked_key_id) tree_entry;
67};
68static int key_id_cmp(struct revoked_key_id *a, struct revoked_key_id *b);
69RB_HEAD(revoked_key_id_tree, revoked_key_id);
70RB_GENERATE_STATIC(revoked_key_id_tree, revoked_key_id, tree_entry, key_id_cmp);
71
72/* Tree of blobs (used for keys and fingerprints) */
73struct revoked_blob {
74 u_char *blob;
75 u_int len;
76 RB_ENTRY(revoked_blob) tree_entry;
77};
78static int blob_cmp(struct revoked_blob *a, struct revoked_blob *b);
79RB_HEAD(revoked_blob_tree, revoked_blob);
80RB_GENERATE_STATIC(revoked_blob_tree, revoked_blob, tree_entry, blob_cmp);
81
82/* Tracks revoked certs for a single CA */
83struct revoked_certs {
84 Key *ca_key;
85 struct revoked_serial_tree revoked_serials;
86 struct revoked_key_id_tree revoked_key_ids;
87 TAILQ_ENTRY(revoked_certs) entry;
88};
89TAILQ_HEAD(revoked_certs_list, revoked_certs);
90
91struct ssh_krl {
92 u_int64_t krl_version;
93 u_int64_t generated_date;
94 u_int64_t flags;
95 char *comment;
96 struct revoked_blob_tree revoked_keys;
97 struct revoked_blob_tree revoked_sha1s;
98 struct revoked_certs_list revoked_certs;
99};
100
101/* Return equal if a and b overlap */
102static int
103serial_cmp(struct revoked_serial *a, struct revoked_serial *b)
104{
105 if (a->hi >= b->lo && a->lo <= b->hi)
106 return 0;
107 return a->lo < b->lo ? -1 : 1;
108}
109
110static int
111key_id_cmp(struct revoked_key_id *a, struct revoked_key_id *b)
112{
113 return strcmp(a->key_id, b->key_id);
114}
115
116static int
117blob_cmp(struct revoked_blob *a, struct revoked_blob *b)
118{
119 int r;
120
121 if (a->len != b->len) {
122 if ((r = memcmp(a->blob, b->blob, MIN(a->len, b->len))) != 0)
123 return r;
124 return a->len > b->len ? 1 : -1;
125 } else
126 return memcmp(a->blob, b->blob, a->len);
127}
128
129struct ssh_krl *
130ssh_krl_init(void)
131{
132 struct ssh_krl *krl;
133
134 if ((krl = calloc(1, sizeof(*krl))) == NULL)
135 return NULL;
136 RB_INIT(&krl->revoked_keys);
137 RB_INIT(&krl->revoked_sha1s);
138 TAILQ_INIT(&krl->revoked_certs);
139 return krl;
140}
141
142static void
143revoked_certs_free(struct revoked_certs *rc)
144{
145 struct revoked_serial *rs, *trs;
146 struct revoked_key_id *rki, *trki;
147
148 RB_FOREACH_SAFE(rs, revoked_serial_tree, &rc->revoked_serials, trs) {
149 RB_REMOVE(revoked_serial_tree, &rc->revoked_serials, rs);
150 free(rs);
151 }
152 RB_FOREACH_SAFE(rki, revoked_key_id_tree, &rc->revoked_key_ids, trki) {
153 RB_REMOVE(revoked_key_id_tree, &rc->revoked_key_ids, rki);
154 free(rki->key_id);
155 free(rki);
156 }
157 if (rc->ca_key != NULL)
158 key_free(rc->ca_key);
159}
160
161void
162ssh_krl_free(struct ssh_krl *krl)
163{
164 struct revoked_blob *rb, *trb;
165 struct revoked_certs *rc, *trc;
166
167 if (krl == NULL)
168 return;
169
170 free(krl->comment);
171 RB_FOREACH_SAFE(rb, revoked_blob_tree, &krl->revoked_keys, trb) {
172 RB_REMOVE(revoked_blob_tree, &krl->revoked_keys, rb);
173 free(rb->blob);
174 free(rb);
175 }
176 RB_FOREACH_SAFE(rb, revoked_blob_tree, &krl->revoked_sha1s, trb) {
177 RB_REMOVE(revoked_blob_tree, &krl->revoked_sha1s, rb);
178 free(rb->blob);
179 free(rb);
180 }
181 TAILQ_FOREACH_SAFE(rc, &krl->revoked_certs, entry, trc) {
182 TAILQ_REMOVE(&krl->revoked_certs, rc, entry);
183 revoked_certs_free(rc);
184 }
185}
186
187void
188ssh_krl_set_version(struct ssh_krl *krl, u_int64_t version)
189{
190 krl->krl_version = version;
191}
192
193void
194ssh_krl_set_comment(struct ssh_krl *krl, const char *comment)
195{
196 free(krl->comment);
197 if ((krl->comment = strdup(comment)) == NULL)
198 fatal("%s: strdup", __func__);
199}
200
201/*
202 * Find the revoked_certs struct for a CA key. If allow_create is set then
203 * create a new one in the tree if one did not exist already.
204 */
205static int
206revoked_certs_for_ca_key(struct ssh_krl *krl, const Key *ca_key,
207 struct revoked_certs **rcp, int allow_create)
208{
209 struct revoked_certs *rc;
210
211 *rcp = NULL;
212 TAILQ_FOREACH(rc, &krl->revoked_certs, entry) {
213 if (key_equal(rc->ca_key, ca_key)) {
214 *rcp = rc;
215 return 0;
216 }
217 }
218 if (!allow_create)
219 return 0;
220 /* If this CA doesn't exist in the list then add it now */
221 if ((rc = calloc(1, sizeof(*rc))) == NULL)
222 return -1;
223 if ((rc->ca_key = key_from_private(ca_key)) == NULL) {
224 free(rc);
225 return -1;
226 }
227 RB_INIT(&rc->revoked_serials);
228 RB_INIT(&rc->revoked_key_ids);
229 TAILQ_INSERT_TAIL(&krl->revoked_certs, rc, entry);
230 debug3("%s: new CA %s", __func__, key_type(ca_key));
231 *rcp = rc;
232 return 0;
233}
234
235static int
236insert_serial_range(struct revoked_serial_tree *rt, u_int64_t lo, u_int64_t hi)
237{
238 struct revoked_serial rs, *ers, *crs, *irs;
239
240 KRL_DBG(("%s: insert %llu:%llu", __func__, lo, hi));
Damien Miller1d2c4562014-02-04 11:18:20 +1100241 memset(&rs, 0, sizeof(rs));
Damien Millerf3747bf2013-01-18 11:44:04 +1100242 rs.lo = lo;
243 rs.hi = hi;
244 ers = RB_NFIND(revoked_serial_tree, rt, &rs);
245 if (ers == NULL || serial_cmp(ers, &rs) != 0) {
246 /* No entry matches. Just insert */
247 if ((irs = malloc(sizeof(rs))) == NULL)
248 return -1;
249 memcpy(irs, &rs, sizeof(*irs));
250 ers = RB_INSERT(revoked_serial_tree, rt, irs);
251 if (ers != NULL) {
252 KRL_DBG(("%s: bad: ers != NULL", __func__));
253 /* Shouldn't happen */
Damien Millera7522d92013-01-20 22:35:31 +1100254 free(irs);
Damien Millerf3747bf2013-01-18 11:44:04 +1100255 return -1;
256 }
257 ers = irs;
258 } else {
259 KRL_DBG(("%s: overlap found %llu:%llu", __func__,
260 ers->lo, ers->hi));
261 /*
262 * The inserted entry overlaps an existing one. Grow the
263 * existing entry.
264 */
265 if (ers->lo > lo)
266 ers->lo = lo;
267 if (ers->hi < hi)
268 ers->hi = hi;
269 }
270 /*
271 * The inserted or revised range might overlap or abut adjacent ones;
272 * coalesce as necessary.
273 */
274
275 /* Check predecessors */
276 while ((crs = RB_PREV(revoked_serial_tree, rt, ers)) != NULL) {
277 KRL_DBG(("%s: pred %llu:%llu", __func__, crs->lo, crs->hi));
278 if (ers->lo != 0 && crs->hi < ers->lo - 1)
279 break;
280 /* This entry overlaps. */
281 if (crs->lo < ers->lo) {
282 ers->lo = crs->lo;
283 KRL_DBG(("%s: pred extend %llu:%llu", __func__,
284 ers->lo, ers->hi));
285 }
286 RB_REMOVE(revoked_serial_tree, rt, crs);
287 free(crs);
288 }
289 /* Check successors */
290 while ((crs = RB_NEXT(revoked_serial_tree, rt, ers)) != NULL) {
291 KRL_DBG(("%s: succ %llu:%llu", __func__, crs->lo, crs->hi));
292 if (ers->hi != (u_int64_t)-1 && crs->lo > ers->hi + 1)
293 break;
294 /* This entry overlaps. */
295 if (crs->hi > ers->hi) {
296 ers->hi = crs->hi;
297 KRL_DBG(("%s: succ extend %llu:%llu", __func__,
298 ers->lo, ers->hi));
299 }
300 RB_REMOVE(revoked_serial_tree, rt, crs);
301 free(crs);
302 }
303 KRL_DBG(("%s: done, final %llu:%llu", __func__, ers->lo, ers->hi));
304 return 0;
305}
306
307int
308ssh_krl_revoke_cert_by_serial(struct ssh_krl *krl, const Key *ca_key,
309 u_int64_t serial)
310{
311 return ssh_krl_revoke_cert_by_serial_range(krl, ca_key, serial, serial);
312}
313
314int
315ssh_krl_revoke_cert_by_serial_range(struct ssh_krl *krl, const Key *ca_key,
316 u_int64_t lo, u_int64_t hi)
317{
318 struct revoked_certs *rc;
319
320 if (lo > hi || lo == 0)
321 return -1;
322 if (revoked_certs_for_ca_key(krl, ca_key, &rc, 1) != 0)
323 return -1;
324 return insert_serial_range(&rc->revoked_serials, lo, hi);
325}
326
327int
328ssh_krl_revoke_cert_by_key_id(struct ssh_krl *krl, const Key *ca_key,
329 const char *key_id)
330{
331 struct revoked_key_id *rki, *erki;
332 struct revoked_certs *rc;
333
334 if (revoked_certs_for_ca_key(krl, ca_key, &rc, 1) != 0)
335 return -1;
336
337 debug3("%s: revoke %s", __func__, key_id);
338 if ((rki = calloc(1, sizeof(*rki))) == NULL ||
339 (rki->key_id = strdup(key_id)) == NULL) {
340 free(rki);
341 fatal("%s: strdup", __func__);
342 }
343 erki = RB_INSERT(revoked_key_id_tree, &rc->revoked_key_ids, rki);
344 if (erki != NULL) {
345 free(rki->key_id);
346 free(rki);
347 }
348 return 0;
349}
350
351/* Convert "key" to a public key blob without any certificate information */
352static int
353plain_key_blob(const Key *key, u_char **blob, u_int *blen)
354{
355 Key *kcopy;
356 int r;
357
358 if ((kcopy = key_from_private(key)) == NULL)
359 return -1;
360 if (key_is_cert(kcopy)) {
361 if (key_drop_cert(kcopy) != 0) {
362 error("%s: key_drop_cert", __func__);
363 key_free(kcopy);
364 return -1;
365 }
366 }
367 r = key_to_blob(kcopy, blob, blen);
368 free(kcopy);
Damien Miller86687062014-07-02 15:28:02 +1000369 return r;
Damien Millerf3747bf2013-01-18 11:44:04 +1100370}
371
372/* Revoke a key blob. Ownership of blob is transferred to the tree */
373static int
374revoke_blob(struct revoked_blob_tree *rbt, u_char *blob, u_int len)
375{
376 struct revoked_blob *rb, *erb;
377
378 if ((rb = calloc(1, sizeof(*rb))) == NULL)
379 return -1;
380 rb->blob = blob;
381 rb->len = len;
382 erb = RB_INSERT(revoked_blob_tree, rbt, rb);
383 if (erb != NULL) {
384 free(rb->blob);
385 free(rb);
386 }
387 return 0;
388}
389
390int
391ssh_krl_revoke_key_explicit(struct ssh_krl *krl, const Key *key)
392{
393 u_char *blob;
394 u_int len;
395
396 debug3("%s: revoke type %s", __func__, key_type(key));
Damien Miller86687062014-07-02 15:28:02 +1000397 if (plain_key_blob(key, &blob, &len) < 0)
Damien Millerf3747bf2013-01-18 11:44:04 +1100398 return -1;
399 return revoke_blob(&krl->revoked_keys, blob, len);
400}
401
402int
403ssh_krl_revoke_key_sha1(struct ssh_krl *krl, const Key *key)
404{
405 u_char *blob;
406 u_int len;
407
408 debug3("%s: revoke type %s by sha1", __func__, key_type(key));
409 if ((blob = key_fingerprint_raw(key, SSH_FP_SHA1, &len)) == NULL)
410 return -1;
411 return revoke_blob(&krl->revoked_sha1s, blob, len);
412}
413
414int
415ssh_krl_revoke_key(struct ssh_krl *krl, const Key *key)
416{
417 if (!key_is_cert(key))
418 return ssh_krl_revoke_key_sha1(krl, key);
419
420 if (key_cert_is_legacy(key) || key->cert->serial == 0) {
421 return ssh_krl_revoke_cert_by_key_id(krl,
422 key->cert->signature_key,
423 key->cert->key_id);
424 } else {
425 return ssh_krl_revoke_cert_by_serial(krl,
426 key->cert->signature_key,
427 key->cert->serial);
428 }
429}
430
431/*
432 * Select a copact next section type to emit in a KRL based on the
433 * current section type, the run length of contiguous revoked serial
434 * numbers and the gaps from the last and to the next revoked serial.
435 * Applies a mostly-accurate bit cost model to select the section type
436 * that will minimise the size of the resultant KRL.
437 */
438static int
439choose_next_state(int current_state, u_int64_t contig, int final,
440 u_int64_t last_gap, u_int64_t next_gap, int *force_new_section)
441{
442 int new_state;
443 u_int64_t cost, cost_list, cost_range, cost_bitmap, cost_bitmap_restart;
444
445 /*
446 * Avoid unsigned overflows.
447 * The limits are high enough to avoid confusing the calculations.
448 */
449 contig = MIN(contig, 1ULL<<31);
450 last_gap = MIN(last_gap, 1ULL<<31);
451 next_gap = MIN(next_gap, 1ULL<<31);
452
453 /*
454 * Calculate the cost to switch from the current state to candidates.
455 * NB. range sections only ever contain a single range, so their
456 * switching cost is independent of the current_state.
457 */
458 cost_list = cost_bitmap = cost_bitmap_restart = 0;
459 cost_range = 8;
460 switch (current_state) {
461 case KRL_SECTION_CERT_SERIAL_LIST:
462 cost_bitmap_restart = cost_bitmap = 8 + 64;
463 break;
464 case KRL_SECTION_CERT_SERIAL_BITMAP:
465 cost_list = 8;
466 cost_bitmap_restart = 8 + 64;
467 break;
468 case KRL_SECTION_CERT_SERIAL_RANGE:
469 case 0:
470 cost_bitmap_restart = cost_bitmap = 8 + 64;
471 cost_list = 8;
472 }
473
474 /* Estimate base cost in bits of each section type */
475 cost_list += 64 * contig + (final ? 0 : 8+64);
476 cost_range += (2 * 64) + (final ? 0 : 8+64);
477 cost_bitmap += last_gap + contig + (final ? 0 : MIN(next_gap, 8+64));
478 cost_bitmap_restart += contig + (final ? 0 : MIN(next_gap, 8+64));
479
480 /* Convert to byte costs for actual comparison */
481 cost_list = (cost_list + 7) / 8;
482 cost_bitmap = (cost_bitmap + 7) / 8;
483 cost_bitmap_restart = (cost_bitmap_restart + 7) / 8;
484 cost_range = (cost_range + 7) / 8;
485
486 /* Now pick the best choice */
487 *force_new_section = 0;
488 new_state = KRL_SECTION_CERT_SERIAL_BITMAP;
489 cost = cost_bitmap;
490 if (cost_range < cost) {
491 new_state = KRL_SECTION_CERT_SERIAL_RANGE;
492 cost = cost_range;
493 }
494 if (cost_list < cost) {
495 new_state = KRL_SECTION_CERT_SERIAL_LIST;
496 cost = cost_list;
497 }
498 if (cost_bitmap_restart < cost) {
499 new_state = KRL_SECTION_CERT_SERIAL_BITMAP;
500 *force_new_section = 1;
501 cost = cost_bitmap_restart;
502 }
503 debug3("%s: contig %llu last_gap %llu next_gap %llu final %d, costs:"
504 "list %llu range %llu bitmap %llu new bitmap %llu, "
Damien Millerd677ad12013-04-23 15:18:51 +1000505 "selected 0x%02x%s", __func__, (long long unsigned)contig,
506 (long long unsigned)last_gap, (long long unsigned)next_gap, final,
507 (long long unsigned)cost_list, (long long unsigned)cost_range,
508 (long long unsigned)cost_bitmap,
509 (long long unsigned)cost_bitmap_restart, new_state,
Damien Millerf3747bf2013-01-18 11:44:04 +1100510 *force_new_section ? " restart" : "");
511 return new_state;
512}
513
514/* Generate a KRL_SECTION_CERTIFICATES KRL section */
515static int
516revoked_certs_generate(struct revoked_certs *rc, Buffer *buf)
517{
518 int final, force_new_sect, r = -1;
519 u_int64_t i, contig, gap, last = 0, bitmap_start = 0;
520 struct revoked_serial *rs, *nrs;
521 struct revoked_key_id *rki;
522 int next_state, state = 0;
523 Buffer sect;
524 u_char *kblob = NULL;
525 u_int klen;
526 BIGNUM *bitmap = NULL;
527
528 /* Prepare CA scope key blob if we have one supplied */
529 if (key_to_blob(rc->ca_key, &kblob, &klen) == 0)
530 return -1;
531
532 buffer_init(&sect);
533
534 /* Store the header */
535 buffer_put_string(buf, kblob, klen);
536 buffer_put_string(buf, NULL, 0); /* Reserved */
537
538 free(kblob);
539
540 /* Store the revoked serials. */
541 for (rs = RB_MIN(revoked_serial_tree, &rc->revoked_serials);
542 rs != NULL;
543 rs = RB_NEXT(revoked_serial_tree, &rc->revoked_serials, rs)) {
544 debug3("%s: serial %llu:%llu state 0x%02x", __func__,
Damien Millerd677ad12013-04-23 15:18:51 +1000545 (long long unsigned)rs->lo, (long long unsigned)rs->hi,
546 state);
Damien Millerf3747bf2013-01-18 11:44:04 +1100547
548 /* Check contiguous length and gap to next section (if any) */
549 nrs = RB_NEXT(revoked_serial_tree, &rc->revoked_serials, rs);
550 final = nrs == NULL;
551 gap = nrs == NULL ? 0 : nrs->lo - rs->hi;
552 contig = 1 + (rs->hi - rs->lo);
553
554 /* Choose next state based on these */
555 next_state = choose_next_state(state, contig, final,
556 state == 0 ? 0 : rs->lo - last, gap, &force_new_sect);
557
558 /*
559 * If the current section is a range section or has a different
560 * type to the next section, then finish it off now.
561 */
562 if (state != 0 && (force_new_sect || next_state != state ||
563 state == KRL_SECTION_CERT_SERIAL_RANGE)) {
564 debug3("%s: finish state 0x%02x", __func__, state);
565 switch (state) {
566 case KRL_SECTION_CERT_SERIAL_LIST:
567 case KRL_SECTION_CERT_SERIAL_RANGE:
568 break;
569 case KRL_SECTION_CERT_SERIAL_BITMAP:
570 buffer_put_bignum2(&sect, bitmap);
571 BN_free(bitmap);
572 bitmap = NULL;
573 break;
574 }
575 buffer_put_char(buf, state);
576 buffer_put_string(buf,
577 buffer_ptr(&sect), buffer_len(&sect));
Damien Miller2cd79292014-07-02 12:48:30 +1000578 buffer_clear(&sect);
Damien Millerf3747bf2013-01-18 11:44:04 +1100579 }
580
581 /* If we are starting a new section then prepare it now */
582 if (next_state != state || force_new_sect) {
583 debug3("%s: start state 0x%02x", __func__, next_state);
584 state = next_state;
585 buffer_clear(&sect);
586 switch (state) {
587 case KRL_SECTION_CERT_SERIAL_LIST:
588 case KRL_SECTION_CERT_SERIAL_RANGE:
589 break;
590 case KRL_SECTION_CERT_SERIAL_BITMAP:
591 if ((bitmap = BN_new()) == NULL)
592 goto out;
593 bitmap_start = rs->lo;
594 buffer_put_int64(&sect, bitmap_start);
595 break;
596 }
597 }
598
599 /* Perform section-specific processing */
600 switch (state) {
601 case KRL_SECTION_CERT_SERIAL_LIST:
Damien Miller13f5f762013-01-18 15:32:03 +1100602 for (i = 0; i < contig; i++)
Damien Millerf3747bf2013-01-18 11:44:04 +1100603 buffer_put_int64(&sect, rs->lo + i);
604 break;
605 case KRL_SECTION_CERT_SERIAL_RANGE:
606 buffer_put_int64(&sect, rs->lo);
607 buffer_put_int64(&sect, rs->hi);
608 break;
609 case KRL_SECTION_CERT_SERIAL_BITMAP:
610 if (rs->lo - bitmap_start > INT_MAX) {
611 error("%s: insane bitmap gap", __func__);
612 goto out;
613 }
614 for (i = 0; i < contig; i++) {
615 if (BN_set_bit(bitmap,
616 rs->lo + i - bitmap_start) != 1)
617 goto out;
618 }
619 break;
620 }
621 last = rs->hi;
622 }
623 /* Flush the remaining section, if any */
624 if (state != 0) {
625 debug3("%s: serial final flush for state 0x%02x",
626 __func__, state);
627 switch (state) {
628 case KRL_SECTION_CERT_SERIAL_LIST:
629 case KRL_SECTION_CERT_SERIAL_RANGE:
630 break;
631 case KRL_SECTION_CERT_SERIAL_BITMAP:
632 buffer_put_bignum2(&sect, bitmap);
633 BN_free(bitmap);
634 bitmap = NULL;
635 break;
636 }
637 buffer_put_char(buf, state);
638 buffer_put_string(buf,
639 buffer_ptr(&sect), buffer_len(&sect));
640 }
641 debug3("%s: serial done ", __func__);
642
643 /* Now output a section for any revocations by key ID */
644 buffer_clear(&sect);
645 RB_FOREACH(rki, revoked_key_id_tree, &rc->revoked_key_ids) {
646 debug3("%s: key ID %s", __func__, rki->key_id);
647 buffer_put_cstring(&sect, rki->key_id);
648 }
649 if (buffer_len(&sect) != 0) {
650 buffer_put_char(buf, KRL_SECTION_CERT_KEY_ID);
651 buffer_put_string(buf, buffer_ptr(&sect),
652 buffer_len(&sect));
653 }
654 r = 0;
655 out:
656 if (bitmap != NULL)
657 BN_free(bitmap);
658 buffer_free(&sect);
659 return r;
660}
661
662int
663ssh_krl_to_blob(struct ssh_krl *krl, Buffer *buf, const Key **sign_keys,
664 u_int nsign_keys)
665{
666 int r = -1;
667 struct revoked_certs *rc;
668 struct revoked_blob *rb;
669 Buffer sect;
670 u_char *kblob = NULL, *sblob = NULL;
671 u_int klen, slen, i;
672
673 if (krl->generated_date == 0)
674 krl->generated_date = time(NULL);
675
676 buffer_init(&sect);
677
678 /* Store the header */
679 buffer_append(buf, KRL_MAGIC, sizeof(KRL_MAGIC) - 1);
680 buffer_put_int(buf, KRL_FORMAT_VERSION);
681 buffer_put_int64(buf, krl->krl_version);
682 buffer_put_int64(buf, krl->generated_date);
683 buffer_put_int64(buf, krl->flags);
684 buffer_put_string(buf, NULL, 0);
685 buffer_put_cstring(buf, krl->comment ? krl->comment : "");
686
687 /* Store sections for revoked certificates */
688 TAILQ_FOREACH(rc, &krl->revoked_certs, entry) {
djm@openbsd.org9f9fad02014-11-17 00:21:40 +0000689 buffer_clear(&sect);
Damien Millerf3747bf2013-01-18 11:44:04 +1100690 if (revoked_certs_generate(rc, &sect) != 0)
691 goto out;
692 buffer_put_char(buf, KRL_SECTION_CERTIFICATES);
693 buffer_put_string(buf, buffer_ptr(&sect),
694 buffer_len(&sect));
695 }
696
697 /* Finally, output sections for revocations by public key/hash */
698 buffer_clear(&sect);
699 RB_FOREACH(rb, revoked_blob_tree, &krl->revoked_keys) {
700 debug3("%s: key len %u ", __func__, rb->len);
701 buffer_put_string(&sect, rb->blob, rb->len);
702 }
703 if (buffer_len(&sect) != 0) {
704 buffer_put_char(buf, KRL_SECTION_EXPLICIT_KEY);
705 buffer_put_string(buf, buffer_ptr(&sect),
706 buffer_len(&sect));
707 }
708 buffer_clear(&sect);
709 RB_FOREACH(rb, revoked_blob_tree, &krl->revoked_sha1s) {
710 debug3("%s: hash len %u ", __func__, rb->len);
711 buffer_put_string(&sect, rb->blob, rb->len);
712 }
713 if (buffer_len(&sect) != 0) {
714 buffer_put_char(buf, KRL_SECTION_FINGERPRINT_SHA1);
715 buffer_put_string(buf, buffer_ptr(&sect),
716 buffer_len(&sect));
717 }
718
719 for (i = 0; i < nsign_keys; i++) {
720 if (key_to_blob(sign_keys[i], &kblob, &klen) == 0)
721 goto out;
722
723 debug3("%s: signature key len %u", __func__, klen);
724 buffer_put_char(buf, KRL_SECTION_SIGNATURE);
725 buffer_put_string(buf, kblob, klen);
726
727 if (key_sign(sign_keys[i], &sblob, &slen,
728 buffer_ptr(buf), buffer_len(buf)) == -1)
729 goto out;
730 debug3("%s: signature sig len %u", __func__, slen);
731 buffer_put_string(buf, sblob, slen);
732 }
733
734 r = 0;
735 out:
736 free(kblob);
737 free(sblob);
738 buffer_free(&sect);
739 return r;
740}
741
742static void
743format_timestamp(u_int64_t timestamp, char *ts, size_t nts)
744{
745 time_t t;
746 struct tm *tm;
747
748 t = timestamp;
749 tm = localtime(&t);
djm@openbsd.orgb6de5ac2014-11-21 01:00:38 +0000750 if (tm == NULL)
751 strlcpy(ts, "<INVALID>", sizeof(nts));
752 else {
753 *ts = '\0';
754 strftime(ts, nts, "%Y%m%dT%H%M%S", tm);
755 }
Damien Millerf3747bf2013-01-18 11:44:04 +1100756}
757
758static int
759parse_revoked_certs(Buffer *buf, struct ssh_krl *krl)
760{
761 int ret = -1, nbits;
Damien Miller633de332014-05-15 13:48:26 +1000762 u_char type;
763 const u_char *blob;
Damien Millerf3747bf2013-01-18 11:44:04 +1100764 u_int blen;
765 Buffer subsect;
766 u_int64_t serial, serial_lo, serial_hi;
767 BIGNUM *bitmap = NULL;
768 char *key_id = NULL;
769 Key *ca_key = NULL;
770
771 buffer_init(&subsect);
772
773 if ((blob = buffer_get_string_ptr_ret(buf, &blen)) == NULL ||
774 buffer_get_string_ptr_ret(buf, NULL) == NULL) { /* reserved */
775 error("%s: buffer error", __func__);
776 goto out;
777 }
778 if ((ca_key = key_from_blob(blob, blen)) == NULL)
779 goto out;
780
781 while (buffer_len(buf) > 0) {
782 if (buffer_get_char_ret(&type, buf) != 0 ||
783 (blob = buffer_get_string_ptr_ret(buf, &blen)) == NULL) {
784 error("%s: buffer error", __func__);
785 goto out;
786 }
787 buffer_clear(&subsect);
788 buffer_append(&subsect, blob, blen);
789 debug3("%s: subsection type 0x%02x", __func__, type);
790 /* buffer_dump(&subsect); */
791
792 switch (type) {
793 case KRL_SECTION_CERT_SERIAL_LIST:
794 while (buffer_len(&subsect) > 0) {
795 if (buffer_get_int64_ret(&serial,
796 &subsect) != 0) {
797 error("%s: buffer error", __func__);
798 goto out;
799 }
800 if (ssh_krl_revoke_cert_by_serial(krl, ca_key,
801 serial) != 0) {
802 error("%s: update failed", __func__);
803 goto out;
804 }
805 }
806 break;
807 case KRL_SECTION_CERT_SERIAL_RANGE:
808 if (buffer_get_int64_ret(&serial_lo, &subsect) != 0 ||
809 buffer_get_int64_ret(&serial_hi, &subsect) != 0) {
810 error("%s: buffer error", __func__);
811 goto out;
812 }
813 if (ssh_krl_revoke_cert_by_serial_range(krl, ca_key,
814 serial_lo, serial_hi) != 0) {
815 error("%s: update failed", __func__);
816 goto out;
817 }
818 break;
819 case KRL_SECTION_CERT_SERIAL_BITMAP:
820 if ((bitmap = BN_new()) == NULL) {
821 error("%s: BN_new", __func__);
822 goto out;
823 }
824 if (buffer_get_int64_ret(&serial_lo, &subsect) != 0 ||
825 buffer_get_bignum2_ret(&subsect, bitmap) != 0) {
826 error("%s: buffer error", __func__);
827 goto out;
828 }
829 if ((nbits = BN_num_bits(bitmap)) < 0) {
830 error("%s: bitmap bits < 0", __func__);
831 goto out;
832 }
833 for (serial = 0; serial < (u_int)nbits; serial++) {
834 if (serial > 0 && serial_lo + serial == 0) {
835 error("%s: bitmap wraps u64", __func__);
836 goto out;
837 }
838 if (!BN_is_bit_set(bitmap, serial))
839 continue;
840 if (ssh_krl_revoke_cert_by_serial(krl, ca_key,
841 serial_lo + serial) != 0) {
842 error("%s: update failed", __func__);
843 goto out;
844 }
845 }
846 BN_free(bitmap);
847 bitmap = NULL;
848 break;
849 case KRL_SECTION_CERT_KEY_ID:
850 while (buffer_len(&subsect) > 0) {
851 if ((key_id = buffer_get_cstring_ret(&subsect,
852 NULL)) == NULL) {
853 error("%s: buffer error", __func__);
854 goto out;
855 }
856 if (ssh_krl_revoke_cert_by_key_id(krl, ca_key,
857 key_id) != 0) {
858 error("%s: update failed", __func__);
859 goto out;
860 }
861 free(key_id);
862 key_id = NULL;
863 }
864 break;
865 default:
866 error("Unsupported KRL certificate section %u", type);
867 goto out;
868 }
869 if (buffer_len(&subsect) > 0) {
870 error("KRL certificate section contains unparsed data");
871 goto out;
872 }
873 }
874
875 ret = 0;
876 out:
877 if (ca_key != NULL)
878 key_free(ca_key);
879 if (bitmap != NULL)
880 BN_free(bitmap);
881 free(key_id);
882 buffer_free(&subsect);
883 return ret;
884}
885
886
887/* Attempt to parse a KRL, checking its signature (if any) with sign_ca_keys. */
888int
889ssh_krl_from_blob(Buffer *buf, struct ssh_krl **krlp,
890 const Key **sign_ca_keys, u_int nsign_ca_keys)
891{
892 Buffer copy, sect;
893 struct ssh_krl *krl;
894 char timestamp[64];
895 int ret = -1, r, sig_seen;
896 Key *key = NULL, **ca_used = NULL;
Damien Miller633de332014-05-15 13:48:26 +1000897 u_char type, *rdata = NULL;
898 const u_char *blob;
Damien Miller30710702013-07-18 16:09:44 +1000899 u_int i, j, sig_off, sects_off, rlen, blen, format_version, nca_used;
Damien Millerf3747bf2013-01-18 11:44:04 +1100900
Damien Miller30710702013-07-18 16:09:44 +1000901 nca_used = 0;
Damien Millerf3747bf2013-01-18 11:44:04 +1100902 *krlp = NULL;
903 if (buffer_len(buf) < sizeof(KRL_MAGIC) - 1 ||
904 memcmp(buffer_ptr(buf), KRL_MAGIC, sizeof(KRL_MAGIC) - 1) != 0) {
905 debug3("%s: not a KRL", __func__);
906 /*
907 * Return success but a NULL *krlp here to signal that the
908 * file might be a simple list of keys.
909 */
910 return 0;
911 }
912
913 /* Take a copy of the KRL buffer so we can verify its signature later */
914 buffer_init(&copy);
915 buffer_append(&copy, buffer_ptr(buf), buffer_len(buf));
916
917 buffer_init(&sect);
918 buffer_consume(&copy, sizeof(KRL_MAGIC) - 1);
919
920 if ((krl = ssh_krl_init()) == NULL) {
921 error("%s: alloc failed", __func__);
922 goto out;
923 }
924
925 if (buffer_get_int_ret(&format_version, &copy) != 0) {
926 error("%s: KRL truncated", __func__);
927 goto out;
928 }
929 if (format_version != KRL_FORMAT_VERSION) {
930 error("%s: KRL unsupported format version %u",
931 __func__, format_version);
932 goto out;
933 }
934 if (buffer_get_int64_ret(&krl->krl_version, &copy) != 0 ||
935 buffer_get_int64_ret(&krl->generated_date, &copy) != 0 ||
936 buffer_get_int64_ret(&krl->flags, &copy) != 0 ||
937 buffer_get_string_ptr_ret(&copy, NULL) == NULL || /* reserved */
938 (krl->comment = buffer_get_cstring_ret(&copy, NULL)) == NULL) {
939 error("%s: buffer error", __func__);
940 goto out;
941 }
942
943 format_timestamp(krl->generated_date, timestamp, sizeof(timestamp));
Damien Millerd677ad12013-04-23 15:18:51 +1000944 debug("KRL version %llu generated at %s%s%s",
945 (long long unsigned)krl->krl_version, timestamp,
946 *krl->comment ? ": " : "", krl->comment);
Damien Millerf3747bf2013-01-18 11:44:04 +1100947
948 /*
949 * 1st pass: verify signatures, if any. This is done to avoid
950 * detailed parsing of data whose provenance is unverified.
951 */
952 sig_seen = 0;
953 sects_off = buffer_len(buf) - buffer_len(&copy);
954 while (buffer_len(&copy) > 0) {
955 if (buffer_get_char_ret(&type, &copy) != 0 ||
956 (blob = buffer_get_string_ptr_ret(&copy, &blen)) == NULL) {
957 error("%s: buffer error", __func__);
958 goto out;
959 }
960 debug3("%s: first pass, section 0x%02x", __func__, type);
961 if (type != KRL_SECTION_SIGNATURE) {
962 if (sig_seen) {
963 error("KRL contains non-signature section "
964 "after signature");
965 goto out;
966 }
967 /* Not interested for now. */
968 continue;
969 }
970 sig_seen = 1;
971 /* First string component is the signing key */
972 if ((key = key_from_blob(blob, blen)) == NULL) {
973 error("%s: invalid signature key", __func__);
974 goto out;
975 }
976 sig_off = buffer_len(buf) - buffer_len(&copy);
977 /* Second string component is the signature itself */
978 if ((blob = buffer_get_string_ptr_ret(&copy, &blen)) == NULL) {
979 error("%s: buffer error", __func__);
980 goto out;
981 }
982 /* Check signature over entire KRL up to this point */
983 if (key_verify(key, blob, blen,
Damien Millerc8669a82013-07-25 11:52:48 +1000984 buffer_ptr(buf), buffer_len(buf) - sig_off) != 1) {
Damien Millerf3747bf2013-01-18 11:44:04 +1100985 error("bad signaure on KRL");
986 goto out;
987 }
988 /* Check if this key has already signed this KRL */
989 for (i = 0; i < nca_used; i++) {
990 if (key_equal(ca_used[i], key)) {
991 error("KRL signed more than once with "
992 "the same key");
993 goto out;
994 }
995 }
996 /* Record keys used to sign the KRL */
Damien Miller0cd2f8e2013-02-12 11:01:39 +1100997 ca_used = xrealloc(ca_used, nca_used + 1, sizeof(*ca_used));
Damien Millerf3747bf2013-01-18 11:44:04 +1100998 ca_used[nca_used++] = key;
999 key = NULL;
1000 break;
1001 }
1002
1003 /*
1004 * 2nd pass: parse and load the KRL, skipping the header to the point
1005 * where the section start.
1006 */
1007 buffer_append(&copy, (u_char*)buffer_ptr(buf) + sects_off,
1008 buffer_len(buf) - sects_off);
1009 while (buffer_len(&copy) > 0) {
1010 if (buffer_get_char_ret(&type, &copy) != 0 ||
1011 (blob = buffer_get_string_ptr_ret(&copy, &blen)) == NULL) {
1012 error("%s: buffer error", __func__);
1013 goto out;
1014 }
1015 debug3("%s: second pass, section 0x%02x", __func__, type);
1016 buffer_clear(&sect);
1017 buffer_append(&sect, blob, blen);
1018
1019 switch (type) {
1020 case KRL_SECTION_CERTIFICATES:
1021 if ((r = parse_revoked_certs(&sect, krl)) != 0)
1022 goto out;
1023 break;
1024 case KRL_SECTION_EXPLICIT_KEY:
1025 case KRL_SECTION_FINGERPRINT_SHA1:
1026 while (buffer_len(&sect) > 0) {
Damien Miller30710702013-07-18 16:09:44 +10001027 if ((rdata = buffer_get_string_ret(&sect,
1028 &rlen)) == NULL) {
Damien Millerf3747bf2013-01-18 11:44:04 +11001029 error("%s: buffer error", __func__);
1030 goto out;
1031 }
1032 if (type == KRL_SECTION_FINGERPRINT_SHA1 &&
Damien Miller30710702013-07-18 16:09:44 +10001033 rlen != 20) {
Damien Millerf3747bf2013-01-18 11:44:04 +11001034 error("%s: bad SHA1 length", __func__);
1035 goto out;
1036 }
1037 if (revoke_blob(
1038 type == KRL_SECTION_EXPLICIT_KEY ?
1039 &krl->revoked_keys : &krl->revoked_sha1s,
Damien Miller30710702013-07-18 16:09:44 +10001040 rdata, rlen) != 0)
1041 goto out;
1042 rdata = NULL; /* revoke_blob frees blob */
Damien Millerf3747bf2013-01-18 11:44:04 +11001043 }
1044 break;
1045 case KRL_SECTION_SIGNATURE:
1046 /* Handled above, but still need to stay in synch */
1047 buffer_clear(&sect);
Damien Millerea078462013-02-12 10:54:37 +11001048 if ((blob = buffer_get_string_ptr_ret(&copy,
Damien Millerf3747bf2013-01-18 11:44:04 +11001049 &blen)) == NULL) {
1050 error("%s: buffer error", __func__);
1051 goto out;
1052 }
1053 break;
1054 default:
1055 error("Unsupported KRL section %u", type);
1056 goto out;
1057 }
1058 if (buffer_len(&sect) > 0) {
1059 error("KRL section contains unparsed data");
1060 goto out;
1061 }
1062 }
1063
1064 /* Check that the key(s) used to sign the KRL weren't revoked */
1065 sig_seen = 0;
1066 for (i = 0; i < nca_used; i++) {
1067 if (ssh_krl_check_key(krl, ca_used[i]) == 0)
1068 sig_seen = 1;
1069 else {
1070 key_free(ca_used[i]);
1071 ca_used[i] = NULL;
1072 }
1073 }
1074 if (nca_used && !sig_seen) {
1075 error("All keys used to sign KRL were revoked");
1076 goto out;
1077 }
1078
1079 /* If we have CA keys, then verify that one was used to sign the KRL */
1080 if (sig_seen && nsign_ca_keys != 0) {
1081 sig_seen = 0;
1082 for (i = 0; !sig_seen && i < nsign_ca_keys; i++) {
1083 for (j = 0; j < nca_used; j++) {
1084 if (ca_used[j] == NULL)
1085 continue;
1086 if (key_equal(ca_used[j], sign_ca_keys[i])) {
1087 sig_seen = 1;
1088 break;
1089 }
1090 }
1091 }
1092 if (!sig_seen) {
1093 error("KRL not signed with any trusted key");
1094 goto out;
1095 }
1096 }
1097
1098 *krlp = krl;
1099 ret = 0;
1100 out:
1101 if (ret != 0)
1102 ssh_krl_free(krl);
1103 for (i = 0; i < nca_used; i++) {
1104 if (ca_used[i] != NULL)
1105 key_free(ca_used[i]);
1106 }
1107 free(ca_used);
Damien Miller30710702013-07-18 16:09:44 +10001108 free(rdata);
Damien Millerf3747bf2013-01-18 11:44:04 +11001109 if (key != NULL)
1110 key_free(key);
1111 buffer_free(&copy);
1112 buffer_free(&sect);
1113 return ret;
1114}
1115
1116/* Checks whether a given key/cert is revoked. Does not check its CA */
1117static int
1118is_key_revoked(struct ssh_krl *krl, const Key *key)
1119{
1120 struct revoked_blob rb, *erb;
1121 struct revoked_serial rs, *ers;
1122 struct revoked_key_id rki, *erki;
1123 struct revoked_certs *rc;
1124
1125 /* Check explicitly revoked hashes first */
Damien Miller1d2c4562014-02-04 11:18:20 +11001126 memset(&rb, 0, sizeof(rb));
Damien Millerf3747bf2013-01-18 11:44:04 +11001127 if ((rb.blob = key_fingerprint_raw(key, SSH_FP_SHA1, &rb.len)) == NULL)
1128 return -1;
1129 erb = RB_FIND(revoked_blob_tree, &krl->revoked_sha1s, &rb);
1130 free(rb.blob);
1131 if (erb != NULL) {
1132 debug("%s: revoked by key SHA1", __func__);
1133 return -1;
1134 }
1135
1136 /* Next, explicit keys */
Damien Miller1d2c4562014-02-04 11:18:20 +11001137 memset(&rb, 0, sizeof(rb));
Damien Miller86687062014-07-02 15:28:02 +10001138 if (plain_key_blob(key, &rb.blob, &rb.len) < 0)
Damien Millerf3747bf2013-01-18 11:44:04 +11001139 return -1;
1140 erb = RB_FIND(revoked_blob_tree, &krl->revoked_keys, &rb);
1141 free(rb.blob);
1142 if (erb != NULL) {
1143 debug("%s: revoked by explicit key", __func__);
1144 return -1;
1145 }
1146
1147 if (!key_is_cert(key))
1148 return 0;
1149
1150 /* Check cert revocation */
1151 if (revoked_certs_for_ca_key(krl, key->cert->signature_key,
1152 &rc, 0) != 0)
1153 return -1;
1154 if (rc == NULL)
1155 return 0; /* No entry for this CA */
1156
1157 /* Check revocation by cert key ID */
Damien Miller1d2c4562014-02-04 11:18:20 +11001158 memset(&rki, 0, sizeof(rki));
Damien Millerf3747bf2013-01-18 11:44:04 +11001159 rki.key_id = key->cert->key_id;
1160 erki = RB_FIND(revoked_key_id_tree, &rc->revoked_key_ids, &rki);
1161 if (erki != NULL) {
1162 debug("%s: revoked by key ID", __func__);
1163 return -1;
1164 }
1165
Damien Miller60565bc2013-02-12 10:56:42 +11001166 /*
1167 * Legacy cert formats lack serial numbers. Zero serials numbers
1168 * are ignored (it's the default when the CA doesn't specify one).
1169 */
1170 if (key_cert_is_legacy(key) || key->cert->serial == 0)
Damien Millerf3747bf2013-01-18 11:44:04 +11001171 return 0;
1172
Damien Miller1d2c4562014-02-04 11:18:20 +11001173 memset(&rs, 0, sizeof(rs));
Damien Millerf3747bf2013-01-18 11:44:04 +11001174 rs.lo = rs.hi = key->cert->serial;
1175 ers = RB_FIND(revoked_serial_tree, &rc->revoked_serials, &rs);
1176 if (ers != NULL) {
1177 KRL_DBG(("%s: %llu matched %llu:%llu", __func__,
1178 key->cert->serial, ers->lo, ers->hi));
1179 debug("%s: revoked by serial", __func__);
1180 return -1;
1181 }
1182 KRL_DBG(("%s: %llu no match", __func__, key->cert->serial));
1183
1184 return 0;
1185}
1186
1187int
1188ssh_krl_check_key(struct ssh_krl *krl, const Key *key)
1189{
1190 int r;
1191
1192 debug2("%s: checking key", __func__);
1193 if ((r = is_key_revoked(krl, key)) != 0)
1194 return r;
1195 if (key_is_cert(key)) {
1196 debug2("%s: checking CA key", __func__);
1197 if ((r = is_key_revoked(krl, key->cert->signature_key)) != 0)
1198 return r;
1199 }
1200 debug3("%s: key okay", __func__);
1201 return 0;
1202}
1203
1204/* Returns 0 on success, -1 on error or key revoked, -2 if path is not a KRL */
1205int
1206ssh_krl_file_contains_key(const char *path, const Key *key)
1207{
1208 Buffer krlbuf;
1209 struct ssh_krl *krl;
1210 int revoked, fd;
1211
1212 if (path == NULL)
1213 return 0;
1214
1215 if ((fd = open(path, O_RDONLY)) == -1) {
1216 error("open %s: %s", path, strerror(errno));
1217 error("Revoked keys file not accessible - refusing public key "
1218 "authentication");
1219 return -1;
1220 }
1221 buffer_init(&krlbuf);
1222 if (!key_load_file(fd, path, &krlbuf)) {
1223 close(fd);
1224 buffer_free(&krlbuf);
1225 error("Revoked keys file not readable - refusing public key "
1226 "authentication");
1227 return -1;
1228 }
1229 close(fd);
1230 if (ssh_krl_from_blob(&krlbuf, &krl, NULL, 0) != 0) {
1231 buffer_free(&krlbuf);
1232 error("Invalid KRL, refusing public key "
1233 "authentication");
1234 return -1;
1235 }
1236 buffer_free(&krlbuf);
1237 if (krl == NULL) {
1238 debug3("%s: %s is not a KRL file", __func__, path);
1239 return -2;
1240 }
1241 debug2("%s: checking KRL %s", __func__, path);
1242 revoked = ssh_krl_check_key(krl, key) != 0;
1243 ssh_krl_free(krl);
1244 return revoked ? -1 : 0;
1245}