src/usr.bin/ssh/ssh.c - diff

Return to ssh.c CVS log

Up to [local] / src / usr.bin / ssh

Diff for /src/usr.bin/ssh/ssh.c between version 1.485 and 1.486

-version 1.485, 2018/07/16 11:05:41
+version 1.486, 2018/07/16 22:25:01
 Line 1384
 Line 1384
 Line 1384
          sensitive_data.nkeys = 0;
          sensitive_data.keys = NULL;
          if (options.hostbased_authentication) {
-                 sensitive_data.nkeys = 11;
+                 sensitive_data.nkeys = 10;
                  sensitive_data.keys = xcalloc(sensitive_data.nkeys,
                      sizeof(struct sshkey));
                  /* XXX check errors? */
- #define L_KEY(t,p,o) \
+ #define L_PUBKEY(p,o) do { \
-         check_load(sshkey_load_private_type(t, p, "", \
+         if ((o) >= sensitive_data.nkeys) \
-             &(sensitive_data.keys[o]), NULL, NULL), p, "key")
+                 fatal("%s pubkey out of array bounds", __func__); \
- #define L_KEYCERT(t,p,o) \
-         check_load(sshkey_load_private_cert(t, p, "", \
-             &(sensitive_data.keys[o]), NULL), p, "cert and key")
- #define L_PUBKEY(p,o) \
          check_load(sshkey_load_public(p, &(sensitive_data.keys[o]), NULL), \
-             p, "pubkey")
+             p, "pubkey"); \
- #define L_CERT(p,o) \
+ } while (0)
-         check_load(sshkey_load_cert(p, &(sensitive_data.keys[o])), p, "cert")
+ #define L_CERT(p,o) do { \
+         if ((o) >= sensitive_data.nkeys) \
+                 fatal("%s cert out of array bounds", __func__); \
+         check_load(sshkey_load_cert(p, &(sensitive_data.keys[o])), p, "cert"); \
+ } while (0)
                  if (options.hostbased_authentication == 1) {
-                         L_CERT(_PATH_HOST_ECDSA_KEY_FILE, 1);
+                         L_CERT(_PATH_HOST_ECDSA_KEY_FILE, 0);
-                         L_CERT(_PATH_HOST_ED25519_KEY_FILE, 2);
+                         L_CERT(_PATH_HOST_ED25519_KEY_FILE, 1);
-                         L_CERT(_PATH_HOST_RSA_KEY_FILE, 3);
+                         L_CERT(_PATH_HOST_RSA_KEY_FILE, 2);
-                         L_CERT(_PATH_HOST_DSA_KEY_FILE, 4);
+                         L_CERT(_PATH_HOST_DSA_KEY_FILE, 3);
-                         L_PUBKEY(_PATH_HOST_ECDSA_KEY_FILE, 5);
+                         L_PUBKEY(_PATH_HOST_ECDSA_KEY_FILE, 4);
-                         L_PUBKEY(_PATH_HOST_ED25519_KEY_FILE, 6);
+                         L_PUBKEY(_PATH_HOST_ED25519_KEY_FILE, 5);
-                         L_PUBKEY(_PATH_HOST_RSA_KEY_FILE, 7);
+                         L_PUBKEY(_PATH_HOST_RSA_KEY_FILE, 6);
-                         L_PUBKEY(_PATH_HOST_DSA_KEY_FILE, 8);
+                         L_PUBKEY(_PATH_HOST_DSA_KEY_FILE, 7);
-                         L_CERT(_PATH_HOST_XMSS_KEY_FILE, 9);
+                         L_CERT(_PATH_HOST_XMSS_KEY_FILE, 8);
-                         L_PUBKEY(_PATH_HOST_XMSS_KEY_FILE, 10);
+                         L_PUBKEY(_PATH_HOST_XMSS_KEY_FILE, 9);
                  }
          }
          /*