]> git.ipfire.org Git - thirdparty/openssh-portable.git/commitdiff
upstream commit
authormarkus@openbsd.org <markus@openbsd.org>
Fri, 4 Dec 2015 16:41:28 +0000 (16:41 +0000)
committerDamien Miller <djm@mindrot.org>
Mon, 7 Dec 2015 01:38:58 +0000 (12:38 +1100)
implement SHA2-{256,512} for RSASSA-PKCS1-v1_5 signatures
 (user and host auth) based on draft-rsa-dsa-sha2-256-03.txt and
 draft-ssh-ext-info-04.txt; with & ok djm@

Upstream-ID: cf82ce532b2733e5c4b34bb7b7c94835632db309

28 files changed:
auth.h
authfd.c
authfd.h
kex.c
kex.h
kexc25519s.c
kexdhs.c
kexecdhs.c
kexgexs.c
key.c
key.h
krl.c
monitor.c
monitor_wrap.c
monitor_wrap.h
myproposal.h
packet.c
serverloop.c
ssh-agent.c
ssh-keygen.c
ssh-keysign.c
ssh-rsa.c
ssh2.h
ssh_api.c
sshconnect2.c
sshd.c
sshkey.c
sshkey.h

diff --git a/auth.h b/auth.h
index b235906cf9ba1cecfe4059ec5f056bf3e6de9bd5..2160154f495c288d5e3b000436f3db1ed4070a66 100644 (file)
--- a/auth.h
+++ b/auth.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: auth.h,v 1.85 2015/11/11 01:48:01 djm Exp $ */
+/* $OpenBSD: auth.h,v 1.86 2015/12/04 16:41:28 markus Exp $ */
 
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
@@ -209,7 +209,7 @@ Key *get_hostkey_private_by_type(int, int, struct ssh *);
 int     get_hostkey_index(Key *, int, struct ssh *);
 int     ssh1_session_key(BIGNUM *);
 int     sshd_hostkey_sign(Key *, Key *, u_char **, size_t *,
-            const u_char *, size_t, u_int);
+            const u_char *, size_t, const char *, u_int);
 
 /* debug messages during authentication */
 void    auth_debug_add(const char *fmt,...) __attribute__((format(printf, 1, 2)));
