diff options
author | djm@openbsd.org <djm@openbsd.org> | 2015-01-26 06:10:03 +0000 |
---|---|---|
committer | Damien Miller <djm@mindrot.org> | 2015-01-27 00:00:57 +1100 |
commit | 5104db7cbd6cdd9c5971f4358e74414862fc1022 (patch) | |
tree | 94692c77a4888f8adade706324fdee3a999bc6b0 /kexgexs.c | |
parent | 8d4f87258f31cb6def9b3b55b6a7321d84728ff2 (diff) |
upstream commit
correctly match ECDSA subtype (== curve) for
offered/recevied host keys. Fixes connection-killing host key mismatches when
a server offers multiple ECDSA keys with different curve type (an extremely
unlikely configuration).
ok markus, "looks mechanical" deraadt@
Diffstat (limited to 'kexgexs.c')
-rw-r--r-- | kexgexs.c | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: kexgexs.c,v 1.23 2015/01/20 23:14:00 deraadt Exp $ */ | 1 | /* $OpenBSD: kexgexs.c,v 1.24 2015/01/26 06:10:03 djm Exp $ */ |
2 | /* | 2 | /* |
3 | * Copyright (c) 2000 Niels Provos. All rights reserved. | 3 | * Copyright (c) 2000 Niels Provos. All rights reserved. |
4 | * Copyright (c) 2001 Markus Friedl. All rights reserved. | 4 | * Copyright (c) 2001 Markus Friedl. All rights reserved. |
@@ -160,8 +160,10 @@ input_kex_dh_gex_init(int type, u_int32_t seq, void *ctxt) | |||
160 | r = SSH_ERR_INVALID_ARGUMENT; | 160 | r = SSH_ERR_INVALID_ARGUMENT; |
161 | goto out; | 161 | goto out; |
162 | } | 162 | } |
163 | server_host_public = kex->load_host_public_key(kex->hostkey_type, ssh); | 163 | server_host_public = kex->load_host_public_key(kex->hostkey_type, |
164 | server_host_private = kex->load_host_private_key(kex->hostkey_type, ssh); | 164 | kex->hostkey_nid, ssh); |
165 | server_host_private = kex->load_host_private_key(kex->hostkey_type, | ||
166 | kex->hostkey_nid, ssh); | ||
165 | if (server_host_public == NULL) { | 167 | if (server_host_public == NULL) { |
166 | r = SSH_ERR_NO_HOSTKEY_LOADED; | 168 | r = SSH_ERR_NO_HOSTKEY_LOADED; |
167 | goto out; | 169 | goto out; |