xref: /freebsd/crypto/openssh/hostfile.c (revision 4d3fc8b0)
14d3fc8b0SEd Maste /* $OpenBSD: hostfile.c,v 1.95 2023/02/21 06:48:18 dtucker Exp $ */
2511b41d2SMark Murray /*
3511b41d2SMark Murray  * Author: Tatu Ylonen <ylo@cs.hut.fi>
4511b41d2SMark Murray  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
5511b41d2SMark Murray  *                    All rights reserved
6511b41d2SMark Murray  * Functions for manipulating the known hosts files.
7511b41d2SMark Murray  *
8c2d3a559SKris Kennaway  * As far as I am concerned, the code I have written for this software
9c2d3a559SKris Kennaway  * can be used freely for any purpose.  Any derived versions of this
10c2d3a559SKris Kennaway  * software must be clearly marked as such, and if the derived work is
11c2d3a559SKris Kennaway  * incompatible with the protocol description in the RFC file, it must be
12c2d3a559SKris Kennaway  * called by a name other than "ssh" or "Secure Shell".
13c2d3a559SKris Kennaway  *
14c2d3a559SKris Kennaway  *
15c2d3a559SKris Kennaway  * Copyright (c) 1999, 2000 Markus Friedl.  All rights reserved.
16c2d3a559SKris Kennaway  * Copyright (c) 1999 Niels Provos.  All rights reserved.
17c2d3a559SKris Kennaway  *
18c2d3a559SKris Kennaway  * Redistribution and use in source and binary forms, with or without
19c2d3a559SKris Kennaway  * modification, are permitted provided that the following conditions
20c2d3a559SKris Kennaway  * are met:
21c2d3a559SKris Kennaway  * 1. Redistributions of source code must retain the above copyright
22c2d3a559SKris Kennaway  *    notice, this list of conditions and the following disclaimer.
23c2d3a559SKris Kennaway  * 2. Redistributions in binary form must reproduce the above copyright
24c2d3a559SKris Kennaway  *    notice, this list of conditions and the following disclaimer in the
25c2d3a559SKris Kennaway  *    documentation and/or other materials provided with the distribution.
26c2d3a559SKris Kennaway  *
27c2d3a559SKris Kennaway  * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
28c2d3a559SKris Kennaway  * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
29c2d3a559SKris Kennaway  * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
30c2d3a559SKris Kennaway  * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
31c2d3a559SKris Kennaway  * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
32c2d3a559SKris Kennaway  * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
33c2d3a559SKris Kennaway  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
34c2d3a559SKris Kennaway  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
35c2d3a559SKris Kennaway  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
36c2d3a559SKris Kennaway  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
37511b41d2SMark Murray  */
38511b41d2SMark Murray 
39511b41d2SMark Murray #include "includes.h"
40aa49c926SDag-Erling Smørgrav 
41333ee039SDag-Erling Smørgrav #include <sys/types.h>
42bc5531deSDag-Erling Smørgrav #include <sys/stat.h>
43333ee039SDag-Erling Smørgrav 
44333ee039SDag-Erling Smørgrav #include <netinet/in.h>
45333ee039SDag-Erling Smørgrav 
46bc5531deSDag-Erling Smørgrav #include <errno.h>
47333ee039SDag-Erling Smørgrav #include <resolv.h>
48333ee039SDag-Erling Smørgrav #include <stdarg.h>
49333ee039SDag-Erling Smørgrav #include <stdio.h>
50333ee039SDag-Erling Smørgrav #include <stdlib.h>
51333ee039SDag-Erling Smørgrav #include <string.h>
52bc5531deSDag-Erling Smørgrav #include <unistd.h>
53333ee039SDag-Erling Smørgrav 
54333ee039SDag-Erling Smørgrav #include "xmalloc.h"
55a8f6863aSKris Kennaway #include "match.h"
56bc5531deSDag-Erling Smørgrav #include "sshkey.h"
57a8f6863aSKris Kennaway #include "hostfile.h"
58ca3176e7SBrian Feldman #include "log.h"
594a421b63SDag-Erling Smørgrav #include "misc.h"
6019261079SEd Maste #include "pathnames.h"
61bc5531deSDag-Erling Smørgrav #include "ssherr.h"
62f7167e0eSDag-Erling Smørgrav #include "digest.h"
63b83788ffSDag-Erling Smørgrav #include "hmac.h"
6419261079SEd Maste #include "sshbuf.h"
65aa49c926SDag-Erling Smørgrav 
66bc5531deSDag-Erling Smørgrav /* XXX hmac is too easy to dictionary attack; use bcrypt? */
67bc5531deSDag-Erling Smørgrav 
68aa49c926SDag-Erling Smørgrav static int
extract_salt(const char * s,u_int l,u_char * salt,size_t salt_len)69e4a9863fSDag-Erling Smørgrav extract_salt(const char *s, u_int l, u_char *salt, size_t salt_len)
70aa49c926SDag-Erling Smørgrav {
71aa49c926SDag-Erling Smørgrav 	char *p, *b64salt;
72aa49c926SDag-Erling Smørgrav 	u_int b64len;
73aa49c926SDag-Erling Smørgrav 	int ret;
74aa49c926SDag-Erling Smørgrav 
75aa49c926SDag-Erling Smørgrav 	if (l < sizeof(HASH_MAGIC) - 1) {
76aa49c926SDag-Erling Smørgrav 		debug2("extract_salt: string too short");
77aa49c926SDag-Erling Smørgrav 		return (-1);
78aa49c926SDag-Erling Smørgrav 	}
79aa49c926SDag-Erling Smørgrav 	if (strncmp(s, HASH_MAGIC, sizeof(HASH_MAGIC) - 1) != 0) {
80aa49c926SDag-Erling Smørgrav 		debug2("extract_salt: invalid magic identifier");
81aa49c926SDag-Erling Smørgrav 		return (-1);
82aa49c926SDag-Erling Smørgrav 	}
83aa49c926SDag-Erling Smørgrav 	s += sizeof(HASH_MAGIC) - 1;
84aa49c926SDag-Erling Smørgrav 	l -= sizeof(HASH_MAGIC) - 1;
85aa49c926SDag-Erling Smørgrav 	if ((p = memchr(s, HASH_DELIM, l)) == NULL) {
86aa49c926SDag-Erling Smørgrav 		debug2("extract_salt: missing salt termination character");
87aa49c926SDag-Erling Smørgrav 		return (-1);
88aa49c926SDag-Erling Smørgrav 	}
89aa49c926SDag-Erling Smørgrav 
90aa49c926SDag-Erling Smørgrav 	b64len = p - s;
91aa49c926SDag-Erling Smørgrav 	/* Sanity check */
92aa49c926SDag-Erling Smørgrav 	if (b64len == 0 || b64len > 1024) {
93aa49c926SDag-Erling Smørgrav 		debug2("extract_salt: bad encoded salt length %u", b64len);
94aa49c926SDag-Erling Smørgrav 		return (-1);
95aa49c926SDag-Erling Smørgrav 	}
96aa49c926SDag-Erling Smørgrav 	b64salt = xmalloc(1 + b64len);
97aa49c926SDag-Erling Smørgrav 	memcpy(b64salt, s, b64len);
98aa49c926SDag-Erling Smørgrav 	b64salt[b64len] = '\0';
99aa49c926SDag-Erling Smørgrav 
100aa49c926SDag-Erling Smørgrav 	ret = __b64_pton(b64salt, salt, salt_len);
101e4a9863fSDag-Erling Smørgrav 	free(b64salt);
102aa49c926SDag-Erling Smørgrav 	if (ret == -1) {
103aa49c926SDag-Erling Smørgrav 		debug2("extract_salt: salt decode error");
104aa49c926SDag-Erling Smørgrav 		return (-1);
105aa49c926SDag-Erling Smørgrav 	}
106b83788ffSDag-Erling Smørgrav 	if (ret != (int)ssh_hmac_bytes(SSH_DIGEST_SHA1)) {
107b83788ffSDag-Erling Smørgrav 		debug2("extract_salt: expected salt len %zd, got %d",
108b83788ffSDag-Erling Smørgrav 		    ssh_hmac_bytes(SSH_DIGEST_SHA1), ret);
109aa49c926SDag-Erling Smørgrav 		return (-1);
110aa49c926SDag-Erling Smørgrav 	}
111aa49c926SDag-Erling Smørgrav 
112aa49c926SDag-Erling Smørgrav 	return (0);
113aa49c926SDag-Erling Smørgrav }
114aa49c926SDag-Erling Smørgrav 
115aa49c926SDag-Erling Smørgrav char *
host_hash(const char * host,const char * name_from_hostfile,u_int src_len)116aa49c926SDag-Erling Smørgrav host_hash(const char *host, const char *name_from_hostfile, u_int src_len)
117aa49c926SDag-Erling Smørgrav {
118b83788ffSDag-Erling Smørgrav 	struct ssh_hmac_ctx *ctx;
119e4a9863fSDag-Erling Smørgrav 	u_char salt[256], result[256];
120e4a9863fSDag-Erling Smørgrav 	char uu_salt[512], uu_result[512];
1211323ec57SEd Maste 	char *encoded = NULL;
122ca86bcf2SDag-Erling Smørgrav 	u_int len;
123aa49c926SDag-Erling Smørgrav 
124b83788ffSDag-Erling Smørgrav 	len = ssh_digest_bytes(SSH_DIGEST_SHA1);
125aa49c926SDag-Erling Smørgrav 
126aa49c926SDag-Erling Smørgrav 	if (name_from_hostfile == NULL) {
127aa49c926SDag-Erling Smørgrav 		/* Create new salt */
128ca86bcf2SDag-Erling Smørgrav 		arc4random_buf(salt, len);
129aa49c926SDag-Erling Smørgrav 	} else {
130aa49c926SDag-Erling Smørgrav 		/* Extract salt from known host entry */
131aa49c926SDag-Erling Smørgrav 		if (extract_salt(name_from_hostfile, src_len, salt,
132aa49c926SDag-Erling Smørgrav 		    sizeof(salt)) == -1)
133aa49c926SDag-Erling Smørgrav 			return (NULL);
134aa49c926SDag-Erling Smørgrav 	}
135aa49c926SDag-Erling Smørgrav 
136b83788ffSDag-Erling Smørgrav 	if ((ctx = ssh_hmac_start(SSH_DIGEST_SHA1)) == NULL ||
137b83788ffSDag-Erling Smørgrav 	    ssh_hmac_init(ctx, salt, len) < 0 ||
138b83788ffSDag-Erling Smørgrav 	    ssh_hmac_update(ctx, host, strlen(host)) < 0 ||
139b83788ffSDag-Erling Smørgrav 	    ssh_hmac_final(ctx, result, sizeof(result)))
14019261079SEd Maste 		fatal_f("ssh_hmac failed");
141b83788ffSDag-Erling Smørgrav 	ssh_hmac_free(ctx);
142aa49c926SDag-Erling Smørgrav 
143aa49c926SDag-Erling Smørgrav 	if (__b64_ntop(salt, len, uu_salt, sizeof(uu_salt)) == -1 ||
144aa49c926SDag-Erling Smørgrav 	    __b64_ntop(result, len, uu_result, sizeof(uu_result)) == -1)
14519261079SEd Maste 		fatal_f("__b64_ntop failed");
1461323ec57SEd Maste 	xasprintf(&encoded, "%s%s%c%s", HASH_MAGIC, uu_salt, HASH_DELIM,
1471323ec57SEd Maste 	    uu_result);
148aa49c926SDag-Erling Smørgrav 
149aa49c926SDag-Erling Smørgrav 	return (encoded);
150aa49c926SDag-Erling Smørgrav }
151511b41d2SMark Murray 
152511b41d2SMark Murray /*
153a8f6863aSKris Kennaway  * Parses an RSA (number of bits, e, n) or DSA key from a string.  Moves the
154a8f6863aSKris Kennaway  * pointer over the key.  Skips any whitespace at the beginning and at end.
155511b41d2SMark Murray  */
156511b41d2SMark Murray 
157511b41d2SMark Murray int
hostfile_read_key(char ** cpp,u_int * bitsp,struct sshkey * ret)158bc5531deSDag-Erling Smørgrav hostfile_read_key(char **cpp, u_int *bitsp, struct sshkey *ret)
159511b41d2SMark Murray {
160511b41d2SMark Murray 	char *cp;
161511b41d2SMark Murray 
162511b41d2SMark Murray 	/* Skip leading whitespace. */
163511b41d2SMark Murray 	for (cp = *cpp; *cp == ' ' || *cp == '\t'; cp++)
164511b41d2SMark Murray 		;
165511b41d2SMark Murray 
16619261079SEd Maste 	if (sshkey_read(ret, &cp) != 0)
167511b41d2SMark Murray 		return 0;
168511b41d2SMark Murray 
169511b41d2SMark Murray 	/* Skip trailing whitespace. */
170511b41d2SMark Murray 	for (; *cp == ' ' || *cp == '\t'; cp++)
171511b41d2SMark Murray 		;
172511b41d2SMark Murray 
173511b41d2SMark Murray 	/* Return results. */
174511b41d2SMark Murray 	*cpp = cp;
175bc5531deSDag-Erling Smørgrav 	if (bitsp != NULL)
176bc5531deSDag-Erling Smørgrav 		*bitsp = sshkey_size(ret);
177a8f6863aSKris Kennaway 	return 1;
178511b41d2SMark Murray }
179511b41d2SMark Murray 
1804a421b63SDag-Erling Smørgrav static HostkeyMarker
check_markers(char ** cpp)181b15c8340SDag-Erling Smørgrav check_markers(char **cpp)
182b15c8340SDag-Erling Smørgrav {
183b15c8340SDag-Erling Smørgrav 	char marker[32], *sp, *cp = *cpp;
184b15c8340SDag-Erling Smørgrav 	int ret = MRK_NONE;
185b15c8340SDag-Erling Smørgrav 
186b15c8340SDag-Erling Smørgrav 	while (*cp == '@') {
187b15c8340SDag-Erling Smørgrav 		/* Only one marker is allowed */
188b15c8340SDag-Erling Smørgrav 		if (ret != MRK_NONE)
189b15c8340SDag-Erling Smørgrav 			return MRK_ERROR;
190b15c8340SDag-Erling Smørgrav 		/* Markers are terminated by whitespace */
191b15c8340SDag-Erling Smørgrav 		if ((sp = strchr(cp, ' ')) == NULL &&
192b15c8340SDag-Erling Smørgrav 		    (sp = strchr(cp, '\t')) == NULL)
193b15c8340SDag-Erling Smørgrav 			return MRK_ERROR;
194b15c8340SDag-Erling Smørgrav 		/* Extract marker for comparison */
195b15c8340SDag-Erling Smørgrav 		if (sp <= cp + 1 || sp >= cp + sizeof(marker))
196b15c8340SDag-Erling Smørgrav 			return MRK_ERROR;
197b15c8340SDag-Erling Smørgrav 		memcpy(marker, cp, sp - cp);
198b15c8340SDag-Erling Smørgrav 		marker[sp - cp] = '\0';
199b15c8340SDag-Erling Smørgrav 		if (strcmp(marker, CA_MARKER) == 0)
200b15c8340SDag-Erling Smørgrav 			ret = MRK_CA;
201b15c8340SDag-Erling Smørgrav 		else if (strcmp(marker, REVOKE_MARKER) == 0)
202b15c8340SDag-Erling Smørgrav 			ret = MRK_REVOKE;
203b15c8340SDag-Erling Smørgrav 		else
204b15c8340SDag-Erling Smørgrav 			return MRK_ERROR;
205b15c8340SDag-Erling Smørgrav 
206b15c8340SDag-Erling Smørgrav 		/* Skip past marker and any whitespace that follows it */
207b15c8340SDag-Erling Smørgrav 		cp = sp;
208b15c8340SDag-Erling Smørgrav 		for (; *cp == ' ' || *cp == '\t'; cp++)
209b15c8340SDag-Erling Smørgrav 			;
210b15c8340SDag-Erling Smørgrav 	}
211b15c8340SDag-Erling Smørgrav 	*cpp = cp;
212b15c8340SDag-Erling Smørgrav 	return ret;
213b15c8340SDag-Erling Smørgrav }
214b15c8340SDag-Erling Smørgrav 
2154a421b63SDag-Erling Smørgrav struct hostkeys *
init_hostkeys(void)2164a421b63SDag-Erling Smørgrav init_hostkeys(void)
2174a421b63SDag-Erling Smørgrav {
2184a421b63SDag-Erling Smørgrav 	struct hostkeys *ret = xcalloc(1, sizeof(*ret));
219511b41d2SMark Murray 
2204a421b63SDag-Erling Smørgrav 	ret->entries = NULL;
2214a421b63SDag-Erling Smørgrav 	return ret;
2224a421b63SDag-Erling Smørgrav }
2234a421b63SDag-Erling Smørgrav 
224bc5531deSDag-Erling Smørgrav struct load_callback_ctx {
225bc5531deSDag-Erling Smørgrav 	const char *host;
226bc5531deSDag-Erling Smørgrav 	u_long num_loaded;
227bc5531deSDag-Erling Smørgrav 	struct hostkeys *hostkeys;
228bc5531deSDag-Erling Smørgrav };
229bc5531deSDag-Erling Smørgrav 
230bc5531deSDag-Erling Smørgrav static int
record_hostkey(struct hostkey_foreach_line * l,void * _ctx)231bc5531deSDag-Erling Smørgrav record_hostkey(struct hostkey_foreach_line *l, void *_ctx)
232bc5531deSDag-Erling Smørgrav {
233bc5531deSDag-Erling Smørgrav 	struct load_callback_ctx *ctx = (struct load_callback_ctx *)_ctx;
234bc5531deSDag-Erling Smørgrav 	struct hostkeys *hostkeys = ctx->hostkeys;
235bc5531deSDag-Erling Smørgrav 	struct hostkey_entry *tmp;
236bc5531deSDag-Erling Smørgrav 
237bc5531deSDag-Erling Smørgrav 	if (l->status == HKF_STATUS_INVALID) {
238557f75e5SDag-Erling Smørgrav 		/* XXX make this verbose() in the future */
239557f75e5SDag-Erling Smørgrav 		debug("%s:%ld: parse error in hostkeys file",
240bc5531deSDag-Erling Smørgrav 		    l->path, l->linenum);
241bc5531deSDag-Erling Smørgrav 		return 0;
242bc5531deSDag-Erling Smørgrav 	}
243bc5531deSDag-Erling Smørgrav 
24419261079SEd Maste 	debug3_f("found %skey type %s in file %s:%lu",
245bc5531deSDag-Erling Smørgrav 	    l->marker == MRK_NONE ? "" :
246bc5531deSDag-Erling Smørgrav 	    (l->marker == MRK_CA ? "ca " : "revoked "),
247bc5531deSDag-Erling Smørgrav 	    sshkey_type(l->key), l->path, l->linenum);
2484f52dfbbSDag-Erling Smørgrav 	if ((tmp = recallocarray(hostkeys->entries, hostkeys->num_entries,
249bc5531deSDag-Erling Smørgrav 	    hostkeys->num_entries + 1, sizeof(*hostkeys->entries))) == NULL)
250bc5531deSDag-Erling Smørgrav 		return SSH_ERR_ALLOC_FAIL;
251bc5531deSDag-Erling Smørgrav 	hostkeys->entries = tmp;
252bc5531deSDag-Erling Smørgrav 	hostkeys->entries[hostkeys->num_entries].host = xstrdup(ctx->host);
253bc5531deSDag-Erling Smørgrav 	hostkeys->entries[hostkeys->num_entries].file = xstrdup(l->path);
254bc5531deSDag-Erling Smørgrav 	hostkeys->entries[hostkeys->num_entries].line = l->linenum;
255bc5531deSDag-Erling Smørgrav 	hostkeys->entries[hostkeys->num_entries].key = l->key;
256bc5531deSDag-Erling Smørgrav 	l->key = NULL; /* steal it */
257bc5531deSDag-Erling Smørgrav 	hostkeys->entries[hostkeys->num_entries].marker = l->marker;
25819261079SEd Maste 	hostkeys->entries[hostkeys->num_entries].note = l->note;
259bc5531deSDag-Erling Smørgrav 	hostkeys->num_entries++;
260bc5531deSDag-Erling Smørgrav 	ctx->num_loaded++;
261bc5531deSDag-Erling Smørgrav 
262bc5531deSDag-Erling Smørgrav 	return 0;
263bc5531deSDag-Erling Smørgrav }
264bc5531deSDag-Erling Smørgrav 
2654a421b63SDag-Erling Smørgrav void
load_hostkeys_file(struct hostkeys * hostkeys,const char * host,const char * path,FILE * f,u_int note)26619261079SEd Maste load_hostkeys_file(struct hostkeys *hostkeys, const char *host,
26719261079SEd Maste     const char *path, FILE *f, u_int note)
268511b41d2SMark Murray {
269bc5531deSDag-Erling Smørgrav 	int r;
270bc5531deSDag-Erling Smørgrav 	struct load_callback_ctx ctx;
271511b41d2SMark Murray 
272bc5531deSDag-Erling Smørgrav 	ctx.host = host;
273bc5531deSDag-Erling Smørgrav 	ctx.num_loaded = 0;
274bc5531deSDag-Erling Smørgrav 	ctx.hostkeys = hostkeys;
275511b41d2SMark Murray 
27619261079SEd Maste 	if ((r = hostkeys_foreach_file(path, f, record_hostkey, &ctx, host,
27719261079SEd Maste 	    NULL, HKF_WANT_MATCH|HKF_WANT_PARSE_KEY, note)) != 0) {
278bc5531deSDag-Erling Smørgrav 		if (r != SSH_ERR_SYSTEM_ERROR && errno != ENOENT)
27919261079SEd Maste 			debug_fr(r, "hostkeys_foreach failed for %s", path);
2804a421b63SDag-Erling Smørgrav 	}
281bc5531deSDag-Erling Smørgrav 	if (ctx.num_loaded != 0)
28219261079SEd Maste 		debug3_f("loaded %lu keys from %s", ctx.num_loaded, host);
28319261079SEd Maste }
28419261079SEd Maste 
28519261079SEd Maste void
load_hostkeys(struct hostkeys * hostkeys,const char * host,const char * path,u_int note)28619261079SEd Maste load_hostkeys(struct hostkeys *hostkeys, const char *host, const char *path,
28719261079SEd Maste     u_int note)
28819261079SEd Maste {
28919261079SEd Maste 	FILE *f;
29019261079SEd Maste 
29119261079SEd Maste 	if ((f = fopen(path, "r")) == NULL) {
29219261079SEd Maste 		debug_f("fopen %s: %s", path, strerror(errno));
29319261079SEd Maste 		return;
29419261079SEd Maste 	}
29519261079SEd Maste 
29619261079SEd Maste 	load_hostkeys_file(hostkeys, host, path, f, note);
29719261079SEd Maste 	fclose(f);
298f388f5efSDag-Erling Smørgrav }
299f388f5efSDag-Erling Smørgrav 
3004a421b63SDag-Erling Smørgrav void
free_hostkeys(struct hostkeys * hostkeys)3014a421b63SDag-Erling Smørgrav free_hostkeys(struct hostkeys *hostkeys)
3024a421b63SDag-Erling Smørgrav {
3034a421b63SDag-Erling Smørgrav 	u_int i;
304f388f5efSDag-Erling Smørgrav 
3054a421b63SDag-Erling Smørgrav 	for (i = 0; i < hostkeys->num_entries; i++) {
306e4a9863fSDag-Erling Smørgrav 		free(hostkeys->entries[i].host);
307e4a9863fSDag-Erling Smørgrav 		free(hostkeys->entries[i].file);
308bc5531deSDag-Erling Smørgrav 		sshkey_free(hostkeys->entries[i].key);
309b83788ffSDag-Erling Smørgrav 		explicit_bzero(hostkeys->entries + i, sizeof(*hostkeys->entries));
310b15c8340SDag-Erling Smørgrav 	}
311e4a9863fSDag-Erling Smørgrav 	free(hostkeys->entries);
31219261079SEd Maste 	freezero(hostkeys, sizeof(*hostkeys));
313b15c8340SDag-Erling Smørgrav }
314b15c8340SDag-Erling Smørgrav 
3154a421b63SDag-Erling Smørgrav static int
check_key_not_revoked(struct hostkeys * hostkeys,struct sshkey * k)316bc5531deSDag-Erling Smørgrav check_key_not_revoked(struct hostkeys *hostkeys, struct sshkey *k)
3174a421b63SDag-Erling Smørgrav {
318bc5531deSDag-Erling Smørgrav 	int is_cert = sshkey_is_cert(k);
3194a421b63SDag-Erling Smørgrav 	u_int i;
3204a421b63SDag-Erling Smørgrav 
3214a421b63SDag-Erling Smørgrav 	for (i = 0; i < hostkeys->num_entries; i++) {
3224a421b63SDag-Erling Smørgrav 		if (hostkeys->entries[i].marker != MRK_REVOKE)
3234a421b63SDag-Erling Smørgrav 			continue;
324bc5531deSDag-Erling Smørgrav 		if (sshkey_equal_public(k, hostkeys->entries[i].key))
3254a421b63SDag-Erling Smørgrav 			return -1;
32619261079SEd Maste 		if (is_cert && k != NULL &&
327bc5531deSDag-Erling Smørgrav 		    sshkey_equal_public(k->cert->signature_key,
3284a421b63SDag-Erling Smørgrav 		    hostkeys->entries[i].key))
3294a421b63SDag-Erling Smørgrav 			return -1;
330511b41d2SMark Murray 	}
3314a421b63SDag-Erling Smørgrav 	return 0;
3324a421b63SDag-Erling Smørgrav }
3334a421b63SDag-Erling Smørgrav 
334511b41d2SMark Murray /*
3354a421b63SDag-Erling Smørgrav  * Match keys against a specified key, or look one up by key type.
3364a421b63SDag-Erling Smørgrav  *
3374a421b63SDag-Erling Smørgrav  * If looking for a keytype (key == NULL) and one is found then return
3384a421b63SDag-Erling Smørgrav  * HOST_FOUND, otherwise HOST_NEW.
3394a421b63SDag-Erling Smørgrav  *
3404a421b63SDag-Erling Smørgrav  * If looking for a key (key != NULL):
3414a421b63SDag-Erling Smørgrav  *  1. If the key is a cert and a matching CA is found, return HOST_OK
3424a421b63SDag-Erling Smørgrav  *  2. If the key is not a cert and a matching key is found, return HOST_OK
3434a421b63SDag-Erling Smørgrav  *  3. If no key matches but a key with a different type is found, then
3444a421b63SDag-Erling Smørgrav  *     return HOST_CHANGED
3454a421b63SDag-Erling Smørgrav  *  4. If no matching keys are found, then return HOST_NEW.
3464a421b63SDag-Erling Smørgrav  *
3474a421b63SDag-Erling Smørgrav  * Finally, check any found key is not revoked.
348511b41d2SMark Murray  */
3494a421b63SDag-Erling Smørgrav static HostStatus
check_hostkeys_by_key_or_type(struct hostkeys * hostkeys,struct sshkey * k,int keytype,int nid,const struct hostkey_entry ** found)3504a421b63SDag-Erling Smørgrav check_hostkeys_by_key_or_type(struct hostkeys *hostkeys,
35119261079SEd Maste     struct sshkey *k, int keytype, int nid, const struct hostkey_entry **found)
3524a421b63SDag-Erling Smørgrav {
3534a421b63SDag-Erling Smørgrav 	u_int i;
3544a421b63SDag-Erling Smørgrav 	HostStatus end_return = HOST_NEW;
355bc5531deSDag-Erling Smørgrav 	int want_cert = sshkey_is_cert(k);
3564a421b63SDag-Erling Smørgrav 	HostkeyMarker want_marker = want_cert ? MRK_CA : MRK_NONE;
3574a421b63SDag-Erling Smørgrav 
3584a421b63SDag-Erling Smørgrav 	if (found != NULL)
3594a421b63SDag-Erling Smørgrav 		*found = NULL;
3604a421b63SDag-Erling Smørgrav 
3614a421b63SDag-Erling Smørgrav 	for (i = 0; i < hostkeys->num_entries; i++) {
3624a421b63SDag-Erling Smørgrav 		if (hostkeys->entries[i].marker != want_marker)
3634a421b63SDag-Erling Smørgrav 			continue;
3644a421b63SDag-Erling Smørgrav 		if (k == NULL) {
3654a421b63SDag-Erling Smørgrav 			if (hostkeys->entries[i].key->type != keytype)
3664a421b63SDag-Erling Smørgrav 				continue;
36719261079SEd Maste 			if (nid != -1 &&
36819261079SEd Maste 			    sshkey_type_plain(keytype) == KEY_ECDSA &&
36919261079SEd Maste 			    hostkeys->entries[i].key->ecdsa_nid != nid)
37019261079SEd Maste 				continue;
3714a421b63SDag-Erling Smørgrav 			end_return = HOST_FOUND;
3724a421b63SDag-Erling Smørgrav 			if (found != NULL)
3734a421b63SDag-Erling Smørgrav 				*found = hostkeys->entries + i;
3744a421b63SDag-Erling Smørgrav 			k = hostkeys->entries[i].key;
3754a421b63SDag-Erling Smørgrav 			break;
3764a421b63SDag-Erling Smørgrav 		}
3774a421b63SDag-Erling Smørgrav 		if (want_cert) {
378bc5531deSDag-Erling Smørgrav 			if (sshkey_equal_public(k->cert->signature_key,
3794a421b63SDag-Erling Smørgrav 			    hostkeys->entries[i].key)) {
3804a421b63SDag-Erling Smørgrav 				/* A matching CA exists */
3814a421b63SDag-Erling Smørgrav 				end_return = HOST_OK;
3824a421b63SDag-Erling Smørgrav 				if (found != NULL)
3834a421b63SDag-Erling Smørgrav 					*found = hostkeys->entries + i;
3844a421b63SDag-Erling Smørgrav 				break;
3854a421b63SDag-Erling Smørgrav 			}
3864a421b63SDag-Erling Smørgrav 		} else {
387bc5531deSDag-Erling Smørgrav 			if (sshkey_equal(k, hostkeys->entries[i].key)) {
3884a421b63SDag-Erling Smørgrav 				end_return = HOST_OK;
3894a421b63SDag-Erling Smørgrav 				if (found != NULL)
3904a421b63SDag-Erling Smørgrav 					*found = hostkeys->entries + i;
3914a421b63SDag-Erling Smørgrav 				break;
3924a421b63SDag-Erling Smørgrav 			}
39319261079SEd Maste 			/* A non-matching key exists */
394511b41d2SMark Murray 			end_return = HOST_CHANGED;
3954a421b63SDag-Erling Smørgrav 			if (found != NULL)
3964a421b63SDag-Erling Smørgrav 				*found = hostkeys->entries + i;
397511b41d2SMark Murray 		}
3984a421b63SDag-Erling Smørgrav 	}
3994a421b63SDag-Erling Smørgrav 	if (check_key_not_revoked(hostkeys, k) != 0) {
4004a421b63SDag-Erling Smørgrav 		end_return = HOST_REVOKED;
4014a421b63SDag-Erling Smørgrav 		if (found != NULL)
4024a421b63SDag-Erling Smørgrav 			*found = NULL;
4034a421b63SDag-Erling Smørgrav 	}
404511b41d2SMark Murray 	return end_return;
405511b41d2SMark Murray }
406511b41d2SMark Murray 
407f388f5efSDag-Erling Smørgrav HostStatus
check_key_in_hostkeys(struct hostkeys * hostkeys,struct sshkey * key,const struct hostkey_entry ** found)408bc5531deSDag-Erling Smørgrav check_key_in_hostkeys(struct hostkeys *hostkeys, struct sshkey *key,
4094a421b63SDag-Erling Smørgrav     const struct hostkey_entry **found)
410f388f5efSDag-Erling Smørgrav {
411f388f5efSDag-Erling Smørgrav 	if (key == NULL)
412f388f5efSDag-Erling Smørgrav 		fatal("no key to look up");
41319261079SEd Maste 	return check_hostkeys_by_key_or_type(hostkeys, key, 0, -1, found);
414f388f5efSDag-Erling Smørgrav }
415f388f5efSDag-Erling Smørgrav 
416f388f5efSDag-Erling Smørgrav int
lookup_key_in_hostkeys_by_type(struct hostkeys * hostkeys,int keytype,int nid,const struct hostkey_entry ** found)41719261079SEd Maste lookup_key_in_hostkeys_by_type(struct hostkeys *hostkeys, int keytype, int nid,
4184a421b63SDag-Erling Smørgrav     const struct hostkey_entry **found)
419f388f5efSDag-Erling Smørgrav {
42019261079SEd Maste 	return (check_hostkeys_by_key_or_type(hostkeys, NULL, keytype, nid,
4214a421b63SDag-Erling Smørgrav 	    found) == HOST_FOUND);
422f388f5efSDag-Erling Smørgrav }
423f388f5efSDag-Erling Smørgrav 
42419261079SEd Maste int
lookup_marker_in_hostkeys(struct hostkeys * hostkeys,int want_marker)42519261079SEd Maste lookup_marker_in_hostkeys(struct hostkeys *hostkeys, int want_marker)
42619261079SEd Maste {
42719261079SEd Maste 	u_int i;
42819261079SEd Maste 
42919261079SEd Maste 	for (i = 0; i < hostkeys->num_entries; i++) {
43019261079SEd Maste 		if (hostkeys->entries[i].marker == (HostkeyMarker)want_marker)
43119261079SEd Maste 			return 1;
43219261079SEd Maste 	}
43319261079SEd Maste 	return 0;
43419261079SEd Maste }
43519261079SEd Maste 
436bc5531deSDag-Erling Smørgrav static int
write_host_entry(FILE * f,const char * host,const char * ip,const struct sshkey * key,int store_hash)437bc5531deSDag-Erling Smørgrav write_host_entry(FILE *f, const char *host, const char *ip,
438bc5531deSDag-Erling Smørgrav     const struct sshkey *key, int store_hash)
439bc5531deSDag-Erling Smørgrav {
440bc5531deSDag-Erling Smørgrav 	int r, success = 0;
441d93a896eSDag-Erling Smørgrav 	char *hashed_host = NULL, *lhost;
442d93a896eSDag-Erling Smørgrav 
443d93a896eSDag-Erling Smørgrav 	lhost = xstrdup(host);
444d93a896eSDag-Erling Smørgrav 	lowercase(lhost);
445bc5531deSDag-Erling Smørgrav 
446bc5531deSDag-Erling Smørgrav 	if (store_hash) {
447d93a896eSDag-Erling Smørgrav 		if ((hashed_host = host_hash(lhost, NULL, 0)) == NULL) {
44819261079SEd Maste 			error_f("host_hash failed");
449d93a896eSDag-Erling Smørgrav 			free(lhost);
450bc5531deSDag-Erling Smørgrav 			return 0;
451bc5531deSDag-Erling Smørgrav 		}
452bc5531deSDag-Erling Smørgrav 		fprintf(f, "%s ", hashed_host);
453bc5531deSDag-Erling Smørgrav 	} else if (ip != NULL)
454d93a896eSDag-Erling Smørgrav 		fprintf(f, "%s,%s ", lhost, ip);
455d93a896eSDag-Erling Smørgrav 	else {
456d93a896eSDag-Erling Smørgrav 		fprintf(f, "%s ", lhost);
457d93a896eSDag-Erling Smørgrav 	}
4581323ec57SEd Maste 	free(hashed_host);
459d93a896eSDag-Erling Smørgrav 	free(lhost);
460bc5531deSDag-Erling Smørgrav 	if ((r = sshkey_write(key, f)) == 0)
461bc5531deSDag-Erling Smørgrav 		success = 1;
462bc5531deSDag-Erling Smørgrav 	else
46319261079SEd Maste 		error_fr(r, "sshkey_write");
464bc5531deSDag-Erling Smørgrav 	fputc('\n', f);
46519261079SEd Maste 	/* If hashing is enabled, the IP address needs to go on its own line */
46619261079SEd Maste 	if (success && store_hash && ip != NULL)
46719261079SEd Maste 		success = write_host_entry(f, ip, NULL, key, 1);
468bc5531deSDag-Erling Smørgrav 	return success;
469bc5531deSDag-Erling Smørgrav }
470bc5531deSDag-Erling Smørgrav 
471511b41d2SMark Murray /*
47219261079SEd Maste  * Create user ~/.ssh directory if it doesn't exist and we want to write to it.
47319261079SEd Maste  * If notify is set, a message will be emitted if the directory is created.
47419261079SEd Maste  */
47519261079SEd Maste void
hostfile_create_user_ssh_dir(const char * filename,int notify)47619261079SEd Maste hostfile_create_user_ssh_dir(const char *filename, int notify)
47719261079SEd Maste {
47819261079SEd Maste 	char *dotsshdir = NULL, *p;
47919261079SEd Maste 	size_t len;
48019261079SEd Maste 	struct stat st;
48119261079SEd Maste 
48219261079SEd Maste 	if ((p = strrchr(filename, '/')) == NULL)
48319261079SEd Maste 		return;
48419261079SEd Maste 	len = p - filename;
48519261079SEd Maste 	dotsshdir = tilde_expand_filename("~/" _PATH_SSH_USER_DIR, getuid());
48619261079SEd Maste 	if (strlen(dotsshdir) > len || strncmp(filename, dotsshdir, len) != 0)
48719261079SEd Maste 		goto out; /* not ~/.ssh prefixed */
48819261079SEd Maste 	if (stat(dotsshdir, &st) == 0)
48919261079SEd Maste 		goto out; /* dir already exists */
49019261079SEd Maste 	else if (errno != ENOENT)
49119261079SEd Maste 		error("Could not stat %s: %s", dotsshdir, strerror(errno));
49219261079SEd Maste 	else {
49319261079SEd Maste #ifdef WITH_SELINUX
49419261079SEd Maste 		ssh_selinux_setfscreatecon(dotsshdir);
49519261079SEd Maste #endif
49619261079SEd Maste 		if (mkdir(dotsshdir, 0700) == -1)
49719261079SEd Maste 			error("Could not create directory '%.200s' (%s).",
49819261079SEd Maste 			    dotsshdir, strerror(errno));
49919261079SEd Maste 		else if (notify)
50019261079SEd Maste 			logit("Created directory '%s'.", dotsshdir);
50119261079SEd Maste #ifdef WITH_SELINUX
50219261079SEd Maste 		ssh_selinux_setfscreatecon(NULL);
50319261079SEd Maste #endif
50419261079SEd Maste 	}
50519261079SEd Maste  out:
50619261079SEd Maste 	free(dotsshdir);
50719261079SEd Maste }
50819261079SEd Maste 
50919261079SEd Maste /*
510511b41d2SMark Murray  * Appends an entry to the host file.  Returns false if the entry could not
511511b41d2SMark Murray  * be appended.
512511b41d2SMark Murray  */
513511b41d2SMark Murray int
add_host_to_hostfile(const char * filename,const char * host,const struct sshkey * key,int store_hash)514bc5531deSDag-Erling Smørgrav add_host_to_hostfile(const char *filename, const char *host,
515bc5531deSDag-Erling Smørgrav     const struct sshkey *key, int store_hash)
516511b41d2SMark Murray {
517511b41d2SMark Murray 	FILE *f;
5184d3fc8b0SEd Maste 	int success, addnl = 0;
519aa49c926SDag-Erling Smørgrav 
520a8f6863aSKris Kennaway 	if (key == NULL)
521e8aafc91SKris Kennaway 		return 1;	/* XXX ? */
52219261079SEd Maste 	hostfile_create_user_ssh_dir(filename, 0);
5234d3fc8b0SEd Maste 	f = fopen(filename, "a+");
524511b41d2SMark Murray 	if (!f)
525511b41d2SMark Murray 		return 0;
5264d3fc8b0SEd Maste 	/* Make sure we have a terminating newline. */
5274d3fc8b0SEd Maste 	if (fseek(f, -1L, SEEK_END) == 0 && fgetc(f) != '\n')
5284d3fc8b0SEd Maste 		addnl = 1;
5294d3fc8b0SEd Maste 	if (fseek(f, 0L, SEEK_END) != 0 || (addnl && fputc('\n', f) != '\n')) {
5304d3fc8b0SEd Maste 		error("Failed to add terminating newline to %s: %s",
5314d3fc8b0SEd Maste 		   filename, strerror(errno));
5324d3fc8b0SEd Maste 		fclose(f);
5334d3fc8b0SEd Maste 		return 0;
5344d3fc8b0SEd Maste 	}
535bc5531deSDag-Erling Smørgrav 	success = write_host_entry(f, host, NULL, key, store_hash);
536aa49c926SDag-Erling Smørgrav 	fclose(f);
537bc5531deSDag-Erling Smørgrav 	return success;
538bc5531deSDag-Erling Smørgrav }
539bc5531deSDag-Erling Smørgrav 
540bc5531deSDag-Erling Smørgrav struct host_delete_ctx {
541bc5531deSDag-Erling Smørgrav 	FILE *out;
542bc5531deSDag-Erling Smørgrav 	int quiet;
54319261079SEd Maste 	const char *host, *ip;
54419261079SEd Maste 	u_int *match_keys;	/* mask of HKF_MATCH_* for this key */
545bc5531deSDag-Erling Smørgrav 	struct sshkey * const *keys;
546bc5531deSDag-Erling Smørgrav 	size_t nkeys;
547bc5531deSDag-Erling Smørgrav 	int modified;
548bc5531deSDag-Erling Smørgrav };
549bc5531deSDag-Erling Smørgrav 
550bc5531deSDag-Erling Smørgrav static int
host_delete(struct hostkey_foreach_line * l,void * _ctx)551bc5531deSDag-Erling Smørgrav host_delete(struct hostkey_foreach_line *l, void *_ctx)
552bc5531deSDag-Erling Smørgrav {
553bc5531deSDag-Erling Smørgrav 	struct host_delete_ctx *ctx = (struct host_delete_ctx *)_ctx;
554bc5531deSDag-Erling Smørgrav 	int loglevel = ctx->quiet ? SYSLOG_LEVEL_DEBUG1 : SYSLOG_LEVEL_VERBOSE;
555bc5531deSDag-Erling Smørgrav 	size_t i;
556bc5531deSDag-Erling Smørgrav 
557bc5531deSDag-Erling Smørgrav 	/* Don't remove CA and revocation lines */
55819261079SEd Maste 	if (l->status == HKF_STATUS_MATCHED && l->marker == MRK_NONE) {
559bc5531deSDag-Erling Smørgrav 		/*
560bc5531deSDag-Erling Smørgrav 		 * If this line contains one of the keys that we will be
561bc5531deSDag-Erling Smørgrav 		 * adding later, then don't change it and mark the key for
562bc5531deSDag-Erling Smørgrav 		 * skipping.
563bc5531deSDag-Erling Smørgrav 		 */
564bc5531deSDag-Erling Smørgrav 		for (i = 0; i < ctx->nkeys; i++) {
56519261079SEd Maste 			if (!sshkey_equal(ctx->keys[i], l->key))
56619261079SEd Maste 				continue;
56719261079SEd Maste 			ctx->match_keys[i] |= l->match;
568bc5531deSDag-Erling Smørgrav 			fprintf(ctx->out, "%s\n", l->line);
56919261079SEd Maste 			debug3_f("%s key already at %s:%ld",
570bc5531deSDag-Erling Smørgrav 			    sshkey_type(l->key), l->path, l->linenum);
571aa49c926SDag-Erling Smørgrav 			return 0;
572aa49c926SDag-Erling Smørgrav 		}
573aa49c926SDag-Erling Smørgrav 
574bc5531deSDag-Erling Smørgrav 		/*
575bc5531deSDag-Erling Smørgrav 		 * Hostname matches and has no CA/revoke marker, delete it
576bc5531deSDag-Erling Smørgrav 		 * by *not* writing the line to ctx->out.
577bc5531deSDag-Erling Smørgrav 		 */
578bc5531deSDag-Erling Smørgrav 		do_log2(loglevel, "%s%s%s:%ld: Removed %s key for host %s",
579bc5531deSDag-Erling Smørgrav 		    ctx->quiet ? __func__ : "", ctx->quiet ? ": " : "",
580bc5531deSDag-Erling Smørgrav 		    l->path, l->linenum, sshkey_type(l->key), ctx->host);
581bc5531deSDag-Erling Smørgrav 		ctx->modified = 1;
582bc5531deSDag-Erling Smørgrav 		return 0;
583511b41d2SMark Murray 	}
584bc5531deSDag-Erling Smørgrav 	/* Retain non-matching hosts and invalid lines when deleting */
585bc5531deSDag-Erling Smørgrav 	if (l->status == HKF_STATUS_INVALID) {
586bc5531deSDag-Erling Smørgrav 		do_log2(loglevel, "%s%s%s:%ld: invalid known_hosts entry",
587bc5531deSDag-Erling Smørgrav 		    ctx->quiet ? __func__ : "", ctx->quiet ? ": " : "",
588bc5531deSDag-Erling Smørgrav 		    l->path, l->linenum);
589bc5531deSDag-Erling Smørgrav 	}
590bc5531deSDag-Erling Smørgrav 	fprintf(ctx->out, "%s\n", l->line);
591bc5531deSDag-Erling Smørgrav 	return 0;
592bc5531deSDag-Erling Smørgrav }
593bc5531deSDag-Erling Smørgrav 
594bc5531deSDag-Erling Smørgrav int
hostfile_replace_entries(const char * filename,const char * host,const char * ip,struct sshkey ** keys,size_t nkeys,int store_hash,int quiet,int hash_alg)595bc5531deSDag-Erling Smørgrav hostfile_replace_entries(const char *filename, const char *host, const char *ip,
596bc5531deSDag-Erling Smørgrav     struct sshkey **keys, size_t nkeys, int store_hash, int quiet, int hash_alg)
597bc5531deSDag-Erling Smørgrav {
598bc5531deSDag-Erling Smørgrav 	int r, fd, oerrno = 0;
599bc5531deSDag-Erling Smørgrav 	int loglevel = quiet ? SYSLOG_LEVEL_DEBUG1 : SYSLOG_LEVEL_VERBOSE;
600bc5531deSDag-Erling Smørgrav 	struct host_delete_ctx ctx;
601bc5531deSDag-Erling Smørgrav 	char *fp, *temp = NULL, *back = NULL;
60219261079SEd Maste 	const char *what;
603bc5531deSDag-Erling Smørgrav 	mode_t omask;
604bc5531deSDag-Erling Smørgrav 	size_t i;
60519261079SEd Maste 	u_int want;
606bc5531deSDag-Erling Smørgrav 
607bc5531deSDag-Erling Smørgrav 	omask = umask(077);
608bc5531deSDag-Erling Smørgrav 
609bc5531deSDag-Erling Smørgrav 	memset(&ctx, 0, sizeof(ctx));
610bc5531deSDag-Erling Smørgrav 	ctx.host = host;
61119261079SEd Maste 	ctx.ip = ip;
612bc5531deSDag-Erling Smørgrav 	ctx.quiet = quiet;
61319261079SEd Maste 
61419261079SEd Maste 	if ((ctx.match_keys = calloc(nkeys, sizeof(*ctx.match_keys))) == NULL)
615bc5531deSDag-Erling Smørgrav 		return SSH_ERR_ALLOC_FAIL;
616bc5531deSDag-Erling Smørgrav 	ctx.keys = keys;
617bc5531deSDag-Erling Smørgrav 	ctx.nkeys = nkeys;
618bc5531deSDag-Erling Smørgrav 	ctx.modified = 0;
619bc5531deSDag-Erling Smørgrav 
620bc5531deSDag-Erling Smørgrav 	/*
621bc5531deSDag-Erling Smørgrav 	 * Prepare temporary file for in-place deletion.
622bc5531deSDag-Erling Smørgrav 	 */
62319261079SEd Maste 	if ((r = asprintf(&temp, "%s.XXXXXXXXXXX", filename)) == -1 ||
62419261079SEd Maste 	    (r = asprintf(&back, "%s.old", filename)) == -1) {
625bc5531deSDag-Erling Smørgrav 		r = SSH_ERR_ALLOC_FAIL;
626bc5531deSDag-Erling Smørgrav 		goto fail;
627bc5531deSDag-Erling Smørgrav 	}
628bc5531deSDag-Erling Smørgrav 
629bc5531deSDag-Erling Smørgrav 	if ((fd = mkstemp(temp)) == -1) {
630bc5531deSDag-Erling Smørgrav 		oerrno = errno;
63119261079SEd Maste 		error_f("mkstemp: %s", strerror(oerrno));
632bc5531deSDag-Erling Smørgrav 		r = SSH_ERR_SYSTEM_ERROR;
633bc5531deSDag-Erling Smørgrav 		goto fail;
634bc5531deSDag-Erling Smørgrav 	}
635bc5531deSDag-Erling Smørgrav 	if ((ctx.out = fdopen(fd, "w")) == NULL) {
636bc5531deSDag-Erling Smørgrav 		oerrno = errno;
637bc5531deSDag-Erling Smørgrav 		close(fd);
63819261079SEd Maste 		error_f("fdopen: %s", strerror(oerrno));
639bc5531deSDag-Erling Smørgrav 		r = SSH_ERR_SYSTEM_ERROR;
640bc5531deSDag-Erling Smørgrav 		goto fail;
641bc5531deSDag-Erling Smørgrav 	}
642bc5531deSDag-Erling Smørgrav 
64319261079SEd Maste 	/* Remove stale/mismatching entries for the specified host */
644bc5531deSDag-Erling Smørgrav 	if ((r = hostkeys_foreach(filename, host_delete, &ctx, host, ip,
64519261079SEd Maste 	    HKF_WANT_PARSE_KEY, 0)) != 0) {
64619261079SEd Maste 		oerrno = errno;
64719261079SEd Maste 		error_fr(r, "hostkeys_foreach");
648bc5531deSDag-Erling Smørgrav 		goto fail;
649bc5531deSDag-Erling Smørgrav 	}
650bc5531deSDag-Erling Smørgrav 
65119261079SEd Maste 	/* Re-add the requested keys */
65219261079SEd Maste 	want = HKF_MATCH_HOST | (ip == NULL ? 0 : HKF_MATCH_IP);
653bc5531deSDag-Erling Smørgrav 	for (i = 0; i < nkeys; i++) {
6541323ec57SEd Maste 		if (keys[i] == NULL || (want & ctx.match_keys[i]) == want)
655bc5531deSDag-Erling Smørgrav 			continue;
656bc5531deSDag-Erling Smørgrav 		if ((fp = sshkey_fingerprint(keys[i], hash_alg,
657bc5531deSDag-Erling Smørgrav 		    SSH_FP_DEFAULT)) == NULL) {
658bc5531deSDag-Erling Smørgrav 			r = SSH_ERR_ALLOC_FAIL;
659bc5531deSDag-Erling Smørgrav 			goto fail;
660bc5531deSDag-Erling Smørgrav 		}
66119261079SEd Maste 		/* write host/ip */
66219261079SEd Maste 		what = "";
66319261079SEd Maste 		if (ctx.match_keys[i] == 0) {
66419261079SEd Maste 			what = "Adding new key";
66519261079SEd Maste 			if (!write_host_entry(ctx.out, host, ip,
66619261079SEd Maste 			    keys[i], store_hash)) {
667bc5531deSDag-Erling Smørgrav 				r = SSH_ERR_INTERNAL_ERROR;
668bc5531deSDag-Erling Smørgrav 				goto fail;
669bc5531deSDag-Erling Smørgrav 			}
67019261079SEd Maste 		} else if ((want & ~ctx.match_keys[i]) == HKF_MATCH_HOST) {
67119261079SEd Maste 			what = "Fixing match (hostname)";
67219261079SEd Maste 			if (!write_host_entry(ctx.out, host, NULL,
67319261079SEd Maste 			    keys[i], store_hash)) {
67419261079SEd Maste 				r = SSH_ERR_INTERNAL_ERROR;
67519261079SEd Maste 				goto fail;
67619261079SEd Maste 			}
67719261079SEd Maste 		} else if ((want & ~ctx.match_keys[i]) == HKF_MATCH_IP) {
67819261079SEd Maste 			what = "Fixing match (address)";
67919261079SEd Maste 			if (!write_host_entry(ctx.out, ip, NULL,
68019261079SEd Maste 			    keys[i], store_hash)) {
68119261079SEd Maste 				r = SSH_ERR_INTERNAL_ERROR;
68219261079SEd Maste 				goto fail;
68319261079SEd Maste 			}
68419261079SEd Maste 		}
68519261079SEd Maste 		do_log2(loglevel, "%s%s%s for %s%s%s to %s: %s %s",
68619261079SEd Maste 		    quiet ? __func__ : "", quiet ? ": " : "", what,
68719261079SEd Maste 		    host, ip == NULL ? "" : ",", ip == NULL ? "" : ip, filename,
68819261079SEd Maste 		    sshkey_ssh_name(keys[i]), fp);
68919261079SEd Maste 		free(fp);
690bc5531deSDag-Erling Smørgrav 		ctx.modified = 1;
691bc5531deSDag-Erling Smørgrav 	}
692bc5531deSDag-Erling Smørgrav 	fclose(ctx.out);
693bc5531deSDag-Erling Smørgrav 	ctx.out = NULL;
694bc5531deSDag-Erling Smørgrav 
695bc5531deSDag-Erling Smørgrav 	if (ctx.modified) {
696bc5531deSDag-Erling Smørgrav 		/* Backup the original file and replace it with the temporary */
697bc5531deSDag-Erling Smørgrav 		if (unlink(back) == -1 && errno != ENOENT) {
698bc5531deSDag-Erling Smørgrav 			oerrno = errno;
69919261079SEd Maste 			error_f("unlink %.100s: %s", back, strerror(errno));
700bc5531deSDag-Erling Smørgrav 			r = SSH_ERR_SYSTEM_ERROR;
701bc5531deSDag-Erling Smørgrav 			goto fail;
702bc5531deSDag-Erling Smørgrav 		}
703bc5531deSDag-Erling Smørgrav 		if (link(filename, back) == -1) {
704bc5531deSDag-Erling Smørgrav 			oerrno = errno;
70519261079SEd Maste 			error_f("link %.100s to %.100s: %s", filename,
70619261079SEd Maste 			    back, strerror(errno));
707bc5531deSDag-Erling Smørgrav 			r = SSH_ERR_SYSTEM_ERROR;
708bc5531deSDag-Erling Smørgrav 			goto fail;
709bc5531deSDag-Erling Smørgrav 		}
710bc5531deSDag-Erling Smørgrav 		if (rename(temp, filename) == -1) {
711bc5531deSDag-Erling Smørgrav 			oerrno = errno;
71219261079SEd Maste 			error_f("rename \"%s\" to \"%s\": %s", temp,
71319261079SEd Maste 			    filename, strerror(errno));
714bc5531deSDag-Erling Smørgrav 			r = SSH_ERR_SYSTEM_ERROR;
715bc5531deSDag-Erling Smørgrav 			goto fail;
716bc5531deSDag-Erling Smørgrav 		}
717bc5531deSDag-Erling Smørgrav 	} else {
718bc5531deSDag-Erling Smørgrav 		/* No changes made; just delete the temporary file */
719bc5531deSDag-Erling Smørgrav 		if (unlink(temp) != 0)
72019261079SEd Maste 			error_f("unlink \"%s\": %s", temp, strerror(errno));
721bc5531deSDag-Erling Smørgrav 	}
722bc5531deSDag-Erling Smørgrav 
723bc5531deSDag-Erling Smørgrav 	/* success */
724bc5531deSDag-Erling Smørgrav 	r = 0;
725bc5531deSDag-Erling Smørgrav  fail:
726bc5531deSDag-Erling Smørgrav 	if (temp != NULL && r != 0)
727bc5531deSDag-Erling Smørgrav 		unlink(temp);
728bc5531deSDag-Erling Smørgrav 	free(temp);
729bc5531deSDag-Erling Smørgrav 	free(back);
730bc5531deSDag-Erling Smørgrav 	if (ctx.out != NULL)
731bc5531deSDag-Erling Smørgrav 		fclose(ctx.out);
73219261079SEd Maste 	free(ctx.match_keys);
733bc5531deSDag-Erling Smørgrav 	umask(omask);
734bc5531deSDag-Erling Smørgrav 	if (r == SSH_ERR_SYSTEM_ERROR)
735bc5531deSDag-Erling Smørgrav 		errno = oerrno;
736bc5531deSDag-Erling Smørgrav 	return r;
737bc5531deSDag-Erling Smørgrav }
738bc5531deSDag-Erling Smørgrav 
739bc5531deSDag-Erling Smørgrav static int
match_maybe_hashed(const char * host,const char * names,int * was_hashed)740bc5531deSDag-Erling Smørgrav match_maybe_hashed(const char *host, const char *names, int *was_hashed)
741bc5531deSDag-Erling Smørgrav {
7421323ec57SEd Maste 	int hashed = *names == HASH_DELIM, ret;
7431323ec57SEd Maste 	char *hashed_host = NULL;
744bc5531deSDag-Erling Smørgrav 	size_t nlen = strlen(names);
745bc5531deSDag-Erling Smørgrav 
746bc5531deSDag-Erling Smørgrav 	if (was_hashed != NULL)
747bc5531deSDag-Erling Smørgrav 		*was_hashed = hashed;
748bc5531deSDag-Erling Smørgrav 	if (hashed) {
749bc5531deSDag-Erling Smørgrav 		if ((hashed_host = host_hash(host, names, nlen)) == NULL)
750bc5531deSDag-Erling Smørgrav 			return -1;
7511323ec57SEd Maste 		ret = (nlen == strlen(hashed_host) &&
7521323ec57SEd Maste 		    strncmp(hashed_host, names, nlen) == 0);
7531323ec57SEd Maste 		free(hashed_host);
7541323ec57SEd Maste 		return ret;
755bc5531deSDag-Erling Smørgrav 	}
756557f75e5SDag-Erling Smørgrav 	return match_hostname(host, names) == 1;
757bc5531deSDag-Erling Smørgrav }
758bc5531deSDag-Erling Smørgrav 
759bc5531deSDag-Erling Smørgrav int
hostkeys_foreach_file(const char * path,FILE * f,hostkeys_foreach_fn * callback,void * ctx,const char * host,const char * ip,u_int options,u_int note)76019261079SEd Maste hostkeys_foreach_file(const char *path, FILE *f, hostkeys_foreach_fn *callback,
76119261079SEd Maste     void *ctx, const char *host, const char *ip, u_int options, u_int note)
762bc5531deSDag-Erling Smørgrav {
763190cef3dSDag-Erling Smørgrav 	char *line = NULL, ktype[128];
764bc5531deSDag-Erling Smørgrav 	u_long linenum = 0;
765bc5531deSDag-Erling Smørgrav 	char *cp, *cp2;
766bc5531deSDag-Erling Smørgrav 	u_int kbits;
767bc5531deSDag-Erling Smørgrav 	int hashed;
768bc5531deSDag-Erling Smørgrav 	int s, r = 0;
769bc5531deSDag-Erling Smørgrav 	struct hostkey_foreach_line lineinfo;
770190cef3dSDag-Erling Smørgrav 	size_t linesize = 0, l;
771bc5531deSDag-Erling Smørgrav 
772bc5531deSDag-Erling Smørgrav 	memset(&lineinfo, 0, sizeof(lineinfo));
773bc5531deSDag-Erling Smørgrav 	if (host == NULL && (options & HKF_WANT_MATCH) != 0)
774bc5531deSDag-Erling Smørgrav 		return SSH_ERR_INVALID_ARGUMENT;
775bc5531deSDag-Erling Smørgrav 
776190cef3dSDag-Erling Smørgrav 	while (getline(&line, &linesize, f) != -1) {
777190cef3dSDag-Erling Smørgrav 		linenum++;
778bc5531deSDag-Erling Smørgrav 		line[strcspn(line, "\n")] = '\0';
779bc5531deSDag-Erling Smørgrav 
780190cef3dSDag-Erling Smørgrav 		free(lineinfo.line);
781bc5531deSDag-Erling Smørgrav 		sshkey_free(lineinfo.key);
782bc5531deSDag-Erling Smørgrav 		memset(&lineinfo, 0, sizeof(lineinfo));
783bc5531deSDag-Erling Smørgrav 		lineinfo.path = path;
784bc5531deSDag-Erling Smørgrav 		lineinfo.linenum = linenum;
785190cef3dSDag-Erling Smørgrav 		lineinfo.line = xstrdup(line);
786bc5531deSDag-Erling Smørgrav 		lineinfo.marker = MRK_NONE;
787bc5531deSDag-Erling Smørgrav 		lineinfo.status = HKF_STATUS_OK;
788bc5531deSDag-Erling Smørgrav 		lineinfo.keytype = KEY_UNSPEC;
78919261079SEd Maste 		lineinfo.note = note;
790bc5531deSDag-Erling Smørgrav 
791bc5531deSDag-Erling Smørgrav 		/* Skip any leading whitespace, comments and empty lines. */
792bc5531deSDag-Erling Smørgrav 		for (cp = line; *cp == ' ' || *cp == '\t'; cp++)
793bc5531deSDag-Erling Smørgrav 			;
794bc5531deSDag-Erling Smørgrav 		if (!*cp || *cp == '#' || *cp == '\n') {
795bc5531deSDag-Erling Smørgrav 			if ((options & HKF_WANT_MATCH) == 0) {
796bc5531deSDag-Erling Smørgrav 				lineinfo.status = HKF_STATUS_COMMENT;
797bc5531deSDag-Erling Smørgrav 				if ((r = callback(&lineinfo, ctx)) != 0)
798bc5531deSDag-Erling Smørgrav 					break;
799bc5531deSDag-Erling Smørgrav 			}
800bc5531deSDag-Erling Smørgrav 			continue;
801bc5531deSDag-Erling Smørgrav 		}
802bc5531deSDag-Erling Smørgrav 
803bc5531deSDag-Erling Smørgrav 		if ((lineinfo.marker = check_markers(&cp)) == MRK_ERROR) {
80419261079SEd Maste 			verbose_f("invalid marker at %s:%lu", path, linenum);
805bc5531deSDag-Erling Smørgrav 			if ((options & HKF_WANT_MATCH) == 0)
806bc5531deSDag-Erling Smørgrav 				goto bad;
807bc5531deSDag-Erling Smørgrav 			continue;
808bc5531deSDag-Erling Smørgrav 		}
809bc5531deSDag-Erling Smørgrav 
810bc5531deSDag-Erling Smørgrav 		/* Find the end of the host name portion. */
811bc5531deSDag-Erling Smørgrav 		for (cp2 = cp; *cp2 && *cp2 != ' ' && *cp2 != '\t'; cp2++)
812bc5531deSDag-Erling Smørgrav 			;
813bc5531deSDag-Erling Smørgrav 		lineinfo.hosts = cp;
814bc5531deSDag-Erling Smørgrav 		*cp2++ = '\0';
815bc5531deSDag-Erling Smørgrav 
816bc5531deSDag-Erling Smørgrav 		/* Check if the host name matches. */
817bc5531deSDag-Erling Smørgrav 		if (host != NULL) {
818bc5531deSDag-Erling Smørgrav 			if ((s = match_maybe_hashed(host, lineinfo.hosts,
819bc5531deSDag-Erling Smørgrav 			    &hashed)) == -1) {
82019261079SEd Maste 				debug2_f("%s:%ld: bad host hash \"%.32s\"",
82119261079SEd Maste 				    path, linenum, lineinfo.hosts);
822bc5531deSDag-Erling Smørgrav 				goto bad;
823bc5531deSDag-Erling Smørgrav 			}
824bc5531deSDag-Erling Smørgrav 			if (s == 1) {
825bc5531deSDag-Erling Smørgrav 				lineinfo.status = HKF_STATUS_MATCHED;
826bc5531deSDag-Erling Smørgrav 				lineinfo.match |= HKF_MATCH_HOST |
827bc5531deSDag-Erling Smørgrav 				    (hashed ? HKF_MATCH_HOST_HASHED : 0);
828bc5531deSDag-Erling Smørgrav 			}
829bc5531deSDag-Erling Smørgrav 			/* Try matching IP address if supplied */
830bc5531deSDag-Erling Smørgrav 			if (ip != NULL) {
831bc5531deSDag-Erling Smørgrav 				if ((s = match_maybe_hashed(ip, lineinfo.hosts,
832bc5531deSDag-Erling Smørgrav 				    &hashed)) == -1) {
83319261079SEd Maste 					debug2_f("%s:%ld: bad ip hash "
83419261079SEd Maste 					    "\"%.32s\"", path, linenum,
83519261079SEd Maste 					    lineinfo.hosts);
836bc5531deSDag-Erling Smørgrav 					goto bad;
837bc5531deSDag-Erling Smørgrav 				}
838bc5531deSDag-Erling Smørgrav 				if (s == 1) {
839bc5531deSDag-Erling Smørgrav 					lineinfo.status = HKF_STATUS_MATCHED;
840bc5531deSDag-Erling Smørgrav 					lineinfo.match |= HKF_MATCH_IP |
841bc5531deSDag-Erling Smørgrav 					    (hashed ? HKF_MATCH_IP_HASHED : 0);
842bc5531deSDag-Erling Smørgrav 				}
843bc5531deSDag-Erling Smørgrav 			}
844bc5531deSDag-Erling Smørgrav 			/*
845bc5531deSDag-Erling Smørgrav 			 * Skip this line if host matching requested and
846bc5531deSDag-Erling Smørgrav 			 * neither host nor address matched.
847bc5531deSDag-Erling Smørgrav 			 */
848bc5531deSDag-Erling Smørgrav 			if ((options & HKF_WANT_MATCH) != 0 &&
849bc5531deSDag-Erling Smørgrav 			    lineinfo.status != HKF_STATUS_MATCHED)
850bc5531deSDag-Erling Smørgrav 				continue;
851bc5531deSDag-Erling Smørgrav 		}
852bc5531deSDag-Erling Smørgrav 
853bc5531deSDag-Erling Smørgrav 		/* Got a match.  Skip host name and any following whitespace */
854bc5531deSDag-Erling Smørgrav 		for (; *cp2 == ' ' || *cp2 == '\t'; cp2++)
855bc5531deSDag-Erling Smørgrav 			;
856bc5531deSDag-Erling Smørgrav 		if (*cp2 == '\0' || *cp2 == '#') {
857bc5531deSDag-Erling Smørgrav 			debug2("%s:%ld: truncated before key type",
858bc5531deSDag-Erling Smørgrav 			    path, linenum);
859bc5531deSDag-Erling Smørgrav 			goto bad;
860bc5531deSDag-Erling Smørgrav 		}
861bc5531deSDag-Erling Smørgrav 		lineinfo.rawkey = cp = cp2;
862bc5531deSDag-Erling Smørgrav 
863bc5531deSDag-Erling Smørgrav 		if ((options & HKF_WANT_PARSE_KEY) != 0) {
864bc5531deSDag-Erling Smørgrav 			/*
865bc5531deSDag-Erling Smørgrav 			 * Extract the key from the line.  This will skip
866bc5531deSDag-Erling Smørgrav 			 * any leading whitespace.  Ignore badly formatted
867bc5531deSDag-Erling Smørgrav 			 * lines.
868bc5531deSDag-Erling Smørgrav 			 */
869bc5531deSDag-Erling Smørgrav 			if ((lineinfo.key = sshkey_new(KEY_UNSPEC)) == NULL) {
87019261079SEd Maste 				error_f("sshkey_new failed");
871bc5531deSDag-Erling Smørgrav 				r = SSH_ERR_ALLOC_FAIL;
872bc5531deSDag-Erling Smørgrav 				break;
873bc5531deSDag-Erling Smørgrav 			}
874bc5531deSDag-Erling Smørgrav 			if (!hostfile_read_key(&cp, &kbits, lineinfo.key)) {
875bc5531deSDag-Erling Smørgrav 				goto bad;
876bc5531deSDag-Erling Smørgrav 			}
877bc5531deSDag-Erling Smørgrav 			lineinfo.keytype = lineinfo.key->type;
878bc5531deSDag-Erling Smørgrav 			lineinfo.comment = cp;
879bc5531deSDag-Erling Smørgrav 		} else {
880bc5531deSDag-Erling Smørgrav 			/* Extract and parse key type */
881bc5531deSDag-Erling Smørgrav 			l = strcspn(lineinfo.rawkey, " \t");
882bc5531deSDag-Erling Smørgrav 			if (l <= 1 || l >= sizeof(ktype) ||
883bc5531deSDag-Erling Smørgrav 			    lineinfo.rawkey[l] == '\0')
884bc5531deSDag-Erling Smørgrav 				goto bad;
885bc5531deSDag-Erling Smørgrav 			memcpy(ktype, lineinfo.rawkey, l);
886bc5531deSDag-Erling Smørgrav 			ktype[l] = '\0';
887bc5531deSDag-Erling Smørgrav 			lineinfo.keytype = sshkey_type_from_name(ktype);
888557f75e5SDag-Erling Smørgrav 
889bc5531deSDag-Erling Smørgrav 			/*
8904f52dfbbSDag-Erling Smørgrav 			 * Assume legacy RSA1 if the first component is a short
891bc5531deSDag-Erling Smørgrav 			 * decimal number.
892bc5531deSDag-Erling Smørgrav 			 */
893bc5531deSDag-Erling Smørgrav 			if (lineinfo.keytype == KEY_UNSPEC && l < 8 &&
894bc5531deSDag-Erling Smørgrav 			    strspn(ktype, "0123456789") == l)
8954f52dfbbSDag-Erling Smørgrav 				goto bad;
896557f75e5SDag-Erling Smørgrav 
897bc5531deSDag-Erling Smørgrav 			/*
898bc5531deSDag-Erling Smørgrav 			 * Check that something other than whitespace follows
899bc5531deSDag-Erling Smørgrav 			 * the key type. This won't catch all corruption, but
900bc5531deSDag-Erling Smørgrav 			 * it does catch trivial truncation.
901bc5531deSDag-Erling Smørgrav 			 */
902bc5531deSDag-Erling Smørgrav 			cp2 += l; /* Skip past key type */
903bc5531deSDag-Erling Smørgrav 			for (; *cp2 == ' ' || *cp2 == '\t'; cp2++)
904bc5531deSDag-Erling Smørgrav 				;
905bc5531deSDag-Erling Smørgrav 			if (*cp2 == '\0' || *cp2 == '#') {
906bc5531deSDag-Erling Smørgrav 				debug2("%s:%ld: truncated after key type",
907bc5531deSDag-Erling Smørgrav 				    path, linenum);
908bc5531deSDag-Erling Smørgrav 				lineinfo.keytype = KEY_UNSPEC;
909bc5531deSDag-Erling Smørgrav 			}
910bc5531deSDag-Erling Smørgrav 			if (lineinfo.keytype == KEY_UNSPEC) {
911bc5531deSDag-Erling Smørgrav  bad:
912bc5531deSDag-Erling Smørgrav 				sshkey_free(lineinfo.key);
913bc5531deSDag-Erling Smørgrav 				lineinfo.key = NULL;
914bc5531deSDag-Erling Smørgrav 				lineinfo.status = HKF_STATUS_INVALID;
915bc5531deSDag-Erling Smørgrav 				if ((r = callback(&lineinfo, ctx)) != 0)
916bc5531deSDag-Erling Smørgrav 					break;
917bc5531deSDag-Erling Smørgrav 				continue;
918bc5531deSDag-Erling Smørgrav 			}
919bc5531deSDag-Erling Smørgrav 		}
920bc5531deSDag-Erling Smørgrav 		if ((r = callback(&lineinfo, ctx)) != 0)
921bc5531deSDag-Erling Smørgrav 			break;
922bc5531deSDag-Erling Smørgrav 	}
923bc5531deSDag-Erling Smørgrav 	sshkey_free(lineinfo.key);
924190cef3dSDag-Erling Smørgrav 	free(lineinfo.line);
925190cef3dSDag-Erling Smørgrav 	free(line);
92619261079SEd Maste 	return r;
92719261079SEd Maste }
92819261079SEd Maste 
92919261079SEd Maste int
hostkeys_foreach(const char * path,hostkeys_foreach_fn * callback,void * ctx,const char * host,const char * ip,u_int options,u_int note)93019261079SEd Maste hostkeys_foreach(const char *path, hostkeys_foreach_fn *callback, void *ctx,
93119261079SEd Maste     const char *host, const char *ip, u_int options, u_int note)
93219261079SEd Maste {
93319261079SEd Maste 	FILE *f;
93419261079SEd Maste 	int r, oerrno;
93519261079SEd Maste 
93619261079SEd Maste 	if ((f = fopen(path, "r")) == NULL)
93719261079SEd Maste 		return SSH_ERR_SYSTEM_ERROR;
93819261079SEd Maste 
93919261079SEd Maste 	debug3_f("reading file \"%s\"", path);
94019261079SEd Maste 	r = hostkeys_foreach_file(path, f, callback, ctx, host, ip,
94119261079SEd Maste 	    options, note);
94219261079SEd Maste 	oerrno = errno;
943511b41d2SMark Murray 	fclose(f);
94419261079SEd Maste 	errno = oerrno;
945bc5531deSDag-Erling Smørgrav 	return r;
946511b41d2SMark Murray }
947