index 12bf1251f68dab275d2fa0d3c98230ba2be31c5d..a634bcb81c583e0f43994c85f6f201cb0d53050d 100644 (file)
--- a/authfd.c
+++ b/authfd.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: authfd.c,v 1.99 2015/09/02 07:51:12 jsg Exp $ */
+/* $OpenBSD: authfd.c,v 1.100 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -426,11 +426,24 @@ ssh_decrypt_challenge(int sock, struct sshkey* key, BIGNUM *challenge,
 }
 #endif
 
+/* encode signature algoritm in flag bits, so we can keep the msg format */
+static u_int
+agent_encode_alg(struct sshkey *key, const char *alg)
+{
+       if (alg != NULL && key->type == KEY_RSA) {
+               if (strcmp(alg, "rsa-sha2-256") == 0)
+                       return SSH_AGENT_RSA_SHA2_256;
+               else if (strcmp(alg, "rsa-sha2-512") == 0)
+                       return SSH_AGENT_RSA_SHA2_512;
+       }
+       return 0;
+}
+
 /* ask agent to sign data, returns err.h code on error, 0 on success */
 int
 ssh_agent_sign(int sock, struct sshkey *key,
     u_char **sigp, size_t *lenp,
-    const u_char *data, size_t datalen, u_int compat)
+    const u_char *data, size_t datalen, const char *alg, u_int compat)
 {
        struct sshbuf *msg;
        u_char *blob = NULL, type;
@@ -449,6 +462,7 @@ ssh_agent_sign(int sock, struct sshkey *key,
                return SSH_ERR_ALLOC_FAIL;
        if ((r = sshkey_to_blob(key, &blob, &blen)) != 0)
                goto out;
+       flags |= agent_encode_alg(key, alg);
        if ((r = sshbuf_put_u8(msg, SSH2_AGENTC_SIGN_REQUEST)) != 0 ||
            (r = sshbuf_put_string(msg, blob, blen)) != 0 ||
            (r = sshbuf_put_string(msg, data, datalen)) != 0 ||
index bea20c26bc967e17d2468555c9c9d7ba7088c6b5..4b417e3f4a2262606c683f400503b6f7f2e27bd1 100644 (file)
--- a/authfd.h
+++ b/authfd.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: authfd.h,v 1.38 2015/01/14 20:05:27 djm Exp $ */
+/* $OpenBSD: authfd.h,v 1.39 2015/12/04 16:41:28 markus Exp $ */
 
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
@@ -41,7 +41,7 @@ int   ssh_decrypt_challenge(int sock, struct sshkey* key, BIGNUM *challenge,
            u_char session_id[16], u_char response[16]);
 int    ssh_agent_sign(int sock, struct sshkey *key,
            u_char **sigp, size_t *lenp,
-           const u_char *data, size_t datalen, u_int compat);
+           const u_char *data, size_t datalen, const char *alg, u_int compat);
 
 /* Messages for the authentication agent connection. */
 #define SSH_AGENTC_REQUEST_RSA_IDENTITIES      1
@@ -86,5 +86,7 @@ int   ssh_agent_sign(int sock, struct sshkey *key,
 #define SSH_COM_AGENT2_FAILURE                 102
 
 #define        SSH_AGENT_OLD_SIGNATURE                 0x01
+#define        SSH_AGENT_RSA_SHA2_256                  0x02
+#define        SSH_AGENT_RSA_SHA2_512                  0x04
 
 #endif                         /* AUTHFD_H */
diff --git a/kex.c b/kex.c
index b409f276bdfd64b33c43a59cd160ce6571512738..c1371c432c4b1ed0a132be7ddf55f7294dfbc269 100644 (file)
--- a/kex.c
+++ b/kex.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: kex.c,v 1.112 2015/11/13 04:39:35 djm Exp $ */
+/* $OpenBSD: kex.c,v 1.113 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2000, 2001 Markus Friedl.  All rights reserved.
  *
@@ -334,6 +334,20 @@ kex_reset_dispatch(struct ssh *ssh)
        ssh_dispatch_set(ssh, SSH2_MSG_KEXINIT, &kex_input_kexinit);
 }
 
+static int
+kex_send_ext_info(struct ssh *ssh)
+{
+       int r;
+
+       if ((r = sshpkt_start(ssh, SSH2_MSG_EXT_INFO)) != 0 ||
+           (r = sshpkt_put_u32(ssh, 1)) != 0 ||
+           (r = sshpkt_put_cstring(ssh, "server-sig-algs")) != 0 ||
+           (r = sshpkt_put_cstring(ssh, "rsa-sha2-256,rsa-sha2-512")) != 0 ||
+           (r = sshpkt_send(ssh)) != 0)
+               return r;
+       return 0;
+}
+
 int
 kex_send_newkeys(struct ssh *ssh)
 {
@@ -346,9 +360,51 @@ kex_send_newkeys(struct ssh *ssh)
        debug("SSH2_MSG_NEWKEYS sent");
        debug("expecting SSH2_MSG_NEWKEYS");
        ssh_dispatch_set(ssh, SSH2_MSG_NEWKEYS, &kex_input_newkeys);
+       if (ssh->kex->ext_info_c)
+               if ((r = kex_send_ext_info(ssh)) != 0)
+                       return r;
        return 0;
 }
 
+int
+kex_input_ext_info(int type, u_int32_t seq, void *ctxt)
+{
+       struct ssh *ssh = ctxt;
+       struct kex *kex = ssh->kex;
+       u_int32_t i, ninfo;
+       char *name, *val, *found;
+       int r;
+
+       debug("SSH2_MSG_EXT_INFO received");
+       ssh_dispatch_set(ssh, SSH2_MSG_EXT_INFO, &kex_protocol_error);
+       if ((r = sshpkt_get_u32(ssh, &ninfo)) != 0)
+               return r;
+       for (i = 0; i < ninfo; i++) {
+               if ((r = sshpkt_get_cstring(ssh, &name, NULL)) != 0)
+                       return r;
+               if ((r = sshpkt_get_cstring(ssh, &val, NULL)) != 0) {
+                       free(name);
+                       return r;
+               }
+               debug("%s: %s=<%s>", __func__, name, val);
+               if (strcmp(name, "server-sig-algs") == 0) {
+                       found = match_list("rsa-sha2-256", val, NULL);
+                       if (found) {
+                               kex->rsa_sha2 = 256;
+                               free(found);
+                       }
+                       found = match_list("rsa-sha2-512", val, NULL);
+                       if (found) {
+                               kex->rsa_sha2 = 512;
+                               free(found);
+                       }
+               }
+               free(name);
+               free(val);
+       }
+       return sshpkt_get_end(ssh);
+}
+
 static int
 kex_input_newkeys(int type, u_int32_t seq, void *ctxt)
 {
@@ -531,6 +587,8 @@ kex_free(struct kex *kex)
        free(kex->client_version_string);
        free(kex->server_version_string);
        free(kex->failed_choice);
+       free(kex->hostkey_alg);
+       free(kex->name);
        free(kex);
 }
 
@@ -627,17 +685,16 @@ choose_kex(struct kex *k, char *client, char *server)
 static int
 choose_hostkeyalg(struct kex *k, char *client, char *server)
 {
-       char *hostkeyalg = match_list(client, server, NULL);
+       k->hostkey_alg = match_list(client, server, NULL);
 
        debug("kex: host key algorithm: %s",
-           hostkeyalg ? hostkeyalg : "(no match)");
-       if (hostkeyalg == NULL)
+           k->hostkey_alg ? k->hostkey_alg : "(no match)");
+       if (k->hostkey_alg == NULL)
                return SSH_ERR_NO_HOSTKEY_ALG_MATCH;
-       k->hostkey_type = sshkey_type_from_name(hostkeyalg);
+       k->hostkey_type = sshkey_type_from_name(k->hostkey_alg);
        if (k->hostkey_type == KEY_UNSPEC)
                return SSH_ERR_INTERNAL_ERROR;
-       k->hostkey_nid = sshkey_ecdsa_nid_from_name(hostkeyalg);
-       free(hostkeyalg);
+       k->hostkey_nid = sshkey_ecdsa_nid_from_name(k->hostkey_alg);
        return 0;
 }
 
@@ -702,6 +759,17 @@ kex_choose_conf(struct ssh *ssh)
                }
        }
 
+       /* Check whether client supports ext_info_c */
+       if (kex->server) {
+               char *ext;
+
+               ext = match_list("ext-info-c", peer[PROPOSAL_KEX_ALGS], NULL);
+               if (ext) {
+                       kex->ext_info_c = 1;
+                       free(ext);
+               }
+       }
+
        /* Algorithm Negotiation */
        if ((r = choose_kex(kex, cprop[PROPOSAL_KEX_ALGS],
            sprop[PROPOSAL_KEX_ALGS])) != 0) {
diff --git a/kex.h b/kex.h
index d71b53293d8d43095fcd4cd1951c4a1c012acd2b..25ccf2e0ea2ca2ba42127624d072a692e2d7c379 100644 (file)
--- a/kex.h
+++ b/kex.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: kex.h,v 1.73 2015/07/30 00:01:34 djm Exp $ */
+/* $OpenBSD: kex.h,v 1.74 2015/12/04 16:41:28 markus Exp $ */
 
 /*
  * Copyright (c) 2000, 2001 Markus Friedl.  All rights reserved.
@@ -129,10 +129,13 @@ struct kex {
        u_int   dh_need;
        int     server;
        char    *name;
+       char    *hostkey_alg;
        int     hostkey_type;
        int     hostkey_nid;
        u_int   kex_type;
        int     roaming;
+       int     rsa_sha2;
+       int     ext_info_c;
        struct sshbuf *my;
        struct sshbuf *peer;
        sig_atomic_t done;
@@ -146,8 +149,8 @@ struct kex {
        struct sshkey *(*load_host_public_key)(int, int, struct ssh *);
        struct sshkey *(*load_host_private_key)(int, int, struct ssh *);
        int     (*host_key_index)(struct sshkey *, int, struct ssh *);
-       int     (*sign)(struct sshkey *, struct sshkey *,
-           u_char **, size_t *, const u_char *, size_t, u_int);
+       int     (*sign)(struct sshkey *, struct sshkey *, u_char **, size_t *,
+           const u_char *, size_t, const char *, u_int);
        int     (*kex[KEX_MAX])(struct ssh *);
        /* kex specific state */
        DH      *dh;                    /* DH */
@@ -174,6 +177,7 @@ void         kex_prop_free(char **);
 
 int     kex_send_kexinit(struct ssh *);
 int     kex_input_kexinit(int, u_int32_t, void *);
+int     kex_input_ext_info(int, u_int32_t, void *);
 int     kex_derive_keys(struct ssh *, u_char *, u_int, const struct sshbuf *);
 int     kex_derive_keys_bn(struct ssh *, u_char *, u_int, const BIGNUM *);
 int     kex_send_newkeys(struct ssh *);
index 24027253363ee8e72f44164cd46666caac855fee..4e77622b0b2ff579122a5e6e87104dd8aa4aa06a 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: kexc25519s.c,v 1.9 2015/04/27 00:37:53 dtucker Exp $ */
+/* $OpenBSD: kexc25519s.c,v 1.10 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2001 Markus Friedl.  All rights reserved.
  * Copyright (c) 2010 Damien Miller.  All rights reserved.
@@ -134,8 +134,8 @@ input_kex_c25519_init(int type, u_int32_t seq, void *ctxt)
        }
 
        /* sign H */
-       if ((r = kex->sign(server_host_private, server_host_public,
-           &signature, &slen, hash, hashlen, ssh->compat)) < 0)
+       if ((r = kex->sign(server_host_private, server_host_public, &signature,
+            &slen, hash, hashlen, kex->hostkey_alg, ssh->compat)) < 0)
                goto out;
 
        /* send server hostkey, ECDH pubkey 'Q_S' and signed H */
index de7c05b17249152265f5506c397d5d479ca21aec..bf933e4c90133d7ff8e7ab36f666a45da8ee619f 100644 (file)
--- a/kexdhs.c
+++ b/kexdhs.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: kexdhs.c,v 1.22 2015/01/26 06:10:03 djm Exp $ */
+/* $OpenBSD: kexdhs.c,v 1.23 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2001 Markus Friedl.  All rights reserved.
  *
@@ -181,8 +181,8 @@ input_kex_dh_init(int type, u_int32_t seq, void *ctxt)
        }
 
        /* sign H */
-       if ((r = kex->sign(server_host_private, server_host_public,
-           &signature, &slen, hash, hashlen, ssh->compat)) < 0)
+       if ((r = kex->sign(server_host_private, server_host_public, &signature,
+            &slen, hash, hashlen, kex->hostkey_alg, ssh->compat)) < 0)
                goto out;
 
        /* destroy_sensitive_data(); */
index 0adb80e6addf178231bffa9db7acb27c0ec3f225..ccdbf70b1cf612f470b499a99d1109649ed6f27a 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: kexecdhs.c,v 1.14 2015/01/26 06:10:03 djm Exp $ */
+/* $OpenBSD: kexecdhs.c,v 1.15 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2001 Markus Friedl.  All rights reserved.
  * Copyright (c) 2010 Damien Miller.  All rights reserved.
@@ -169,8 +169,8 @@ input_kex_ecdh_init(int type, u_int32_t seq, void *ctxt)
        }
 
        /* sign H */
-       if ((r = kex->sign(server_host_private, server_host_public,
-           &signature, &slen, hash, hashlen, ssh->compat)) < 0)
+       if ((r = kex->sign(server_host_private, server_host_public, &signature,
+            &slen, hash, hashlen, kex->hostkey_alg, ssh->compat)) < 0)
                goto out;
 
        /* destroy_sensitive_data(); */
index ff6c6879e73a4a4f0ec8c4a748039819bdbf14ef..8c5adf7e43058c859efedec59172bb811bc1e129 100644 (file)
--- a/kexgexs.c
+++ b/kexgexs.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: kexgexs.c,v 1.25 2015/04/13 02:04:08 djm Exp $ */
+/* $OpenBSD: kexgexs.c,v 1.26 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2000 Niels Provos.  All rights reserved.
  * Copyright (c) 2001 Markus Friedl.  All rights reserved.
@@ -220,8 +220,8 @@ input_kex_dh_gex_init(int type, u_int32_t seq, void *ctxt)
        }
 
        /* sign H */
-       if ((r = kex->sign(server_host_private, server_host_public,
-           &signature, &slen, hash, hashlen, ssh->compat)) < 0)
+       if ((r = kex->sign(server_host_private, server_host_public, &signature,
+            &slen, hash, hashlen, kex->hostkey_alg, ssh->compat)) < 0)
                goto out;
 
        /* destroy_sensitive_data(); */
diff --git a/key.c b/key.c
index 0ba98b6f35bdb85e8b76247091ca8068470a754b..28d7c62071456d63dd4015113329adf42236e822 100644 (file)
--- a/key.c
+++ b/key.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: key.c,v 1.128 2015/07/03 03:43:18 djm Exp $ */
+/* $OpenBSD: key.c,v 1.129 2015/12/04 16:41:28 markus Exp $ */
 /*
  * placed in the public domain
  */
@@ -132,7 +132,7 @@ key_to_blob(const Key *key, u_char **blobp, u_int *lenp)
 
 int
 key_sign(const Key *key, u_char **sigp, u_int *lenp,
-    const u_char *data, u_int datalen)
+    const u_char *data, u_int datalen, const char *alg)
 {
        int r;
        u_char *sig;
@@ -143,7 +143,7 @@ key_sign(const Key *key, u_char **sigp, u_int *lenp,
        if (lenp != NULL)
                *lenp = 0;
        if ((r = sshkey_sign(key, &sig, &siglen,
-           data, datalen, datafellows)) != 0) {
+           data, datalen, alg, datafellows)) != 0) {
                fatal_on_fatal_errors(r, __func__, 0);
                error("%s: %s", __func__, ssh_err(r));
                return -1;
diff --git a/key.h b/key.h
index 903bdf6730de31ac034b2815a8d96cd69d80c2f7..34c992bd300b063d372e0e820600c85e6bf098a6 100644 (file)
--- a/key.h
+++ b/key.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: key.h,v 1.48 2015/07/03 03:43:18 djm Exp $ */
+/* $OpenBSD: key.h,v 1.49 2015/12/04 16:41:28 markus Exp $ */
 
 /*
  * Copyright (c) 2000, 2001 Markus Friedl.  All rights reserved.
@@ -84,7 +84,8 @@ int    key_ec_validate_private(const EC_KEY *);
 Key    *key_from_blob(const u_char *, u_int);
 int     key_to_blob(const Key *, u_char **, u_int *);
 
-int     key_sign(const Key *, u_char **, u_int *, const u_char *, u_int);
+int     key_sign(const Key *, u_char **, u_int *, const u_char *, u_int,
+    const char *);
 int     key_verify(const Key *, const u_char *, u_int, const u_char *, u_int);
 
 void     key_private_serialize(const Key *, struct sshbuf *);
diff --git a/krl.c b/krl.c
index 570a4f7474848e83e8094a502162d887ed10314c..0194f1c72be23faf874f7088fb82469415106b0b 100644 (file)
--- a/krl.c
+++ b/krl.c
@@ -14,7 +14,7 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-/* $OpenBSD: krl.c,v 1.34 2015/09/02 07:51:12 jsg Exp $ */
+/* $OpenBSD: krl.c,v 1.35 2015/12/04 16:41:28 markus Exp $ */
 
 #include "includes.h"
 
@@ -772,7 +772,7 @@ ssh_krl_to_blob(struct ssh_krl *krl, struct sshbuf *buf,
                        goto out;
 
                if ((r = sshkey_sign(sign_keys[i], &sblob, &slen,
-                   sshbuf_ptr(buf), sshbuf_len(buf), 0)) != 0)
+                   sshbuf_ptr(buf), sshbuf_len(buf), NULL, 0)) != 0)
                        goto out;
                KRL_DBG(("%s: signature sig len %zu", __func__, slen));
                if ((r = sshbuf_put_string(buf, sblob, slen)) != 0)
index 4060a6ec9b29020e556548fdcf6cfb2d6009eb54..b3edd648b7fca5ba08b6be4dfd75351768991d79 100644 (file)
--- a/monitor.c
+++ b/monitor.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: monitor.c,v 1.154 2015/10/20 23:24:25 mmcc Exp $ */
+/* $OpenBSD: monitor.c,v 1.155 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright 2002 Niels Provos <provos@citi.umich.edu>
  * Copyright 2002 Markus Friedl <markus@openbsd.org>
@@ -688,14 +688,16 @@ mm_answer_sign(int sock, Buffer *m)
        struct sshbuf *sigbuf;
        u_char *p;
        u_char *signature;
-       size_t datlen, siglen;
+       char *alg;
+       size_t datlen, siglen, alglen;
        int r, keyid, is_proof = 0;
        const char proof_req[] = "hostkeys-prove-00@openssh.com";
 
        debug3("%s", __func__);
 
        if ((r = sshbuf_get_u32(m, &keyid)) != 0 ||
-           (r = sshbuf_get_string(m, &p, &datlen)) != 0)
+           (r = sshbuf_get_string(m, &p, &datlen)) != 0 ||
+           (r = sshbuf_get_cstring(m, &alg, &alglen)) != 0)
                fatal("%s: buffer error: %s", __func__, ssh_err(r));
 
        /*
@@ -742,14 +744,14 @@ mm_answer_sign(int sock, Buffer *m)
        }
 
        if ((key = get_hostkey_by_index(keyid)) != NULL) {
-               if ((r = sshkey_sign(key, &signature, &siglen, p, datlen,
+               if ((r = sshkey_sign(key, &signature, &siglen, p, datlen, alg,
                    datafellows)) != 0)
                        fatal("%s: sshkey_sign failed: %s",
                            __func__, ssh_err(r));
        } else if ((key = get_hostkey_public_by_index(keyid, ssh)) != NULL &&
            auth_sock > 0) {
                if ((r = ssh_agent_sign(auth_sock, key, &signature, &siglen,
-                   p, datlen, datafellows)) != 0) {
+                   p, datlen, alg, datafellows)) != 0) {
                        fatal("%s: ssh_agent_sign failed: %s",
                            __func__, ssh_err(r));
                }
index eac421ba145bf69061e9344c1ad4dfd281c1ae20..d4bfaf372dcb089a32126f710fbfe9e0ad301b96 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: monitor_wrap.c,v 1.85 2015/05/01 03:23:51 djm Exp $ */
+/* $OpenBSD: monitor_wrap.c,v 1.86 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright 2002 Niels Provos <provos@citi.umich.edu>
  * Copyright 2002 Markus Friedl <markus@openbsd.org>
@@ -218,7 +218,7 @@ mm_choose_dh(int min, int nbits, int max)
 
 int
 mm_key_sign(Key *key, u_char **sigp, u_int *lenp,
-    const u_char *data, u_int datalen)
+    const u_char *data, u_int datalen, const char *hostkey_alg)
 {
        struct kex *kex = *pmonitor->m_pkex;
        Buffer m;
@@ -228,6 +228,7 @@ mm_key_sign(Key *key, u_char **sigp, u_int *lenp,
        buffer_init(&m);
        buffer_put_int(&m, kex->host_key_index(key, 0, active_state));
        buffer_put_string(&m, data, datalen);
+       buffer_put_cstring(&m, hostkey_alg);
 
        mm_request_send(pmonitor->m_recvfd, MONITOR_REQ_SIGN, &m);
 
index 9a6aff68479c9030fe92c4b0a965772e73a67c64..eb820aeea743a9b2630801d02cbda21518681039 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: monitor_wrap.h,v 1.28 2015/11/11 01:48:01 djm Exp $ */
+/* $OpenBSD: monitor_wrap.h,v 1.29 2015/12/04 16:41:28 markus Exp $ */
 
 /*
  * Copyright 2002 Niels Provos <provos@citi.umich.edu>
@@ -40,7 +40,7 @@ struct Authctxt;
 void mm_log_handler(LogLevel, const char *, void *);
 int mm_is_monitor(void);
 DH *mm_choose_dh(int, int, int);
-int mm_key_sign(Key *, u_char **, u_int *, const u_char *, u_int);
+int mm_key_sign(Key *, u_char **, u_int *, const u_char *, u_int, const char *);
 void mm_inform_authserv(char *, char *);
 struct passwd *mm_getpwnamallow(const char *);
 char *mm_auth2_read_banner(void);
index 46e5b988d463e70e3e3098f429443ab9b8ff1236..c6429588a0cfd6e901685e8bc14572426da14d86 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: myproposal.h,v 1.47 2015/07/10 06:21:53 markus Exp $ */
+/* $OpenBSD: myproposal.h,v 1.48 2015/12/04 16:41:28 markus Exp $ */
 
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
        "ssh-rsa-cert-v01@openssh.com," \
        HOSTKEY_ECDSA_METHODS \
        "ssh-ed25519," \
-       "ssh-rsa" \
+       "rsa-sha2-256," \
+       "rsa-sha2-512," \
+       "ssh-rsa"
 
 /* the actual algorithms */
 
index 4f6433b47980bcf2023307ae56416ad7c5d98196..378906956153c815bbf848bea8e167773db70291 100644 (file)
--- a/packet.c
+++ b/packet.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: packet.c,v 1.217 2015/11/08 21:59:11 djm Exp $ */
+/* $OpenBSD: packet.c,v 1.218 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -1215,7 +1215,8 @@ ssh_packet_send2(struct ssh *ssh)
                if ((type < SSH2_MSG_TRANSPORT_MIN) ||
                    (type > SSH2_MSG_TRANSPORT_MAX) ||
                    (type == SSH2_MSG_SERVICE_REQUEST) ||
-                   (type == SSH2_MSG_SERVICE_ACCEPT)) {
+                   (type == SSH2_MSG_SERVICE_ACCEPT) ||
+                   (type == SSH2_MSG_EXT_INFO)) {
                        debug("enqueue packet: %u", type);
                        p = calloc(1, sizeof(*p));
                        if (p == NULL)
index 4d0c0edb1feb3a4f47eaf5f2cefb90d0e934400d..85fc8d3af6318ffe5dd0f8ae73bacf803a5232e7 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: serverloop.c,v 1.179 2015/11/28 06:41:03 djm Exp $ */
+/* $OpenBSD: serverloop.c,v 1.180 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -1201,7 +1201,7 @@ server_input_hostkeys_prove(struct sshbuf **respp)
                    ssh->kex->session_id, ssh->kex->session_id_len)) != 0 ||
                    (r = sshkey_puts(key, sigbuf)) != 0 ||
                    (r = ssh->kex->sign(key_prv, key_pub, &sig, &slen,
-                   sshbuf_ptr(sigbuf), sshbuf_len(sigbuf), 0)) != 0 ||
+                   sshbuf_ptr(sigbuf), sshbuf_len(sigbuf), NULL, 0)) != 0 ||
                    (r = sshbuf_put_string(resp, sig, slen)) != 0) {
                        error("%s: couldn't prepare signature: %s",
                            __func__, ssh_err(r));
index 2a7ae88f9649c84a11265d3298a3b4cac2b69ba7..ed5dc571d8a6ba76119d2fbd9027e73d2d472999 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh-agent.c,v 1.207 2015/12/02 08:30:50 doug Exp $ */
+/* $OpenBSD: ssh-agent.c,v 1.208 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -368,6 +368,18 @@ process_authentication_challenge1(SocketEntry *e)
 }
 #endif
 
+static char *
+agent_decode_alg(struct sshkey *key, u_int flags)
+{
+       if (key->type == KEY_RSA) {
+               if (flags & SSH_AGENT_RSA_SHA2_256)
+                       return "rsa-sha2-256";
+               else if (flags & SSH_AGENT_RSA_SHA2_512)
+                       return "rsa-sha2-512";
+       }
+       return NULL;
+}
+
 /* ssh2 only */
 static void
 process_sign_request2(SocketEntry *e)
@@ -401,7 +413,7 @@ process_sign_request2(SocketEntry *e)
                goto send;
        }
        if ((r = sshkey_sign(id->key, &signature, &slen,
-           data, dlen, compat)) != 0) {
+           data, dlen, agent_decode_alg(key, flags), compat)) != 0) {
                error("%s: sshkey_sign: %s", __func__, ssh_err(ok));
                goto send;
        }
index f6a1c2847dd1f145c16e6fd881c296871a1e5ae5..6ac1fa60359cdb95dd419f2efc82415a811afec5 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh-keygen.c,v 1.284 2015/11/28 06:50:52 deraadt Exp $ */
+/* $OpenBSD: ssh-keygen.c,v 1.285 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1994 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -523,7 +523,7 @@ do_convert_private_ssh2_from_blob(u_char *blob, u_int blen)
        sshbuf_free(b);
 
        /* try the key */
-       if (sshkey_sign(key, &sig, &slen, data, sizeof(data), 0) != 0 ||
+       if (sshkey_sign(key, &sig, &slen, data, sizeof(data), NULL, 0) != 0 ||
            sshkey_verify(key, sig, slen, data, sizeof(data), 0) != 0) {
                sshkey_free(key);
                free(sig);
index 4c99609b2df89aab72464e7e2f48def1e7451ae4..1d49861ae7cf1e1347fce0b9ba56ebd9c7f60827 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh-keysign.c,v 1.50 2015/11/29 22:18:37 djm Exp $ */
+/* $OpenBSD: ssh-keysign.c,v 1.51 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2002 Markus Friedl.  All rights reserved.
  *
@@ -291,7 +291,8 @@ main(int argc, char **argv)
                    sshkey_type(key), fp ? fp : "");
        }
 
-       if ((r = sshkey_sign(keys[i], &signature, &slen, data, dlen, 0)) != 0)
+       if ((r = sshkey_sign(keys[i], &signature, &slen, data, dlen, NULL, 0))
+           != 0)
                fatal("sshkey_sign failed: %s", ssh_err(r));
        free(data);
 
index 08090d14ee2d8d675a0e3a1421bea61feaa83522..81dab05b3ffc0b9bbbaeee5b3efe743cdb8931d1 100644 (file)
--- a/ssh-rsa.c
+++ b/ssh-rsa.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh-rsa.c,v 1.54 2015/09/09 00:52:44 djm Exp $ */
+/* $OpenBSD: ssh-rsa.c,v 1.55 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2000, 2003 Markus Friedl <markus@openbsd.org>
  *
 
 static int openssh_RSA_verify(int, u_char *, size_t, u_char *, size_t, RSA *);
 
+static const char *
+rsa_hash_alg_ident(int hash_alg)
+{
+       switch (hash_alg) {
+       case SSH_DIGEST_SHA1:
+               return "ssh-rsa";
+       case SSH_DIGEST_SHA256:
+               return "rsa-sha2-256";
+       case SSH_DIGEST_SHA512:
+               return "rsa-sha2-512";
+       }
+       return NULL;
+}
+
+static int
+rsa_hash_alg_from_ident(const char *ident)
+{
+       if (ident == NULL || strlen(ident) == 0)
+               return SSH_DIGEST_SHA1;
+       if (strcmp(ident, "ssh-rsa") == 0)
+               return SSH_DIGEST_SHA1;
+       if (strcmp(ident, "rsa-sha2-256") == 0)
+               return SSH_DIGEST_SHA256;
+       if (strcmp(ident, "rsa-sha2-512") == 0)
+               return SSH_DIGEST_SHA512;
+       if (strncmp(ident, "ssh-rsa-cert", strlen("ssh-rsa-cert")) == 0)
+               return SSH_DIGEST_SHA1;
+       return -1;
+}
+
+static int
+rsa_hash_alg_nid(int type)
+{
+       switch (type) {
+       case SSH_DIGEST_SHA1:
+               return NID_sha1;
+       case SSH_DIGEST_SHA256:
+               return NID_sha256;
+       case SSH_DIGEST_SHA512:
+               return NID_sha512;
+       default:
+               return -1;
+       }
+}
+
 /* RSASSA-PKCS1-v1_5 (PKCS #1 v2.0 signature) with SHA1 */
 int
 ssh_rsa_sign(const struct sshkey *key, u_char **sigp, size_t *lenp,
-    const u_char *data, size_t datalen, u_int compat)
+    const u_char *data, size_t datalen, const char *alg_ident)
 {
-       int hash_alg;
        u_char digest[SSH_DIGEST_MAX_LENGTH], *sig = NULL;
        size_t slen;
        u_int dlen, len;
-       int nid, ret = SSH_ERR_INTERNAL_ERROR;
+       int nid, hash_alg, ret = SSH_ERR_INTERNAL_ERROR;
        struct sshbuf *b = NULL;
 
        if (lenp != NULL)
@@ -53,16 +97,17 @@ ssh_rsa_sign(const struct sshkey *key, u_char **sigp, size_t *lenp,
        if (sigp != NULL)
                *sigp = NULL;
 
-       if (key == NULL || key->rsa == NULL ||
-           sshkey_type_plain(key->type) != KEY_RSA)
+       hash_alg = rsa_hash_alg_from_ident(alg_ident);
+       if (key == NULL || key->rsa == NULL || hash_alg == -1 ||
+           sshkey_type_plain(key->type) != KEY_RSA ||
+           BN_num_bits(key->rsa->n) < SSH_RSA_MINIMUM_MODULUS_SIZE)
                return SSH_ERR_INVALID_ARGUMENT;
        slen = RSA_size(key->rsa);
        if (slen <= 0 || slen > SSHBUF_MAX_BIGNUM)
                return SSH_ERR_INVALID_ARGUMENT;
 
        /* hash the data */
-       hash_alg = SSH_DIGEST_SHA1;
-       nid = NID_sha1;
+       nid = rsa_hash_alg_nid(hash_alg);
        if ((dlen = ssh_digest_bytes(hash_alg)) == 0)
                return SSH_ERR_INTERNAL_ERROR;
        if ((ret = ssh_digest_memory(hash_alg, data, datalen,
@@ -91,7 +136,7 @@ ssh_rsa_sign(const struct sshkey *key, u_char **sigp, size_t *lenp,
                ret = SSH_ERR_ALLOC_FAIL;
                goto out;
        }
-       if ((ret = sshbuf_put_cstring(b, "ssh-rsa")) != 0 ||
+       if ((ret = sshbuf_put_cstring(b, rsa_hash_alg_ident(hash_alg))) != 0 ||
            (ret = sshbuf_put_string(b, sig, slen)) != 0)
                goto out;
        len = sshbuf_len(b);
@@ -118,8 +163,7 @@ ssh_rsa_sign(const struct sshkey *key, u_char **sigp, size_t *lenp,
 
 int
 ssh_rsa_verify(const struct sshkey *key,
-    const u_char *signature, size_t signaturelen,
-    const u_char *data, size_t datalen, u_int compat)
+    const u_char *sig, size_t siglen, const u_char *data, size_t datalen)
 {
        char *ktype = NULL;
        int hash_alg, ret = SSH_ERR_INTERNAL_ERROR;
@@ -132,13 +176,13 @@ ssh_rsa_verify(const struct sshkey *key,
            BN_num_bits(key->rsa->n) < SSH_RSA_MINIMUM_MODULUS_SIZE)
                return SSH_ERR_INVALID_ARGUMENT;
 
-       if ((b = sshbuf_from(signature, signaturelen)) == NULL)
+       if ((b = sshbuf_from(sig, siglen)) == NULL)
                return SSH_ERR_ALLOC_FAIL;
        if (sshbuf_get_cstring(b, &ktype, NULL) != 0) {
                ret = SSH_ERR_INVALID_FORMAT;
                goto out;
        }
-       if (strcmp("ssh-rsa", ktype) != 0) {
+       if ((hash_alg = rsa_hash_alg_from_ident(ktype)) == -1) {
                ret = SSH_ERR_KEY_TYPE_MISMATCH;
                goto out;
        }
@@ -167,7 +211,6 @@ ssh_rsa_verify(const struct sshkey *key,
                explicit_bzero(sigblob, diff);
                len = modlen;
        }
-       hash_alg = SSH_DIGEST_SHA1;
        if ((dlen = ssh_digest_bytes(hash_alg)) == 0) {
                ret = SSH_ERR_INTERNAL_ERROR;
                goto out;
@@ -196,6 +239,7 @@ ssh_rsa_verify(const struct sshkey *key,
  * http://www.rsasecurity.com/rsalabs/pkcs/pkcs-1/
  * ftp://ftp.rsasecurity.com/pub/pkcs/pkcs-1/pkcs-1v2-1.asn
  */
+
 /*
  * id-sha1 OBJECT IDENTIFIER ::= { iso(1) identified-organization(3)
  *     oiw(14) secsig(3) algorithms(2) 26 }
@@ -209,6 +253,58 @@ static const u_char id_sha1[] = {
        0x04, 0x14  /* Octet string, length 0x14 (20), followed by sha1 hash */
 };
 
+/*
+ * See http://csrc.nist.gov/groups/ST/crypto_apps_infra/csor/algorithms.html
+ * id-sha256 OBJECT IDENTIFIER ::= { joint-iso-itu-t(2) country(16) us(840)
+ *      organization(1) gov(101) csor(3) nistAlgorithm(4) hashAlgs(2)
+ *      id-sha256(1) }
+ */
+static const u_char id_sha256[] = {
+       0x30, 0x31, /* type Sequence, length 0x31 (49) */
+       0x30, 0x0d, /* type Sequence, length 0x0d (13) */
+       0x06, 0x09, /* type OID, length 0x09 */
+       0x60, 0x86, 0x48, 0x01, 0x65, 0x03, 0x04, 0x02, 0x01, /* id-sha256 */
+       0x05, 0x00, /* NULL */
+       0x04, 0x20  /* Octet string, length 0x20 (32), followed by sha256 hash */
+};
+
+/*
+ * See http://csrc.nist.gov/groups/ST/crypto_apps_infra/csor/algorithms.html
+ * id-sha512 OBJECT IDENTIFIER ::= { joint-iso-itu-t(2) country(16) us(840)
+ *      organization(1) gov(101) csor(3) nistAlgorithm(4) hashAlgs(2)
+ *      id-sha256(3) }
+ */
+static const u_char id_sha512[] = {
+       0x30, 0x51, /* type Sequence, length 0x51 (81) */
+       0x30, 0x0d, /* type Sequence, length 0x0d (13) */
+       0x06, 0x09, /* type OID, length 0x09 */
+       0x60, 0x86, 0x48, 0x01, 0x65, 0x03, 0x04, 0x02, 0x03, /* id-sha512 */
+       0x05, 0x00, /* NULL */
+       0x04, 0x40  /* Octet string, length 0x40 (64), followed by sha512 hash */
+};
+
+static int
+rsa_hash_alg_oid(int hash_alg, const u_char **oidp, size_t *oidlenp)
+{
+       switch (hash_alg) {
+       case SSH_DIGEST_SHA1:
+               *oidp = id_sha1;
+               *oidlenp = sizeof(id_sha1);
+               break;
+       case SSH_DIGEST_SHA256:
+               *oidp = id_sha256;
+               *oidlenp = sizeof(id_sha256);
+               break;
+       case SSH_DIGEST_SHA512:
+               *oidp = id_sha512;
+               *oidlenp = sizeof(id_sha512);
+               break;
+       default:
+               return SSH_ERR_INVALID_ARGUMENT;
+       }
+       return 0;
+}
+
 static int
 openssh_RSA_verify(int hash_alg, u_char *hash, size_t hashlen,
     u_char *sigbuf, size_t siglen, RSA *rsa)
@@ -218,16 +314,10 @@ openssh_RSA_verify(int hash_alg, u_char *hash, size_t hashlen,
        const u_char *oid = NULL;
        u_char *decrypted = NULL;
 
+       if ((ret = rsa_hash_alg_oid(hash_alg, &oid, &oidlen)) != 0)
+               return ret;
        ret = SSH_ERR_INTERNAL_ERROR;
-       switch (hash_alg) {
-       case SSH_DIGEST_SHA1:
-               oid = id_sha1;
-               oidlen = sizeof(id_sha1);
-               hlen = 20;
-               break;
-       default:
-               goto done;
-       }
+       hlen = ssh_digest_bytes(hash_alg);
        if (hashlen != hlen) {
                ret = SSH_ERR_INVALID_ARGUMENT;
                goto done;
diff --git a/ssh2.h b/ssh2.h
index 59417e6121d1cb905536a62e2891207fc1e1ed95..bdff6c5bdb5187472ea8529d5d25f64c49bfa53a 100644 (file)
--- a/ssh2.h
+++ b/ssh2.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh2.h,v 1.15 2014/01/29 06:18:35 djm Exp $ */
+/* $OpenBSD: ssh2.h,v 1.16 2015/12/04 16:41:28 markus Exp $ */
 
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
@@ -80,6 +80,7 @@
 #define SSH2_MSG_DEBUG                                 4
 #define SSH2_MSG_SERVICE_REQUEST                       5
 #define SSH2_MSG_SERVICE_ACCEPT                                6
+#define SSH2_MSG_EXT_INFO                              7
 
 /* transport layer: alg negotiation */
 
index 6c712584f49e406aa6399b354d3271d05a7663fa..f544f006b20f1082455c6dc71191423e0f61bee2 100644 (file)
--- a/ssh_api.c
+++ b/ssh_api.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh_api.c,v 1.4 2015/02/16 22:13:32 djm Exp $ */
+/* $OpenBSD: ssh_api.c,v 1.5 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2012 Markus Friedl.  All rights reserved.
  *
@@ -40,8 +40,8 @@ int   _ssh_order_hostkeyalgs(struct ssh *);
 int    _ssh_verify_host_key(struct sshkey *, struct ssh *);
 struct sshkey *_ssh_host_public_key(int, int, struct ssh *);
 struct sshkey *_ssh_host_private_key(int, int, struct ssh *);
-int    _ssh_host_key_sign(struct sshkey *, struct sshkey *, u_char **,
-    size_t *, const u_char *, size_t, u_int);
+int    _ssh_host_key_sign(struct sshkey *, struct sshkey *,
+    u_char **, size_t *, const u_char *, size_t, const char *, u_int);
 
 /*
  * stubs for the server side implementation of kex.
@@ -49,7 +49,7 @@ int   _ssh_host_key_sign(struct sshkey *, struct sshkey *, u_char **,
  */
 int    use_privsep = 0;
 int    mm_sshkey_sign(struct sshkey *, u_char **, u_int *,
-    u_char *, u_int, u_int);
+    u_char *, u_int, char *, u_int);
 DH     *mm_choose_dh(int, int, int);
 
 /* Define these two variables here so that they are part of the library */
@@ -58,7 +58,7 @@ u_int session_id2_len = 0;
 
 int
 mm_sshkey_sign(struct sshkey *key, u_char **sigp, u_int *lenp,
-    u_char *data, u_int datalen, u_int compat)
+    u_char *data, u_int datalen, char *alg, u_int compat)
 {
        return (-1);
 }
@@ -530,8 +530,8 @@ _ssh_order_hostkeyalgs(struct ssh *ssh)
 
 int
 _ssh_host_key_sign(struct sshkey *privkey, struct sshkey *pubkey,
-    u_char **signature, size_t *slen,
-    const u_char *data, size_t dlen, u_int compat)
+    u_char **signature, size_t *slen, const u_char *data, size_t dlen,
+    const char *alg, u_int compat)
 {
-       return sshkey_sign(privkey, signature, slen, data, dlen, compat);
+       return sshkey_sign(privkey, signature, slen, data, dlen, alg, compat);
 }
index d29f630f2d2f81f8c2e1e24c85cc35cfbea5a5d7..da1bd384787ac64db85cd10fb8057aefce15895b 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: sshconnect2.c,v 1.230 2015/12/04 00:24:55 djm Exp $ */
+/* $OpenBSD: sshconnect2.c,v 1.231 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
  * Copyright (c) 2008 Damien Miller.  All rights reserved.
@@ -157,14 +157,16 @@ void
 ssh_kex2(char *host, struct sockaddr *hostaddr, u_short port)
 {
        char *myproposal[PROPOSAL_MAX] = { KEX_CLIENT };
+       char *s;
        struct kex *kex;
        int r;
 
        xxx_host = host;
        xxx_hostaddr = hostaddr;
 
-       myproposal[PROPOSAL_KEX_ALGS] = compat_kex_proposal(
-           options.kex_algorithms);
+       if ((s = kex_names_cat(options.kex_algorithms, "ext-info-c")) == NULL)
+               fatal("%s: kex_names_cat", __func__);
+       myproposal[PROPOSAL_KEX_ALGS] = compat_kex_proposal(s);
        myproposal[PROPOSAL_ENC_ALGS_CTOS] =
            compat_cipher_proposal(options.ciphers);
        myproposal[PROPOSAL_ENC_ALGS_STOC] =
@@ -221,6 +223,11 @@ ssh_kex2(char *host, struct sockaddr *hostaddr, u_short port)
                debug("Roaming not allowed by server");
                options.use_roaming = 0;
        }
+       /* remove ext-info from the KEX proposals for rekeying */
+       myproposal[PROPOSAL_KEX_ALGS] =
+           compat_kex_proposal(options.kex_algorithms);
+       if ((r = kex_prop2buf(kex->my, myproposal)) != 0)
+               fatal("kex_prop2buf: %s", ssh_err(r));
 
        session_id2 = kex->session_id;
        session_id2_len = kex->session_id_len;
@@ -284,6 +291,8 @@ struct cauthmethod {
        int     *batch_flag;    /* flag in option struct that disables method */
 };
 
+int    input_userauth_service_accept(int, u_int32_t, void *);
+int    input_userauth_ext_info(int, u_int32_t, void *);
 int    input_userauth_success(int, u_int32_t, void *);
 int    input_userauth_success_unexpected(int, u_int32_t, void *);
 int    input_userauth_failure(int, u_int32_t, void *);
@@ -359,30 +368,12 @@ void
 ssh_userauth2(const char *local_user, const char *server_user, char *host,
     Sensitive *sensitive)
 {
+       struct ssh *ssh = active_state;
        Authctxt authctxt;
-       int type;
+       int r;
 
        if (options.challenge_response_authentication)
                options.kbd_interactive_authentication = 1;
-
-       packet_start(SSH2_MSG_SERVICE_REQUEST);
-       packet_put_cstring("ssh-userauth");
-       packet_send();
-       debug("SSH2_MSG_SERVICE_REQUEST sent");
-       packet_write_wait();
-       type = packet_read();
-       if (type != SSH2_MSG_SERVICE_ACCEPT)
-               fatal("Server denied authentication request: %d", type);
-       if (packet_remaining() > 0) {
-               char *reply = packet_get_string(NULL);
-               debug2("service_accept: %s", reply);
-               free(reply);
-       } else {
-               debug2("buggy server: service_accept w/o service");
-       }
-       packet_check_eom();
-       debug("SSH2_MSG_SERVICE_ACCEPT received");
-
        if (options.preferred_authentications == NULL)
                options.preferred_authentications = authmethods_get();
 
@@ -404,21 +395,63 @@ ssh_userauth2(const char *local_user, const char *server_user, char *host,
        if (authctxt.method == NULL)
                fatal("ssh_userauth2: internal error: cannot send userauth none request");
 
-       /* initial userauth request */
-       userauth_none(&authctxt);
+       if ((r = sshpkt_start(ssh, SSH2_MSG_SERVICE_REQUEST)) != 0 ||
+           (r = sshpkt_put_cstring(ssh, "ssh-userauth")) != 0 ||
+           (r = sshpkt_send(ssh)) != 0)
+               fatal("%s: %s", __func__, ssh_err(r));
 
-       dispatch_init(&input_userauth_error);
-       dispatch_set(SSH2_MSG_USERAUTH_SUCCESS, &input_userauth_success);
-       dispatch_set(SSH2_MSG_USERAUTH_FAILURE, &input_userauth_failure);
-       dispatch_set(SSH2_MSG_USERAUTH_BANNER, &input_userauth_banner);
-       dispatch_run(DISPATCH_BLOCK, &authctxt.success, &authctxt);     /* loop until success */
+       ssh_dispatch_init(ssh, &input_userauth_error);
+       ssh_dispatch_set(ssh, SSH2_MSG_EXT_INFO, &input_userauth_ext_info);
+       ssh_dispatch_set(ssh, SSH2_MSG_SERVICE_ACCEPT, &input_userauth_service_accept);
+       ssh_dispatch_run(ssh, DISPATCH_BLOCK, &authctxt.success, &authctxt);    /* loop until success */
 
        pubkey_cleanup(&authctxt);
-       dispatch_range(SSH2_MSG_USERAUTH_MIN, SSH2_MSG_USERAUTH_MAX, NULL);
+       ssh_dispatch_range(ssh, SSH2_MSG_USERAUTH_MIN, SSH2_MSG_USERAUTH_MAX, NULL);
 
        debug("Authentication succeeded (%s).", authctxt.method->name);
 }
 
+/* ARGSUSED */
+int
+input_userauth_service_accept(int type, u_int32_t seqnr, void *ctxt)
+{
+       Authctxt *authctxt = ctxt;
+       struct ssh *ssh = active_state;
+       int r;
+
+       if (ssh_packet_remaining(ssh) > 0) {
+               char *reply;
+
+               if ((r = sshpkt_get_cstring(ssh, &reply, NULL)) != 0)
+                       goto out;
+               debug2("service_accept: %s", reply);
+               free(reply);
+       } else {
+               debug2("buggy server: service_accept w/o service");
+       }
+       if ((r = sshpkt_get_end(ssh)) != 0)
+               goto out;
+       debug("SSH2_MSG_SERVICE_ACCEPT received");
+
+       /* initial userauth request */
+       userauth_none(authctxt);
+
+       ssh_dispatch_set(ssh, SSH2_MSG_EXT_INFO, &input_userauth_error);
+       ssh_dispatch_set(ssh, SSH2_MSG_USERAUTH_SUCCESS, &input_userauth_success);
+       ssh_dispatch_set(ssh, SSH2_MSG_USERAUTH_FAILURE, &input_userauth_failure);
+       ssh_dispatch_set(ssh, SSH2_MSG_USERAUTH_BANNER, &input_userauth_banner);
+       r = 0;
+ out:
+       return r;
+}
+
+/* ARGSUSED */
+int
+input_userauth_ext_info(int type, u_int32_t seqnr, void *ctxt)
+{
+       return kex_input_ext_info(type, seqnr, active_state);
+}
+
 void
 userauth(Authctxt *authctxt, char *authlist)
 {
@@ -970,29 +1003,48 @@ input_userauth_passwd_changereq(int type, u_int32_t seqnr, void *ctxt)
        return 0;
 }
 
+static const char *
+identity_sign_encode(struct identity *id)
+{
+       struct ssh *ssh = active_state;
+
+       if (id->key->type == KEY_RSA) {
+               switch (ssh->kex->rsa_sha2) {
+               case 256:
+                       return "rsa-sha2-256";
+               case 512:
+                       return "rsa-sha2-512";
+               }
+       }
+       return key_ssh_name(id->key);
+}
+
 static int
 identity_sign(struct identity *id, u_char **sigp, size_t *lenp,
     const u_char *data, size_t datalen, u_int compat)
 {
        Key *prv;
        int ret;
+       const char *alg;
+
+       alg = identity_sign_encode(id);
 
        /* the agent supports this key */
        if (id->agent_fd != -1)
                return ssh_agent_sign(id->agent_fd, id->key, sigp, lenp,
-                   data, datalen, compat);
+                   data, datalen, alg, compat);
 
        /*
         * we have already loaded the private key or
         * the private key is stored in external hardware
         */
        if (id->isprivate || (id->key->flags & SSHKEY_FLAG_EXT))
-               return (sshkey_sign(id->key, sigp, lenp, data, datalen,
+               return (sshkey_sign(id->key, sigp, lenp, data, datalen, alg,
                    compat));
        /* load the private key from the file */
        if ((prv = load_identity_file(id)) == NULL)
                return (-1); /* XXX return decent error code */
-       ret = sshkey_sign(prv, sigp, lenp, data, datalen, compat);
+       ret = sshkey_sign(prv, sigp, lenp, data, datalen, alg, compat);
        sshkey_free(prv);
        return (ret);
 }
@@ -1039,7 +1091,7 @@ sign_and_send_pubkey(Authctxt *authctxt, Identity *id)
        } else {
                buffer_put_cstring(&b, authctxt->method->name);
                buffer_put_char(&b, have_sig);
-               buffer_put_cstring(&b, key_ssh_name(id->key));
+               buffer_put_cstring(&b, identity_sign_encode(id));
        }
        buffer_put_string(&b, blob, bloblen);
 
@@ -1139,7 +1191,7 @@ send_pubkey_test(Authctxt *authctxt, Identity *id)
        packet_put_cstring(authctxt->method->name);
        packet_put_char(have_sig);
        if (!(datafellows & SSH_BUG_PKAUTH))
-               packet_put_cstring(key_ssh_name(id->key));
+               packet_put_cstring(identity_sign_encode(id));
        packet_put_string(blob, bloblen);
        free(blob);
        packet_send();
@@ -1743,7 +1795,7 @@ userauth_hostbased(Authctxt *authctxt)
                r = ssh_keysign(private, &sig, &siglen,
                    sshbuf_ptr(b), sshbuf_len(b));
        else if ((r = sshkey_sign(private, &sig, &siglen,
-           sshbuf_ptr(b), sshbuf_len(b), datafellows)) != 0)
+           sshbuf_ptr(b), sshbuf_len(b), NULL, datafellows)) != 0)
                debug("%s: sshkey_sign: %s", __func__, ssh_err(r));
        if (r != 0) {
                error("sign using hostkey %s %s failed",
diff --git a/sshd.c b/sshd.c
index a823999b32bda39c8aa4859efc31a5bf032bdd69..2f3f5b551ee3859b026b070019a96980c5b0dba8 100644 (file)
--- a/sshd.c
+++ b/sshd.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: sshd.c,v 1.460 2015/11/16 22:51:05 djm Exp $ */
+/* $OpenBSD: sshd.c,v 1.461 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -816,6 +816,12 @@ list_hostkey_types(void)
                                buffer_append(&b, ",", 1);
                        p = key_ssh_name(key);
                        buffer_append(&b, p, strlen(p));
+
+                       /* for RSA we also support SHA2 signatures */
+                       if (key->type == KEY_RSA) {
+                               p = ",rsa-sha2-512,rsa-sha2-256";
+                               buffer_append(&b, p, strlen(p));
+                       }
                        break;
                }
                /* If the private key has a cert peer, then list that too */
@@ -2507,24 +2513,26 @@ do_ssh1_kex(void)
 
 int
 sshd_hostkey_sign(Key *privkey, Key *pubkey, u_char **signature, size_t *slen,
-    const u_char *data, size_t dlen, u_int flag)
+    const u_char *data, size_t dlen, const char *alg, u_int flag)
 {
        int r;
        u_int xxx_slen, xxx_dlen = dlen;
 
        if (privkey) {
-               if (PRIVSEP(key_sign(privkey, signature, &xxx_slen, data, xxx_dlen) < 0))
+               if (PRIVSEP(key_sign(privkey, signature, &xxx_slen, data, xxx_dlen,
+                   alg) < 0))
                        fatal("%s: key_sign failed", __func__);
                if (slen)
                        *slen = xxx_slen;
        } else if (use_privsep) {
-               if (mm_key_sign(pubkey, signature, &xxx_slen, data, xxx_dlen) < 0)
+               if (mm_key_sign(pubkey, signature, &xxx_slen, data, xxx_dlen,
+                   alg) < 0)
                        fatal("%s: pubkey_sign failed", __func__);
                if (slen)
                        *slen = xxx_slen;
        } else {
                if ((r = ssh_agent_sign(auth_sock, pubkey, signature, slen,
-                   data, dlen, datafellows)) != 0)
+                   data, dlen, alg, datafellows)) != 0)
                        fatal("%s: ssh_agent_sign failed: %s",
                            __func__, ssh_err(r));
        }
index dc16fe92c21c595fca940aae85e8f4450c87fcf6..587bf5b844084196d15be3ef7561be780002d989 100644 (file)
--- a/sshkey.c
+++ b/sshkey.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: sshkey.c,v 1.27 2015/11/19 01:08:55 djm Exp $ */
+/* $OpenBSD: sshkey.c,v 1.28 2015/12/04 16:41:28 markus Exp $ */
 /*
  * Copyright (c) 2000, 2001 Markus Friedl.  All rights reserved.
  * Copyright (c) 2008 Alexander von Gernler.  All rights reserved.
@@ -83,36 +83,39 @@ struct keytype {
        int type;
        int nid;
        int cert;
+       int sigonly;
 };
 static const struct keytype keytypes[] = {
-       { "ssh-ed25519", "ED25519", KEY_ED25519, 0, 0 },
+       { "ssh-ed25519", "ED25519", KEY_ED25519, 0, 0, 0 },
        { "ssh-ed25519-cert-v01@openssh.com", "ED25519-CERT",
-           KEY_ED25519_CERT, 0, 1 },
+           KEY_ED25519_CERT, 0, 1, 0 },
 #ifdef WITH_OPENSSL
-       { NULL, "RSA1", KEY_RSA1, 0, 0 },
-       { "ssh-rsa", "RSA", KEY_RSA, 0, 0 },
-       { "ssh-dss", "DSA", KEY_DSA, 0, 0 },
+       { NULL, "RSA1", KEY_RSA1, 0, 0, 0 },
+       { "ssh-rsa", "RSA", KEY_RSA, 0, 0, 0 },
+       { "rsa-sha2-256", "RSA", KEY_RSA, 0, 0, 1 },
+       { "rsa-sha2-512", "RSA", KEY_RSA, 0, 0, 1 },
+       { "ssh-dss", "DSA", KEY_DSA, 0, 0, 0 },
 # ifdef OPENSSL_HAS_ECC
-       { "ecdsa-sha2-nistp256", "ECDSA", KEY_ECDSA, NID_X9_62_prime256v1, 0 },
-       { "ecdsa-sha2-nistp384", "ECDSA", KEY_ECDSA, NID_secp384r1, 0 },
+       { "ecdsa-sha2-nistp256", "ECDSA", KEY_ECDSA, NID_X9_62_prime256v1, 0, 0 },
+       { "ecdsa-sha2-nistp384", "ECDSA", KEY_ECDSA, NID_secp384r1, 0, 0 },
 #  ifdef OPENSSL_HAS_NISTP521
-       { "ecdsa-sha2-nistp521", "ECDSA", KEY_ECDSA, NID_secp521r1, 0 },
+       { "ecdsa-sha2-nistp521", "ECDSA", KEY_ECDSA, NID_secp521r1, 0, 0 },
 #  endif /* OPENSSL_HAS_NISTP521 */
 # endif /* OPENSSL_HAS_ECC */
-       { "ssh-rsa-cert-v01@openssh.com", "RSA-CERT", KEY_RSA_CERT, 0, 1 },
-       { "ssh-dss-cert-v01@openssh.com", "DSA-CERT", KEY_DSA_CERT, 0, 1 },
+       { "ssh-rsa-cert-v01@openssh.com", "RSA-CERT", KEY_RSA_CERT, 0, 1, 0 },
+       { "ssh-dss-cert-v01@openssh.com", "DSA-CERT", KEY_DSA_CERT, 0, 1, 0 },
 # ifdef OPENSSL_HAS_ECC
        { "ecdsa-sha2-nistp256-cert-v01@openssh.com", "ECDSA-CERT",
-           KEY_ECDSA_CERT, NID_X9_62_prime256v1, 1 },
+           KEY_ECDSA_CERT, NID_X9_62_prime256v1, 1, 0 },
        { "ecdsa-sha2-nistp384-cert-v01@openssh.com", "ECDSA-CERT",
-           KEY_ECDSA_CERT, NID_secp384r1, 1 },
+           KEY_ECDSA_CERT, NID_secp384r1, 1, 0 },
 #  ifdef OPENSSL_HAS_NISTP521
        { "ecdsa-sha2-nistp521-cert-v01@openssh.com", "ECDSA-CERT",
-           KEY_ECDSA_CERT, NID_secp521r1, 1 },
+           KEY_ECDSA_CERT, NID_secp521r1, 1, 0 },
 #  endif /* OPENSSL_HAS_NISTP521 */
 # endif /* OPENSSL_HAS_ECC */
 #endif /* WITH_OPENSSL */
-       { NULL, NULL, -1, -1, 0 }
+       { NULL, NULL, -1, -1, 0, 0 }
 };
 
 const char *
@@ -200,7 +203,7 @@ key_alg_list(int certs_only, int plain_only)
        const struct keytype *kt;
 
        for (kt = keytypes; kt->type != -1; kt++) {
-               if (kt->name == NULL)
+               if (kt->name == NULL || kt->sigonly)
                        continue;
                if ((certs_only && !kt->cert) || (plain_only && kt->cert))
                        continue;
@@ -2174,7 +2177,7 @@ sshkey_froms(struct sshbuf *buf, struct sshkey **keyp)
 int
 sshkey_sign(const struct sshkey *key,
     u_char **sigp, size_t *lenp,
-    const u_char *data, size_t datalen, u_int compat)
+    const u_char *data, size_t datalen, const char *alg, u_int compat)
 {
        if (sigp != NULL)
                *sigp = NULL;
@@ -2194,7 +2197,7 @@ sshkey_sign(const struct sshkey *key,
 # endif /* OPENSSL_HAS_ECC */
        case KEY_RSA_CERT:
        case KEY_RSA:
-               return ssh_rsa_sign(key, sigp, lenp, data, datalen, compat);
+               return ssh_rsa_sign(key, sigp, lenp, data, datalen, alg);
 #endif /* WITH_OPENSSL */
        case KEY_ED25519:
        case KEY_ED25519_CERT:
@@ -2226,7 +2229,7 @@ sshkey_verify(const struct sshkey *key,
 # endif /* OPENSSL_HAS_ECC */
        case KEY_RSA_CERT:
        case KEY_RSA:
-               return ssh_rsa_verify(key, sig, siglen, data, dlen, compat);
+               return ssh_rsa_verify(key, sig, siglen, data, dlen);
 #endif /* WITH_OPENSSL */
        case KEY_ED25519:
        case KEY_ED25519_CERT:
@@ -2460,7 +2463,7 @@ sshkey_certify(struct sshkey *k, struct sshkey *ca)
 
        /* Sign the whole mess */
        if ((ret = sshkey_sign(ca, &sig_blob, &sig_len, sshbuf_ptr(cert),
-           sshbuf_len(cert), 0)) != 0)
+           sshbuf_len(cert), NULL, 0)) != 0)
                goto out;
 
        /* Append signature and we are done */
index 5a8cccd42e1c76e6cce1d7355f7ab27a1294e7d2..a20a14f9e74bde5b82b7b7ef6d77f63cc0517fb8 100644 (file)
--- a/sshkey.h
+++ b/sshkey.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: sshkey.h,v 1.11 2015/11/19 01:08:55 djm Exp $ */
+/* $OpenBSD: sshkey.h,v 1.12 2015/12/04 16:41:28 markus Exp $ */
 
 /*
  * Copyright (c) 2000, 2001 Markus Friedl.  All rights reserved.
@@ -169,7 +169,7 @@ int  sshkey_plain_to_blob(const struct sshkey *, u_char **, size_t *);
 int     sshkey_putb_plain(const struct sshkey *, struct sshbuf *);
 
 int     sshkey_sign(const struct sshkey *, u_char **, size_t *,
-    const u_char *, size_t, u_int);
+    const u_char *, size_t, const char *, u_int);
 int     sshkey_verify(const struct sshkey *, const u_char *, size_t,
     const u_char *, size_t, u_int);
 
@@ -193,11 +193,11 @@ int       sshkey_parse_private_fileblob_type(struct sshbuf *blob, int type,
     const char *passphrase, struct sshkey **keyp, char **commentp);
 
 #ifdef SSHKEY_INTERNAL
-int ssh_rsa_sign(const struct sshkey *key, u_char **sigp, size_t *lenp,
-    const u_char *data, size_t datalen, u_int compat);
+int ssh_rsa_sign(const struct sshkey *key,
+    u_char **sigp, size_t *lenp, const u_char *data, size_t datalen,
+    const char *ident);
 int ssh_rsa_verify(const struct sshkey *key,
-    const u_char *signature, size_t signaturelen,
-    const u_char *data, size_t datalen, u_int compat);
+    const u_char *sig, size_t siglen, const u_char *data, size_t datalen);
 int ssh_dss_sign(const struct sshkey *key, u_char **sigp, size_t *lenp,
     const u_char *data, size_t datalen, u_int compat);
 int ssh_dss_verify(const struct sshkey *key,