[BACK]Return to ssh-keygen.c CVS log [TXT][DIR] Up to [local] / src / usr.bin / ssh

Diff for /src/usr.bin/ssh/ssh-keygen.c between version 1.406 and 1.421

version 1.406, 2020/04/17 07:16:07 version 1.421, 2020/10/18 11:32:02
Line 127 
Line 127 
 static char *certflags_src_addr = NULL;  static char *certflags_src_addr = NULL;
   
 /* Arbitrary extensions specified by user */  /* Arbitrary extensions specified by user */
 struct cert_userext {  struct cert_ext {
         char *key;          char *key;
         char *val;          char *val;
         int crit;          int crit;
 };  };
 static struct cert_userext *cert_userext;  static struct cert_ext *cert_ext;
 static size_t ncert_userext;  static size_t ncert_ext;
   
 /* Conversion to/from various formats */  /* Conversion to/from various formats */
 enum {  enum {
Line 309 
Line 309 
         if ((r = sshkey_load_private(filename, "", &prv, commentp)) == 0)          if ((r = sshkey_load_private(filename, "", &prv, commentp)) == 0)
                 return prv;                  return prv;
         if (r != SSH_ERR_KEY_WRONG_PASSPHRASE)          if (r != SSH_ERR_KEY_WRONG_PASSPHRASE)
                 fatal("Load key \"%s\": %s", filename, ssh_err(r));                  fatal_r(r, "Load key \"%s\"", filename);
         if (identity_passphrase)          if (identity_passphrase)
                 pass = xstrdup(identity_passphrase);                  pass = xstrdup(identity_passphrase);
         else          else
Line 317 
Line 317 
         r = sshkey_load_private(filename, pass, &prv, commentp);          r = sshkey_load_private(filename, pass, &prv, commentp);
         freezero(pass, strlen(pass));          freezero(pass, strlen(pass));
         if (r != 0)          if (r != 0)
                 fatal("Load key \"%s\": %s", filename, ssh_err(r));                  fatal_r(r, "Load key \"%s\"", filename);
         return prv;          return prv;
 }  }
   
Line 335 
Line 335 
         int r;          int r;
   
         if ((b = sshbuf_new()) == NULL)          if ((b = sshbuf_new()) == NULL)
                 fatal("%s: sshbuf_new failed", __func__);                  fatal_f("sshbuf_new failed");
         if ((r = sshkey_putb(k, b)) != 0)          if ((r = sshkey_putb(k, b)) != 0)
                 fatal("key_to_blob failed: %s", ssh_err(r));                  fatal_fr(r, "put key");
         if ((b64 = sshbuf_dtob64_string(b, 1)) == NULL)          if ((b64 = sshbuf_dtob64_string(b, 1)) == NULL)
                 fatal("%s: sshbuf_dtob64_string failed", __func__);                  fatal_f("sshbuf_dtob64_string failed");
   
         /* Comment + surrounds must fit into 72 chars (RFC 4716 sec 3.3) */          /* Comment + surrounds must fit into 72 chars (RFC 4716 sec 3.3) */
         snprintf(comment, sizeof(comment),          snprintf(comment, sizeof(comment),
Line 374 
Line 374 
                         fatal("PEM_write_EC_PUBKEY failed");                          fatal("PEM_write_EC_PUBKEY failed");
                 break;                  break;
         default:          default:
                 fatal("%s: unsupported key type %s", __func__, sshkey_type(k));                  fatal_f("unsupported key type %s", sshkey_type(k));
         }          }
         exit(0);          exit(0);
 }  }
Line 396 
Line 396 
                         fatal("PEM_write_EC_PUBKEY failed");                          fatal("PEM_write_EC_PUBKEY failed");
                 break;                  break;
         default:          default:
                 fatal("%s: unsupported key type %s", __func__, sshkey_type(k));                  fatal_f("unsupported key type %s", sshkey_type(k));
         }          }
         exit(0);          exit(0);
 }  }
Line 425 
Line 425 
                 do_convert_to_pem(k);                  do_convert_to_pem(k);
                 break;                  break;
         default:          default:
                 fatal("%s: unknown key format %d", __func__, convert_format);                  fatal_f("unknown key format %d", convert_format);
         }          }
         exit(0);          exit(0);
 }  }
Line 441 
Line 441 
         int r;          int r;
   
         if ((r = sshbuf_get_u32(b, &bignum_bits)) != 0)          if ((r = sshbuf_get_u32(b, &bignum_bits)) != 0)
                 fatal("%s: buffer error: %s", __func__, ssh_err(r));                  fatal_fr(r, "parse");
         bytes = (bignum_bits + 7) / 8;          bytes = (bignum_bits + 7) / 8;
         if (sshbuf_len(b) < bytes)          if (sshbuf_len(b) < bytes)
                 fatal("%s: input buffer too small: need %d have %zu",                  fatal_f("input buffer too small: need %d have %zu",
                     __func__, bytes, sshbuf_len(b));                      bytes, sshbuf_len(b));
         if (BN_bin2bn(sshbuf_ptr(b), bytes, value) == NULL)          if (BN_bin2bn(sshbuf_ptr(b), bytes, value) == NULL)
                 fatal("%s: BN_bin2bn failed", __func__);                  fatal_f("BN_bin2bn failed");
         if ((r = sshbuf_consume(b, bytes)) != 0)          if ((r = sshbuf_consume(b, bytes)) != 0)
                 fatal("%s: buffer error: %s", __func__, ssh_err(r));                  fatal_fr(r, "consume");
 }  }
   
 static struct sshkey *  static struct sshkey *
Line 468 
Line 468 
         BIGNUM *rsa_p = NULL, *rsa_q = NULL, *rsa_iqmp = NULL;          BIGNUM *rsa_p = NULL, *rsa_q = NULL, *rsa_iqmp = NULL;
   
         if ((r = sshbuf_get_u32(b, &magic)) != 0)          if ((r = sshbuf_get_u32(b, &magic)) != 0)
                 fatal("%s: buffer error: %s", __func__, ssh_err(r));                  fatal_fr(r, "parse magic");
   
         if (magic != SSH_COM_PRIVATE_KEY_MAGIC) {          if (magic != SSH_COM_PRIVATE_KEY_MAGIC) {
                 error("bad magic 0x%x != 0x%x", magic,                  error("bad magic 0x%x != 0x%x", magic,
Line 481 
Line 481 
             (r = sshbuf_get_u32(b, &i2)) != 0 ||              (r = sshbuf_get_u32(b, &i2)) != 0 ||
             (r = sshbuf_get_u32(b, &i3)) != 0 ||              (r = sshbuf_get_u32(b, &i3)) != 0 ||
             (r = sshbuf_get_u32(b, &i4)) != 0)              (r = sshbuf_get_u32(b, &i4)) != 0)
                 fatal("%s: buffer error: %s", __func__, ssh_err(r));                  fatal_fr(r, "parse");
         debug("ignore (%d %d %d %d)", i1, i2, i3, i4);          debug("ignore (%d %d %d %d)", i1, i2, i3, i4);
         if (strcmp(cipher, "none") != 0) {          if (strcmp(cipher, "none") != 0) {
                 error("unsupported cipher %s", cipher);                  error("unsupported cipher %s", cipher);
Line 510 
Line 510 
                     (dsa_g = BN_new()) == NULL ||                      (dsa_g = BN_new()) == NULL ||
                     (dsa_pub_key = BN_new()) == NULL ||                      (dsa_pub_key = BN_new()) == NULL ||
                     (dsa_priv_key = BN_new()) == NULL)                      (dsa_priv_key = BN_new()) == NULL)
                         fatal("%s: BN_new", __func__);                          fatal_f("BN_new");
                 buffer_get_bignum_bits(b, dsa_p);                  buffer_get_bignum_bits(b, dsa_p);
                 buffer_get_bignum_bits(b, dsa_g);                  buffer_get_bignum_bits(b, dsa_g);
                 buffer_get_bignum_bits(b, dsa_q);                  buffer_get_bignum_bits(b, dsa_q);
                 buffer_get_bignum_bits(b, dsa_pub_key);                  buffer_get_bignum_bits(b, dsa_pub_key);
                 buffer_get_bignum_bits(b, dsa_priv_key);                  buffer_get_bignum_bits(b, dsa_priv_key);
                 if (!DSA_set0_pqg(key->dsa, dsa_p, dsa_q, dsa_g))                  if (!DSA_set0_pqg(key->dsa, dsa_p, dsa_q, dsa_g))
                         fatal("%s: DSA_set0_pqg failed", __func__);                          fatal_f("DSA_set0_pqg failed");
                 dsa_p = dsa_q = dsa_g = NULL; /* transferred */                  dsa_p = dsa_q = dsa_g = NULL; /* transferred */
                 if (!DSA_set0_key(key->dsa, dsa_pub_key, dsa_priv_key))                  if (!DSA_set0_key(key->dsa, dsa_pub_key, dsa_priv_key))
                         fatal("%s: DSA_set0_key failed", __func__);                          fatal_f("DSA_set0_key failed");
                 dsa_pub_key = dsa_priv_key = NULL; /* transferred */                  dsa_pub_key = dsa_priv_key = NULL; /* transferred */
                 break;                  break;
         case KEY_RSA:          case KEY_RSA:
                 if ((r = sshbuf_get_u8(b, &e1)) != 0 ||                  if ((r = sshbuf_get_u8(b, &e1)) != 0 ||
                     (e1 < 30 && (r = sshbuf_get_u8(b, &e2)) != 0) ||                      (e1 < 30 && (r = sshbuf_get_u8(b, &e2)) != 0) ||
                     (e1 < 30 && (r = sshbuf_get_u8(b, &e3)) != 0))                      (e1 < 30 && (r = sshbuf_get_u8(b, &e3)) != 0))
                         fatal("%s: buffer error: %s", __func__, ssh_err(r));                          fatal_fr(r, "parse RSA");
                 e = e1;                  e = e1;
                 debug("e %lx", e);                  debug("e %lx", e);
                 if (e < 30) {                  if (e < 30) {
Line 539 
Line 539 
                         debug("e %lx", e);                          debug("e %lx", e);
                 }                  }
                 if ((rsa_e = BN_new()) == NULL)                  if ((rsa_e = BN_new()) == NULL)
                         fatal("%s: BN_new", __func__);                          fatal_f("BN_new");
                 if (!BN_set_word(rsa_e, e)) {                  if (!BN_set_word(rsa_e, e)) {
                         BN_clear_free(rsa_e);                          BN_clear_free(rsa_e);
                         sshkey_free(key);                          sshkey_free(key);
Line 550 
Line 550 
                     (rsa_p = BN_new()) == NULL ||                      (rsa_p = BN_new()) == NULL ||
                     (rsa_q = BN_new()) == NULL ||                      (rsa_q = BN_new()) == NULL ||
                     (rsa_iqmp = BN_new()) == NULL)                      (rsa_iqmp = BN_new()) == NULL)
                         fatal("%s: BN_new", __func__);                          fatal_f("BN_new");
                 buffer_get_bignum_bits(b, rsa_d);                  buffer_get_bignum_bits(b, rsa_d);
                 buffer_get_bignum_bits(b, rsa_n);                  buffer_get_bignum_bits(b, rsa_n);
                 buffer_get_bignum_bits(b, rsa_iqmp);                  buffer_get_bignum_bits(b, rsa_iqmp);
                 buffer_get_bignum_bits(b, rsa_q);                  buffer_get_bignum_bits(b, rsa_q);
                 buffer_get_bignum_bits(b, rsa_p);                  buffer_get_bignum_bits(b, rsa_p);
                 if (!RSA_set0_key(key->rsa, rsa_n, rsa_e, rsa_d))                  if (!RSA_set0_key(key->rsa, rsa_n, rsa_e, rsa_d))
                         fatal("%s: RSA_set0_key failed", __func__);                          fatal_f("RSA_set0_key failed");
                 rsa_n = rsa_e = rsa_d = NULL; /* transferred */                  rsa_n = rsa_e = rsa_d = NULL; /* transferred */
                 if (!RSA_set0_factors(key->rsa, rsa_p, rsa_q))                  if (!RSA_set0_factors(key->rsa, rsa_p, rsa_q))
                         fatal("%s: RSA_set0_factors failed", __func__);                          fatal_f("RSA_set0_factors failed");
                 rsa_p = rsa_q = NULL; /* transferred */                  rsa_p = rsa_q = NULL; /* transferred */
                 if ((r = ssh_rsa_complete_crt_parameters(key, rsa_iqmp)) != 0)                  if ((r = ssh_rsa_complete_crt_parameters(key, rsa_iqmp)) != 0)
                         fatal("generate RSA parameters failed: %s", ssh_err(r));                          fatal_fr(r, "generate RSA parameters");
                 BN_clear_free(rsa_iqmp);                  BN_clear_free(rsa_iqmp);
                 break;                  break;
         }          }
         rlen = sshbuf_len(b);          rlen = sshbuf_len(b);
         if (rlen != 0)          if (rlen != 0)
                 error("%s: remaining bytes in key blob %d", __func__, rlen);                  error_f("remaining bytes in key blob %d", rlen);
   
         /* try the key */          /* try the key */
         if (sshkey_sign(key, &sig, &slen, data, sizeof(data),          if (sshkey_sign(key, &sig, &slen, data, sizeof(data),
             NULL, NULL, 0) != 0 ||              NULL, NULL, NULL, 0) != 0 ||
             sshkey_verify(key, sig, slen, data, sizeof(data),              sshkey_verify(key, sig, slen, data, sizeof(data),
             NULL, 0, NULL) != 0) {              NULL, 0, NULL) != 0) {
                 sshkey_free(key);                  sshkey_free(key);
Line 652 
Line 652 
             (encoded[len-3] == '='))              (encoded[len-3] == '='))
                 encoded[len-3] = '\0';                  encoded[len-3] = '\0';
         if ((r = sshbuf_b64tod(buf, encoded)) != 0)          if ((r = sshbuf_b64tod(buf, encoded)) != 0)
                 fatal("%s: base64 decoding failed: %s", __func__, ssh_err(r));                  fatal_fr(r, "base64 decode");
         if (*private)          if (*private) {
                 *k = do_convert_private_ssh2(buf);                  if ((*k = do_convert_private_ssh2(buf)) == NULL)
         else if ((r = sshkey_fromb(buf, k)) != 0)                          fatal_f("private key conversion failed");
                 fatal("decode blob failed: %s", ssh_err(r));          } else if ((r = sshkey_fromb(buf, k)) != 0)
                   fatal_fr(r, "parse key");
         sshbuf_free(buf);          sshbuf_free(buf);
         fclose(fp);          fclose(fp);
 }  }
Line 670 
Line 671 
         if ((fp = fopen(identity_file, "r")) == NULL)          if ((fp = fopen(identity_file, "r")) == NULL)
                 fatal("%s: %s: %s", __progname, identity_file, strerror(errno));                  fatal("%s: %s: %s", __progname, identity_file, strerror(errno));
         if ((pubkey = PEM_read_PUBKEY(fp, NULL, NULL, NULL)) == NULL) {          if ((pubkey = PEM_read_PUBKEY(fp, NULL, NULL, NULL)) == NULL) {
                 fatal("%s: %s is not a recognised public key format", __func__,                  fatal_f("%s is not a recognised public key format",
                     identity_file);                      identity_file);
         }          }
         fclose(fp);          fclose(fp);
Line 695 
Line 696 
                 (*k)->ecdsa_nid = sshkey_ecdsa_key_to_nid((*k)->ecdsa);                  (*k)->ecdsa_nid = sshkey_ecdsa_key_to_nid((*k)->ecdsa);
                 break;                  break;
         default:          default:
                 fatal("%s: unsupported pubkey type %d", __func__,                  fatal_f("unsupported pubkey type %d",
                     EVP_PKEY_base_id(pubkey));                      EVP_PKEY_base_id(pubkey));
         }          }
         EVP_PKEY_free(pubkey);          EVP_PKEY_free(pubkey);
Line 718 
Line 719 
                 fclose(fp);                  fclose(fp);
                 return;                  return;
         }          }
         fatal("%s: unrecognised raw private key format", __func__);          fatal_f("unrecognised raw private key format");
 }  }
   
 static void  static void
Line 744 
Line 745 
                 do_convert_from_pem(&k, &private);                  do_convert_from_pem(&k, &private);
                 break;                  break;
         default:          default:
                 fatal("%s: unknown key format %d", __func__, convert_format);                  fatal_f("unknown key format %d", convert_format);
         }          }
   
         if (!private) {          if (!private) {
Line 767 
Line 768 
                             NULL, 0, NULL, NULL);                              NULL, 0, NULL, NULL);
                         break;                          break;
                 default:                  default:
                         fatal("%s: unsupported key type %s", __func__,                          fatal_f("unsupported key type %s", sshkey_type(k));
                             sshkey_type(k));  
                 }                  }
         }          }
   
Line 793 
Line 793 
                 fatal("%s: %s", identity_file, strerror(errno));                  fatal("%s: %s", identity_file, strerror(errno));
         prv = load_identity(identity_file, &comment);          prv = load_identity(identity_file, &comment);
         if ((r = sshkey_write(prv, stdout)) != 0)          if ((r = sshkey_write(prv, stdout)) != 0)
                 error("sshkey_write failed: %s", ssh_err(r));                  fatal_fr(r, "write key");
         sshkey_free(prv);  
         if (comment != NULL && *comment != '\0')          if (comment != NULL && *comment != '\0')
                 fprintf(stdout, " %s", comment);                  fprintf(stdout, " %s", comment);
         fprintf(stdout, "\n");          fprintf(stdout, "\n");
           if (sshkey_is_sk(prv)) {
                   debug("sk_application: \"%s\", sk_flags 0x%02x",
                           prv->sk_application, prv->sk_flags);
           }
           sshkey_free(prv);
         free(comment);          free(comment);
         exit(0);          exit(0);
 }  }
Line 825 
Line 829 
                         ra = sshkey_fingerprint(keys[i], fingerprint_hash,                          ra = sshkey_fingerprint(keys[i], fingerprint_hash,
                             SSH_FP_RANDOMART);                              SSH_FP_RANDOMART);
                         if (fp == NULL || ra == NULL)                          if (fp == NULL || ra == NULL)
                                 fatal("%s: sshkey_fingerprint fail", __func__);                                  fatal_f("sshkey_fingerprint fail");
                         printf("%u %s %s (PKCS11 key)\n", sshkey_size(keys[i]),                          printf("%u %s %s (PKCS11 key)\n", sshkey_size(keys[i]),
                             fp, sshkey_type(keys[i]));                              fp, sshkey_type(keys[i]));
                         if (log_level_get() >= SYSLOG_LEVEL_VERBOSE)                          if (log_level_get() >= SYSLOG_LEVEL_VERBOSE)
Line 876 
Line 880 
         fp = sshkey_fingerprint(public, fptype, rep);          fp = sshkey_fingerprint(public, fptype, rep);
         ra = sshkey_fingerprint(public, fingerprint_hash, SSH_FP_RANDOMART);          ra = sshkey_fingerprint(public, fingerprint_hash, SSH_FP_RANDOMART);
         if (fp == NULL || ra == NULL)          if (fp == NULL || ra == NULL)
                 fatal("%s: sshkey_fingerprint failed", __func__);                  fatal_f("sshkey_fingerprint failed");
         mprintf("%u %s %s (%s)\n", sshkey_size(public), fp,          mprintf("%u %s %s (%s)\n", sshkey_size(public), fp,
             comment ? comment : "no comment", sshkey_type(public));              comment ? comment : "no comment", sshkey_type(public));
         if (log_level_get() >= SYSLOG_LEVEL_VERBOSE)          if (log_level_get() >= SYSLOG_LEVEL_VERBOSE)
Line 890 
Line 894 
 {  {
         struct stat st;          struct stat st;
         char *comment = NULL;          char *comment = NULL;
         struct sshkey *key = NULL;          struct sshkey *privkey = NULL, *pubkey = NULL;
         int r;          int r;
   
         if (stat(identity_file, &st) == -1)          if (stat(identity_file, &st) == -1)
                 fatal("%s: %s", path, strerror(errno));                  fatal("%s: %s", path, strerror(errno));
         if ((r = sshkey_load_private(path, NULL, &key, &comment)) != 0) {          if ((r = sshkey_load_public(path, &pubkey, &comment)) != 0)
                 debug("load private \"%s\": %s", path, ssh_err(r));                  debug_r(r, "load public \"%s\"", path);
                 if ((r = sshkey_load_public(path, &key, &comment)) != 0) {          if (pubkey == NULL || comment == NULL || *comment == '\0') {
                         debug("load public \"%s\": %s", path, ssh_err(r));                  free(comment);
                         fatal("%s is not a key file.", path);                  if ((r = sshkey_load_private(path, NULL,
                 }                      &privkey, &comment)) != 0)
                           debug_r(r, "load private \"%s\"", path);
         }          }
           if (pubkey == NULL && privkey == NULL)
                   fatal("%s is not a key file.", path);
   
         fingerprint_one_key(key, comment);          fingerprint_one_key(pubkey == NULL ? privkey : pubkey, comment);
         sshkey_free(key);          sshkey_free(pubkey);
           sshkey_free(privkey);
         free(comment);          free(comment);
 }  }
   
Line 1030 
Line 1038 
         struct sshkey *private, *public;          struct sshkey *private, *public;
         char comment[1024], *prv_tmp, *pub_tmp, *prv_file, *pub_file;          char comment[1024], *prv_tmp, *pub_tmp, *prv_file, *pub_file;
         int i, type, fd, r;          int i, type, fd, r;
         FILE *f;  
   
         for (i = 0; key_types[i].key_type; i++) {          for (i = 0; key_types[i].key_type; i++) {
                 public = private = NULL;                  public = private = NULL;
Line 1068 
Line 1075 
                 fflush(stdout);                  fflush(stdout);
                 type = sshkey_type_from_name(key_types[i].key_type);                  type = sshkey_type_from_name(key_types[i].key_type);
                 if ((fd = mkstemp(prv_tmp)) == -1) {                  if ((fd = mkstemp(prv_tmp)) == -1) {
                         error("Could not save your public key in %s: %s",                          error("Could not save your private key in %s: %s",
                             prv_tmp, strerror(errno));                              prv_tmp, strerror(errno));
                         goto failnext;                          goto failnext;
                 }                  }
                 close(fd); /* just using mkstemp() to generate/reserve a name */                  (void)close(fd); /* just using mkstemp() to reserve a name */
                 bits = 0;                  bits = 0;
                 type_bits_valid(type, NULL, &bits);                  type_bits_valid(type, NULL, &bits);
                 if ((r = sshkey_generate(type, bits, &private)) != 0) {                  if ((r = sshkey_generate(type, bits, &private)) != 0) {
                         error("sshkey_generate failed: %s", ssh_err(r));                          error_r(r, "sshkey_generate failed");
                         goto failnext;                          goto failnext;
                 }                  }
                 if ((r = sshkey_from_private(private, &public)) != 0)                  if ((r = sshkey_from_private(private, &public)) != 0)
                         fatal("sshkey_from_private failed: %s", ssh_err(r));                          fatal_fr(r, "sshkey_from_private");
                 snprintf(comment, sizeof comment, "%s@%s", pw->pw_name,                  snprintf(comment, sizeof comment, "%s@%s", pw->pw_name,
                     hostname);                      hostname);
                 if ((r = sshkey_save_private(private, prv_tmp, "",                  if ((r = sshkey_save_private(private, prv_tmp, "",
                     comment, private_key_format, openssh_format_cipher,                      comment, private_key_format, openssh_format_cipher,
                     rounds)) != 0) {                      rounds)) != 0) {
                         error("Saving key \"%s\" failed: %s",                          error_r(r, "Saving key \"%s\" failed", prv_tmp);
                             prv_tmp, ssh_err(r));  
                         goto failnext;                          goto failnext;
                 }                  }
                 if ((fd = mkstemp(pub_tmp)) == -1) {                  if ((fd = mkstemp(pub_tmp)) == -1) {
Line 1096 
Line 1102 
                         goto failnext;                          goto failnext;
                 }                  }
                 (void)fchmod(fd, 0644);                  (void)fchmod(fd, 0644);
                 f = fdopen(fd, "w");                  (void)close(fd);
                 if (f == NULL) {                  if ((r = sshkey_save_public(public, pub_tmp, comment)) != 0) {
                         error("fdopen %s failed: %s", pub_tmp, strerror(errno));                          error_r(r, "Unable to save public key to %s",
                         close(fd);                              identity_file);
                         goto failnext;                          goto failnext;
                 }                  }
                 if ((r = sshkey_write(public, f)) != 0) {  
                         error("write key failed: %s", ssh_err(r));  
                         fclose(f);  
                         goto failnext;  
                 }  
                 fprintf(f, " %s\n", comment);  
                 if (ferror(f) != 0) {  
                         error("write key failed: %s", strerror(errno));  
                         fclose(f);  
                         goto failnext;  
                 }  
                 if (fclose(f) != 0) {  
                         error("key close failed: %s", strerror(errno));  
                         goto failnext;  
                 }  
   
                 /* Rename temporary files to their permanent locations. */                  /* Rename temporary files to their permanent locations. */
                 if (rename(pub_tmp, pub_file) != 0) {                  if (rename(pub_tmp, pub_file) != 0) {
Line 1248 
Line 1239 
                                 ra = sshkey_fingerprint(l->key,                                  ra = sshkey_fingerprint(l->key,
                                     fingerprint_hash, SSH_FP_RANDOMART);                                      fingerprint_hash, SSH_FP_RANDOMART);
                                 if (fp == NULL || ra == NULL)                                  if (fp == NULL || ra == NULL)
                                         fatal("%s: sshkey_fingerprint failed",                                          fatal_f("sshkey_fingerprint failed");
                                             __func__);  
                                 mprintf("%s %s %s%s%s\n", ctx->host,                                  mprintf("%s %s %s%s%s\n", ctx->host,
                                     sshkey_type(l->key), fp,                                      sshkey_type(l->key), fp,
                                     l->comment[0] ? " " : "",                                      l->comment[0] ? " " : "",
Line 1281 
Line 1271 
         int r, fd, oerrno, inplace = 0;          int r, fd, oerrno, inplace = 0;
         struct known_hosts_ctx ctx;          struct known_hosts_ctx ctx;
         u_int foreach_options;          u_int foreach_options;
           struct stat sb;
   
         if (!have_identity) {          if (!have_identity) {
                 cp = tilde_expand_filename(_PATH_SSH_USER_HOSTFILE, pw->pw_uid);                  cp = tilde_expand_filename(_PATH_SSH_USER_HOSTFILE, pw->pw_uid);
Line 1290 
Line 1281 
                 free(cp);                  free(cp);
                 have_identity = 1;                  have_identity = 1;
         }          }
           if (stat(identity_file, &sb) != 0)
                   fatal("Cannot stat %s: %s", identity_file, strerror(errno));
   
         memset(&ctx, 0, sizeof(ctx));          memset(&ctx, 0, sizeof(ctx));
         ctx.out = stdout;          ctx.out = stdout;
Line 1316 
Line 1309 
                         unlink(tmp);                          unlink(tmp);
                         fatal("fdopen: %s", strerror(oerrno));                          fatal("fdopen: %s", strerror(oerrno));
                 }                  }
                   fchmod(fd, sb.st_mode & 0644);
                 inplace = 1;                  inplace = 1;
         }          }
         /* XXX support identity_file == "-" for stdin */          /* XXX support identity_file == "-" for stdin */
Line 1326 
Line 1320 
             foreach_options)) != 0) {              foreach_options)) != 0) {
                 if (inplace)                  if (inplace)
                         unlink(tmp);                          unlink(tmp);
                 fatal("%s: hostkeys_foreach failed: %s", __func__, ssh_err(r));                  fatal_fr(r, "hostkeys_foreach");
         }          }
   
         if (inplace)          if (inplace)
Line 1405 
Line 1399 
                         goto badkey;                          goto badkey;
         } else if (r != 0) {          } else if (r != 0) {
  badkey:   badkey:
                 fatal("Failed to load key %s: %s", identity_file, ssh_err(r));                  fatal_r(r, "Failed to load key %s", identity_file);
         }          }
         if (comment)          if (comment)
                 mprintf("Key has comment '%s'\n", comment);                  mprintf("Key has comment '%s'\n", comment);
Line 1437 
Line 1431 
         /* Save the file using the new passphrase. */          /* Save the file using the new passphrase. */
         if ((r = sshkey_save_private(private, identity_file, passphrase1,          if ((r = sshkey_save_private(private, identity_file, passphrase1,
             comment, private_key_format, openssh_format_cipher, rounds)) != 0) {              comment, private_key_format, openssh_format_cipher, rounds)) != 0) {
                 error("Saving key \"%s\" failed: %s.",                  error_r(r, "Saving key \"%s\" failed", identity_file);
                     identity_file, ssh_err(r));  
                 freezero(passphrase1, strlen(passphrase1));                  freezero(passphrase1, strlen(passphrase1));
                 sshkey_free(private);                  sshkey_free(private);
                 free(comment);                  free(comment);
Line 1466 
Line 1459 
         int r;          int r;
   
         if (fname == NULL)          if (fname == NULL)
                 fatal("%s: no filename", __func__);                  fatal_f("no filename");
         if (stat(fname, &st) == -1) {          if (stat(fname, &st) == -1) {
                 if (errno == ENOENT)                  if (errno == ENOENT)
                         return 0;                          return 0;
                 fatal("%s: %s", fname, strerror(errno));                  fatal("%s: %s", fname, strerror(errno));
         }          }
         if ((r = sshkey_load_public(fname, &public, &comment)) != 0)          if ((r = sshkey_load_public(fname, &public, &comment)) != 0)
                 fatal("Failed to read v2 public key from \"%s\": %s.",                  fatal_r(r, "Failed to read v2 public key from \"%s\"", fname);
                     fname, ssh_err(r));  
         export_dns_rr(hname, public, stdout, print_generic);          export_dns_rr(hname, public, stdout, print_generic);
         sshkey_free(public);          sshkey_free(public);
         free(comment);          free(comment);
Line 1491 
Line 1483 
         struct sshkey *private;          struct sshkey *private;
         struct sshkey *public;          struct sshkey *public;
         struct stat st;          struct stat st;
         FILE *f;          int r;
         int r, fd;  
   
         if (!have_identity)          if (!have_identity)
                 ask_filename(pw, "Enter file in which the key is");                  ask_filename(pw, "Enter file in which the key is");
Line 1502 
Line 1493 
             &private, &comment)) == 0)              &private, &comment)) == 0)
                 passphrase = xstrdup("");                  passphrase = xstrdup("");
         else if (r != SSH_ERR_KEY_WRONG_PASSPHRASE)          else if (r != SSH_ERR_KEY_WRONG_PASSPHRASE)
                 fatal("Cannot load private key \"%s\": %s.",                  fatal_r(r, "Cannot load private key \"%s\"", identity_file);
                     identity_file, ssh_err(r));  
         else {          else {
                 if (identity_passphrase)                  if (identity_passphrase)
                         passphrase = xstrdup(identity_passphrase);                          passphrase = xstrdup(identity_passphrase);
Line 1516 
Line 1506 
                 if ((r = sshkey_load_private(identity_file, passphrase,                  if ((r = sshkey_load_private(identity_file, passphrase,
                     &private, &comment)) != 0) {                      &private, &comment)) != 0) {
                         freezero(passphrase, strlen(passphrase));                          freezero(passphrase, strlen(passphrase));
                         fatal("Cannot load private key \"%s\": %s.",                          fatal_r(r, "Cannot load private key \"%s\"",
                             identity_file, ssh_err(r));                              identity_file);
                 }                  }
         }          }
   
Line 1558 
Line 1548 
         if ((r = sshkey_save_private(private, identity_file, passphrase,          if ((r = sshkey_save_private(private, identity_file, passphrase,
             new_comment, private_key_format, openssh_format_cipher,              new_comment, private_key_format, openssh_format_cipher,
             rounds)) != 0) {              rounds)) != 0) {
                 error("Saving key \"%s\" failed: %s",                  error_r(r, "Saving key \"%s\" failed", identity_file);
                     identity_file, ssh_err(r));  
                 freezero(passphrase, strlen(passphrase));                  freezero(passphrase, strlen(passphrase));
                 sshkey_free(private);                  sshkey_free(private);
                 free(comment);                  free(comment);
Line 1567 
Line 1556 
         }          }
         freezero(passphrase, strlen(passphrase));          freezero(passphrase, strlen(passphrase));
         if ((r = sshkey_from_private(private, &public)) != 0)          if ((r = sshkey_from_private(private, &public)) != 0)
                 fatal("sshkey_from_private failed: %s", ssh_err(r));                  fatal_fr(r, "sshkey_from_private");
         sshkey_free(private);          sshkey_free(private);
   
         strlcat(identity_file, ".pub", sizeof(identity_file));          strlcat(identity_file, ".pub", sizeof(identity_file));
         fd = open(identity_file, O_WRONLY | O_CREAT | O_TRUNC, 0644);          if ((r = sshkey_save_public(public, identity_file, new_comment)) != 0)
         if (fd == -1)                  fatal_r(r, "Unable to save public key to %s", identity_file);
                 fatal("Could not save your public key in %s", identity_file);  
         f = fdopen(fd, "w");  
         if (f == NULL)  
                 fatal("fdopen %s failed: %s", identity_file, strerror(errno));  
         if ((r = sshkey_write(public, f)) != 0)  
                 fatal("write key failed: %s", ssh_err(r));  
         sshkey_free(public);          sshkey_free(public);
         fprintf(f, " %s\n", new_comment);  
         fclose(f);  
   
         free(comment);          free(comment);
   
         if (strlen(new_comment) > 0)          if (strlen(new_comment) > 0)
Line 1594 
Line 1574 
 }  }
   
 static void  static void
 add_flag_option(struct sshbuf *c, const char *name)  cert_ext_add(const char *key, const char *value, int iscrit)
 {  {
         int r;          cert_ext = xreallocarray(cert_ext, ncert_ext + 1, sizeof(*cert_ext));
           cert_ext[ncert_ext].key = xstrdup(key);
         debug3("%s: %s", __func__, name);          cert_ext[ncert_ext].val = value == NULL ? NULL : xstrdup(value);
         if ((r = sshbuf_put_cstring(c, name)) != 0 ||          cert_ext[ncert_ext].crit = iscrit;
             (r = sshbuf_put_string(c, NULL, 0)) != 0)          ncert_ext++;
                 fatal("%s: buffer error: %s", __func__, ssh_err(r));  
 }  }
   
 static void  /* qsort(3) comparison function for certificate extensions */
 add_string_option(struct sshbuf *c, const char *name, const char *value)  static int
   cert_ext_cmp(const void *_a, const void *_b)
 {  {
         struct sshbuf *b;          const struct cert_ext *a = (const struct cert_ext *)_a;
           const struct cert_ext *b = (const struct cert_ext *)_b;
         int r;          int r;
   
         debug3("%s: %s=%s", __func__, name, value);          if (a->crit != b->crit)
         if ((b = sshbuf_new()) == NULL)                  return (a->crit < b->crit) ? -1 : 1;
                 fatal("%s: sshbuf_new failed", __func__);          if ((r = strcmp(a->key, b->key)) != 0)
         if ((r = sshbuf_put_cstring(b, value)) != 0 ||                  return r;
             (r = sshbuf_put_cstring(c, name)) != 0 ||          if ((a->val == NULL) != (b->val == NULL))
             (r = sshbuf_put_stringb(c, b)) != 0)                  return (a->val == NULL) ? -1 : 1;
                 fatal("%s: buffer error: %s", __func__, ssh_err(r));          if (a->val != NULL && (r = strcmp(a->val, b->val)) != 0)
                   return r;
         sshbuf_free(b);          return 0;
 }  }
   
 #define OPTIONS_CRITICAL        1  #define OPTIONS_CRITICAL        1
Line 1626 
Line 1607 
 static void  static void
 prepare_options_buf(struct sshbuf *c, int which)  prepare_options_buf(struct sshbuf *c, int which)
 {  {
           struct sshbuf *b;
         size_t i;          size_t i;
           int r;
           const struct cert_ext *ext;
   
           if ((b = sshbuf_new()) == NULL)
                   fatal_f("sshbuf_new failed");
         sshbuf_reset(c);          sshbuf_reset(c);
         if ((which & OPTIONS_CRITICAL) != 0 &&          for (i = 0; i < ncert_ext; i++) {
             certflags_command != NULL)                  ext = &cert_ext[i];
                 add_string_option(c, "force-command", certflags_command);                  if ((ext->crit && (which & OPTIONS_EXTENSIONS)) ||
         if ((which & OPTIONS_EXTENSIONS) != 0 &&                      (!ext->crit && (which & OPTIONS_CRITICAL)))
             (certflags_flags & CERTOPT_X_FWD) != 0)  
                 add_flag_option(c, "permit-X11-forwarding");  
         if ((which & OPTIONS_EXTENSIONS) != 0 &&  
             (certflags_flags & CERTOPT_AGENT_FWD) != 0)  
                 add_flag_option(c, "permit-agent-forwarding");  
         if ((which & OPTIONS_EXTENSIONS) != 0 &&  
             (certflags_flags & CERTOPT_PORT_FWD) != 0)  
                 add_flag_option(c, "permit-port-forwarding");  
         if ((which & OPTIONS_EXTENSIONS) != 0 &&  
             (certflags_flags & CERTOPT_PTY) != 0)  
                 add_flag_option(c, "permit-pty");  
         if ((which & OPTIONS_EXTENSIONS) != 0 &&  
             (certflags_flags & CERTOPT_USER_RC) != 0)  
                 add_flag_option(c, "permit-user-rc");  
         if ((which & OPTIONS_EXTENSIONS) != 0 &&  
             (certflags_flags & CERTOPT_NO_REQUIRE_USER_PRESENCE) != 0)  
                 add_flag_option(c, "no-touch-required");  
         if ((which & OPTIONS_CRITICAL) != 0 &&  
             certflags_src_addr != NULL)  
                 add_string_option(c, "source-address", certflags_src_addr);  
         for (i = 0; i < ncert_userext; i++) {  
                 if ((cert_userext[i].crit && (which & OPTIONS_EXTENSIONS)) ||  
                     (!cert_userext[i].crit && (which & OPTIONS_CRITICAL)))  
                         continue;                          continue;
                 if (cert_userext[i].val == NULL)                  if (ext->val == NULL) {
                         add_flag_option(c, cert_userext[i].key);                          /* flag option */
                 else {                          debug3_f("%s", ext->key);
                         add_string_option(c, cert_userext[i].key,                          if ((r = sshbuf_put_cstring(c, ext->key)) != 0 ||
                             cert_userext[i].val);                              (r = sshbuf_put_string(c, NULL, 0)) != 0)
                                   fatal_fr(r, "prepare flag");
                   } else {
                           /* key/value option */
                           debug3_f("%s=%s", ext->key, ext->val);
                           sshbuf_reset(b);
                           if ((r = sshbuf_put_cstring(c, ext->key)) != 0 ||
                               (r = sshbuf_put_cstring(b, ext->val)) != 0 ||
                               (r = sshbuf_put_stringb(c, b)) != 0)
                                   fatal_fr(r, "prepare k/v");
                 }                  }
         }          }
           sshbuf_free(b);
 }  }
   
   static void
   finalise_cert_exts(void)
   {
           /* critical options */
           if (certflags_command != NULL)
                   cert_ext_add("force-command", certflags_command, 1);
           if (certflags_src_addr != NULL)
                   cert_ext_add("source-address", certflags_src_addr, 1);
           /* extensions */
           if ((certflags_flags & CERTOPT_X_FWD) != 0)
                   cert_ext_add("permit-X11-forwarding", NULL, 0);
           if ((certflags_flags & CERTOPT_AGENT_FWD) != 0)
                   cert_ext_add("permit-agent-forwarding", NULL, 0);
           if ((certflags_flags & CERTOPT_PORT_FWD) != 0)
                   cert_ext_add("permit-port-forwarding", NULL, 0);
           if ((certflags_flags & CERTOPT_PTY) != 0)
                   cert_ext_add("permit-pty", NULL, 0);
           if ((certflags_flags & CERTOPT_USER_RC) != 0)
                   cert_ext_add("permit-user-rc", NULL, 0);
           if ((certflags_flags & CERTOPT_NO_REQUIRE_USER_PRESENCE) != 0)
                   cert_ext_add("no-touch-required", NULL, 0);
           /* order lexically by key */
           if (ncert_ext > 0)
                   qsort(cert_ext, ncert_ext, sizeof(*cert_ext), cert_ext_cmp);
   }
   
 static struct sshkey *  static struct sshkey *
 load_pkcs11_key(char *path)  load_pkcs11_key(char *path)
 {  {
Line 1674 
Line 1673 
         int r, i, nkeys;          int r, i, nkeys;
   
         if ((r = sshkey_load_public(path, &public, NULL)) != 0)          if ((r = sshkey_load_public(path, &public, NULL)) != 0)
                 fatal("Couldn't load CA public key \"%s\": %s",                  fatal_r(r, "Couldn't load CA public key \"%s\"", path);
                     path, ssh_err(r));  
   
         nkeys = pkcs11_add_provider(pkcs11provider, identity_passphrase,          nkeys = pkcs11_add_provider(pkcs11provider, identity_passphrase,
             &keys, NULL);              &keys, NULL);
         debug3("%s: %d keys", __func__, nkeys);          debug3_f("%d keys", nkeys);
         if (nkeys <= 0)          if (nkeys <= 0)
                 fatal("cannot read public key from pkcs11");                  fatal("cannot read public key from pkcs11");
         for (i = 0; i < nkeys; i++) {          for (i = 0; i < nkeys; i++) {
Line 1701 
Line 1699 
 static int  static int
 agent_signer(struct sshkey *key, u_char **sigp, size_t *lenp,  agent_signer(struct sshkey *key, u_char **sigp, size_t *lenp,
     const u_char *data, size_t datalen,      const u_char *data, size_t datalen,
     const char *alg, const char *provider, u_int compat, void *ctx)      const char *alg, const char *provider, const char *pin,
       u_int compat, void *ctx)
 {  {
         int *agent_fdp = (int *)ctx;          int *agent_fdp = (int *)ctx;
   
Line 1714 
Line 1713 
     unsigned long long cert_serial, int cert_serial_autoinc,      unsigned long long cert_serial, int cert_serial_autoinc,
     int argc, char **argv)      int argc, char **argv)
 {  {
         int r, i, fd, found, agent_fd = -1;          int r, i, found, agent_fd = -1;
         u_int n;          u_int n;
         struct sshkey *ca, *public;          struct sshkey *ca, *public;
         char valid[64], *otmp, *tmp, *cp, *out, *comment;          char valid[64], *otmp, *tmp, *cp, *out, *comment;
         char *ca_fp = NULL, **plist = NULL;          char *ca_fp = NULL, **plist = NULL, *pin = NULL;
         FILE *f;  
         struct ssh_identitylist *agent_ids;          struct ssh_identitylist *agent_ids;
         size_t j;          size_t j;
         struct notifier_ctx *notifier = NULL;          struct notifier_ctx *notifier = NULL;
Line 1739 
Line 1737 
                  * agent.                   * agent.
                  */                   */
                 if ((r = sshkey_load_public(tmp, &ca, NULL)) != 0)                  if ((r = sshkey_load_public(tmp, &ca, NULL)) != 0)
                         fatal("Cannot load CA public key %s: %s",                          fatal_r(r, "Cannot load CA public key %s", tmp);
                             tmp, ssh_err(r));  
                 if ((r = ssh_get_authentication_socket(&agent_fd)) != 0)                  if ((r = ssh_get_authentication_socket(&agent_fd)) != 0)
                         fatal("Cannot use public key for CA signature: %s",                          fatal_r(r, "Cannot use public key for CA signature");
                             ssh_err(r));  
                 if ((r = ssh_fetch_identitylist(agent_fd, &agent_ids)) != 0)                  if ((r = ssh_fetch_identitylist(agent_fd, &agent_ids)) != 0)
                         fatal("Retrieve agent key list: %s", ssh_err(r));                          fatal_r(r, "Retrieve agent key list");
                 found = 0;                  found = 0;
                 for (j = 0; j < agent_ids->nkeys; j++) {                  for (j = 0; j < agent_ids->nkeys; j++) {
                         if (sshkey_equal(ca, agent_ids->keys[j])) {                          if (sshkey_equal(ca, agent_ids->keys[j])) {
Line 1760 
Line 1756 
         } else {          } else {
                 /* CA key is assumed to be a private key on the filesystem */                  /* CA key is assumed to be a private key on the filesystem */
                 ca = load_identity(tmp, NULL);                  ca = load_identity(tmp, NULL);
                   if (sshkey_is_sk(ca) &&
                       (ca->sk_flags & SSH_SK_USER_VERIFICATION_REQD)) {
                           if ((pin = read_passphrase("Enter PIN for CA key: ",
                               RP_ALLOW_STDIN)) == NULL)
                                   fatal_f("couldn't read PIN");
                   }
         }          }
         free(tmp);          free(tmp);
   
Line 1774 
Line 1776 
         }          }
         ca_fp = sshkey_fingerprint(ca, fingerprint_hash, SSH_FP_DEFAULT);          ca_fp = sshkey_fingerprint(ca, fingerprint_hash, SSH_FP_DEFAULT);
   
           finalise_cert_exts();
         for (i = 0; i < argc; i++) {          for (i = 0; i < argc; i++) {
                 /* Split list of principals */                  /* Split list of principals */
                 n = 0;                  n = 0;
Line 1792 
Line 1795 
   
                 tmp = tilde_expand_filename(argv[i], pw->pw_uid);                  tmp = tilde_expand_filename(argv[i], pw->pw_uid);
                 if ((r = sshkey_load_public(tmp, &public, &comment)) != 0)                  if ((r = sshkey_load_public(tmp, &public, &comment)) != 0)
                         fatal("%s: unable to open \"%s\": %s",                          fatal_r(r, "load pubkey \"%s\"", tmp);
                             __func__, tmp, ssh_err(r));  
                 if (sshkey_is_cert(public))                  if (sshkey_is_cert(public))
                         fatal("%s: key \"%s\" type %s cannot be certified",                          fatal_f("key \"%s\" type %s cannot be certified",
                             __func__, tmp, sshkey_type(public));                              tmp, sshkey_type(public));
   
                 /* Prepare certificate to sign */                  /* Prepare certificate to sign */
                 if ((r = sshkey_to_certified(public)) != 0)                  if ((r = sshkey_to_certified(public)) != 0)
                         fatal("Could not upgrade key %s to certificate: %s",                          fatal_r(r, "Could not upgrade key %s to certificate", tmp);
                             tmp, ssh_err(r));  
                 public->cert->type = cert_key_type;                  public->cert->type = cert_key_type;
                 public->cert->serial = (u_int64_t)cert_serial;                  public->cert->serial = (u_int64_t)cert_serial;
                 public->cert->key_id = xstrdup(cert_key_id);                  public->cert->key_id = xstrdup(cert_key_id);
Line 1814 
Line 1815 
                     OPTIONS_EXTENSIONS);                      OPTIONS_EXTENSIONS);
                 if ((r = sshkey_from_private(ca,                  if ((r = sshkey_from_private(ca,
                     &public->cert->signature_key)) != 0)                      &public->cert->signature_key)) != 0)
                         fatal("sshkey_from_private (ca key): %s", ssh_err(r));                          fatal_r(r, "sshkey_from_private (ca key)");
   
                 if (agent_fd != -1 && (ca->flags & SSHKEY_FLAG_EXT) != 0) {                  if (agent_fd != -1 && (ca->flags & SSHKEY_FLAG_EXT) != 0) {
                         if ((r = sshkey_certify_custom(public, ca,                          if ((r = sshkey_certify_custom(public, ca,
                             key_type_name, sk_provider, agent_signer,                              key_type_name, sk_provider, NULL, agent_signer,
                             &agent_fd)) != 0)                              &agent_fd)) != 0)
                                 fatal("Couldn't certify key %s via agent: %s",                                  fatal_r(r, "Couldn't certify %s via agent", tmp);
                                     tmp, ssh_err(r));  
                 } else {                  } else {
                         if (sshkey_is_sk(ca) &&                          if (sshkey_is_sk(ca) &&
                             (ca->sk_flags & SSH_SK_USER_PRESENCE_REQD)) {                              (ca->sk_flags & SSH_SK_USER_PRESENCE_REQD)) {
Line 1830 
Line 1830 
                                     sshkey_type(ca), ca_fp);                                      sshkey_type(ca), ca_fp);
                         }                          }
                         r = sshkey_certify(public, ca, key_type_name,                          r = sshkey_certify(public, ca, key_type_name,
                             sk_provider);                              sk_provider, pin);
                         notify_complete(notifier);                          notify_complete(notifier);
                         if (r != 0)                          if (r != 0)
                                 fatal("Couldn't certify key %s: %s",                                  fatal_r(r, "Couldn't certify key %s", tmp);
                                     tmp, ssh_err(r));  
                 }                  }
   
                 if ((cp = strrchr(tmp, '.')) != NULL && strcmp(cp, ".pub") == 0)                  if ((cp = strrchr(tmp, '.')) != NULL && strcmp(cp, ".pub") == 0)
Line 1842 
Line 1841 
                 xasprintf(&out, "%s-cert.pub", tmp);                  xasprintf(&out, "%s-cert.pub", tmp);
                 free(tmp);                  free(tmp);
   
                 if ((fd = open(out, O_WRONLY|O_CREAT|O_TRUNC, 0644)) == -1)                  if ((r = sshkey_save_public(public, out, comment)) != 0) {
                         fatal("Could not open \"%s\" for writing: %s", out,                          fatal_r(r, "Unable to save public key to %s",
                             strerror(errno));                              identity_file);
                 if ((f = fdopen(fd, "w")) == NULL)                  }
                         fatal("%s: fdopen: %s", __func__, strerror(errno));  
                 if ((r = sshkey_write(public, f)) != 0)  
                         fatal("Could not write certified key to %s: %s",  
                             out, ssh_err(r));  
                 fprintf(f, " %s\n", comment);  
                 fclose(f);  
   
                 if (!quiet) {                  if (!quiet) {
                         sshkey_format_cert_validity(public->cert,                          sshkey_format_cert_validity(public->cert,
Line 1870 
Line 1863 
                 if (cert_serial_autoinc)                  if (cert_serial_autoinc)
                         cert_serial++;                          cert_serial++;
         }          }
           if (pin != NULL)
                   freezero(pin, strlen(pin));
         free(ca_fp);          free(ca_fp);
 #ifdef ENABLE_PKCS11  #ifdef ENABLE_PKCS11
         pkcs11_terminate();          pkcs11_terminate();
Line 1994 
Line 1989 
                 val = xstrdup(strchr(opt, ':') + 1);                  val = xstrdup(strchr(opt, ':') + 1);
                 if ((cp = strchr(val, '=')) != NULL)                  if ((cp = strchr(val, '=')) != NULL)
                         *cp++ = '\0';                          *cp++ = '\0';
                 cert_userext = xreallocarray(cert_userext, ncert_userext + 1,                  cert_ext_add(val, cp, iscrit);
                     sizeof(*cert_userext));                  free(val);
                 cert_userext[ncert_userext].key = val;  
                 cert_userext[ncert_userext].val = cp == NULL ?  
                     NULL : xstrdup(cp);  
                 cert_userext[ncert_userext].crit = iscrit;  
                 ncert_userext++;  
         } else          } else
                 fatal("Unsupported certificate option \"%s\"", opt);                  fatal("Unsupported certificate option \"%s\"", opt);
 }  }
Line 2008 
Line 1998 
 static void  static void
 show_options(struct sshbuf *optbuf, int in_critical)  show_options(struct sshbuf *optbuf, int in_critical)
 {  {
         char *name, *arg;          char *name, *arg, *hex;
         struct sshbuf *options, *option = NULL;          struct sshbuf *options, *option = NULL;
         int r;          int r;
   
         if ((options = sshbuf_fromb(optbuf)) == NULL)          if ((options = sshbuf_fromb(optbuf)) == NULL)
                 fatal("%s: sshbuf_fromb failed", __func__);                  fatal_f("sshbuf_fromb failed");
         while (sshbuf_len(options) != 0) {          while (sshbuf_len(options) != 0) {
                 sshbuf_free(option);                  sshbuf_free(option);
                 option = NULL;                  option = NULL;
                 if ((r = sshbuf_get_cstring(options, &name, NULL)) != 0 ||                  if ((r = sshbuf_get_cstring(options, &name, NULL)) != 0 ||
                     (r = sshbuf_froms(options, &option)) != 0)                      (r = sshbuf_froms(options, &option)) != 0)
                         fatal("%s: buffer error: %s", __func__, ssh_err(r));                          fatal_fr(r, "parse option");
                 printf("                %s", name);                  printf("                %s", name);
                 if (!in_critical &&                  if (!in_critical &&
                     (strcmp(name, "permit-X11-forwarding") == 0 ||                      (strcmp(name, "permit-X11-forwarding") == 0 ||
Line 2033 
Line 2023 
                     (strcmp(name, "force-command") == 0 ||                      (strcmp(name, "force-command") == 0 ||
                     strcmp(name, "source-address") == 0)) {                      strcmp(name, "source-address") == 0)) {
                         if ((r = sshbuf_get_cstring(option, &arg, NULL)) != 0)                          if ((r = sshbuf_get_cstring(option, &arg, NULL)) != 0)
                                 fatal("%s: buffer error: %s",                                  fatal_fr(r, "parse critical");
                                     __func__, ssh_err(r));  
                         printf(" %s\n", arg);                          printf(" %s\n", arg);
                         free(arg);                          free(arg);
                 } else {                  } else if (sshbuf_len(option) > 0) {
                         printf(" UNKNOWN OPTION (len %zu)\n",                          hex = sshbuf_dtob16(option);
                             sshbuf_len(option));                          printf(" UNKNOWN OPTION: %s (len %zu)\n",
                               hex, sshbuf_len(option));
                         sshbuf_reset(option);                          sshbuf_reset(option);
                 }                          free(hex);
                   } else
                           printf(" UNKNOWN FLAG OPTION\n");
                 free(name);                  free(name);
                 if (sshbuf_len(option) != 0)                  if (sshbuf_len(option) != 0)
                         fatal("Option corrupt: extra data at end");                          fatal("Option corrupt: extra data at end");
Line 2060 
Line 2052 
         ca_fp = sshkey_fingerprint(key->cert->signature_key,          ca_fp = sshkey_fingerprint(key->cert->signature_key,
             fingerprint_hash, SSH_FP_DEFAULT);              fingerprint_hash, SSH_FP_DEFAULT);
         if (key_fp == NULL || ca_fp == NULL)          if (key_fp == NULL || ca_fp == NULL)
                 fatal("%s: sshkey_fingerprint fail", __func__);                  fatal_f("sshkey_fingerprint fail");
         sshkey_format_cert_validity(key->cert, valid, sizeof(valid));          sshkey_format_cert_validity(key->cert, valid, sizeof(valid));
   
         printf("        Type: %s %s certificate\n", sshkey_ssh_name(key),          printf("        Type: %s %s certificate\n", sshkey_ssh_name(key),
Line 2133 
Line 2125 
                 if ((key = sshkey_new(KEY_UNSPEC)) == NULL)                  if ((key = sshkey_new(KEY_UNSPEC)) == NULL)
                         fatal("sshkey_new");                          fatal("sshkey_new");
                 if ((r = sshkey_read(key, &cp)) != 0) {                  if ((r = sshkey_read(key, &cp)) != 0) {
                         error("%s:%lu: invalid key: %s", path,                          error_r(r, "%s:%lu: invalid key", path, lnum);
                             lnum, ssh_err(r));  
                         continue;                          continue;
                 }                  }
                 if (!sshkey_is_cert(key)) {                  if (!sshkey_is_cert(key)) {
Line 2161 
Line 2152 
         int r;          int r;
   
         if ((r = sshbuf_load_file(path, &krlbuf)) != 0)          if ((r = sshbuf_load_file(path, &krlbuf)) != 0)
                 fatal("Unable to load KRL: %s", ssh_err(r));                  fatal_r(r, "Unable to load KRL %s", path);
         /* XXX check sigs */          /* XXX check sigs */
         if ((r = ssh_krl_from_blob(krlbuf, krlp, NULL, 0)) != 0 ||          if ((r = ssh_krl_from_blob(krlbuf, krlp, NULL, 0)) != 0 ||
             *krlp == NULL)              *krlp == NULL)
                 fatal("Invalid KRL file: %s", ssh_err(r));                  fatal_r(r, "Invalid KRL file %s", path);
         sshbuf_free(krlbuf);          sshbuf_free(krlbuf);
 }  }
   
Line 2194 
Line 2185 
                 tmp[tlen] = '\0';                  tmp[tlen] = '\0';
         }          }
         if ((b = sshbuf_new()) == NULL)          if ((b = sshbuf_new()) == NULL)
                 fatal("%s: sshbuf_new failed", __func__);                  fatal_f("sshbuf_new failed");
         if ((r = sshbuf_b64tod(b, tmp)) != 0)          if ((r = sshbuf_b64tod(b, tmp)) != 0)
                 fatal("%s:%lu: decode hash failed: %s", file, lnum, ssh_err(r));                  fatal_r(r, "%s:%lu: decode hash failed", file, lnum);
         free(tmp);          free(tmp);
         *lenp = sshbuf_len(b);          *lenp = sshbuf_len(b);
         *blobp = xmalloc(*lenp);          *blobp = xmalloc(*lenp);
Line 2282 
Line 2273 
                         }                          }
                         if (ssh_krl_revoke_cert_by_serial_range(krl,                          if (ssh_krl_revoke_cert_by_serial_range(krl,
                             ca, serial, serial2) != 0) {                              ca, serial, serial2) != 0) {
                                 fatal("%s: revoke serial failed",                                  fatal_f("revoke serial failed");
                                     __func__);  
                         }                          }
                 } else if (strncasecmp(cp, "id:", 3) == 0) {                  } else if (strncasecmp(cp, "id:", 3) == 0) {
                         if (ca == NULL && !wild_ca) {                          if (ca == NULL && !wild_ca) {
Line 2293 
Line 2283 
                         cp += 3;                          cp += 3;
                         cp = cp + strspn(cp, " \t");                          cp = cp + strspn(cp, " \t");
                         if (ssh_krl_revoke_cert_by_key_id(krl, ca, cp) != 0)                          if (ssh_krl_revoke_cert_by_key_id(krl, ca, cp) != 0)
                                 fatal("%s: revoke key ID failed", __func__);                                  fatal_f("revoke key ID failed");
                 } else if (strncasecmp(cp, "hash:", 5) == 0) {                  } else if (strncasecmp(cp, "hash:", 5) == 0) {
                         cp += 5;                          cp += 5;
                         cp = cp + strspn(cp, " \t");                          cp = cp + strspn(cp, " \t");
                         hash_to_blob(cp, &blob, &blen, file, lnum);                          hash_to_blob(cp, &blob, &blen, file, lnum);
                         r = ssh_krl_revoke_key_sha256(krl, blob, blen);                          r = ssh_krl_revoke_key_sha256(krl, blob, blen);
                         if (r != 0)                          if (r != 0)
                                 fatal("%s: revoke key failed: %s",                                  fatal_fr(r, "revoke key failed");
                                     __func__, ssh_err(r));  
                 } else {                  } else {
                         if (strncasecmp(cp, "key:", 4) == 0) {                          if (strncasecmp(cp, "key:", 4) == 0) {
                                 cp += 4;                                  cp += 4;
Line 2323 
Line 2312 
                         if ((key = sshkey_new(KEY_UNSPEC)) == NULL)                          if ((key = sshkey_new(KEY_UNSPEC)) == NULL)
                                 fatal("sshkey_new");                                  fatal("sshkey_new");
                         if ((r = sshkey_read(key, &cp)) != 0)                          if ((r = sshkey_read(key, &cp)) != 0)
                                 fatal("%s:%lu: invalid key: %s",                                  fatal_r(r, "%s:%lu: invalid key", path, lnum);
                                     path, lnum, ssh_err(r));  
                         if (was_explicit_key)                          if (was_explicit_key)
                                 r = ssh_krl_revoke_key_explicit(krl, key);                                  r = ssh_krl_revoke_key_explicit(krl, key);
                         else if (was_sha1) {                          else if (was_sha1) {
Line 2344 
Line 2332 
                         } else                          } else
                                 r = ssh_krl_revoke_key(krl, key);                                  r = ssh_krl_revoke_key(krl, key);
                         if (r != 0)                          if (r != 0)
                                 fatal("%s: revoke key failed: %s",                                  fatal_fr(r, "revoke key failed");
                                     __func__, ssh_err(r));  
                         freezero(blob, blen);                          freezero(blob, blen);
                         blob = NULL;                          blob = NULL;
                         blen = 0;                          blen = 0;
Line 2385 
Line 2372 
                 else {                  else {
                         tmp = tilde_expand_filename(ca_key_path, pw->pw_uid);                          tmp = tilde_expand_filename(ca_key_path, pw->pw_uid);
                         if ((r = sshkey_load_public(tmp, &ca, NULL)) != 0)                          if ((r = sshkey_load_public(tmp, &ca, NULL)) != 0)
                                 fatal("Cannot load CA public key %s: %s",                                  fatal_r(r, "Cannot load CA public key %s", tmp);
                                     tmp, ssh_err(r));  
                         free(tmp);                          free(tmp);
                 }                  }
         }          }
Line 2430 
Line 2416 
                 krl_dump(krl, stdout);                  krl_dump(krl, stdout);
         for (i = 0; i < argc; i++) {          for (i = 0; i < argc; i++) {
                 if ((r = sshkey_load_public(argv[i], &k, &comment)) != 0)                  if ((r = sshkey_load_public(argv[i], &k, &comment)) != 0)
                         fatal("Cannot load public key %s: %s",                          fatal_r(r, "Cannot load public key %s", argv[i]);
                             argv[i], ssh_err(r));  
                 r = ssh_krl_check_key(krl, k);                  r = ssh_krl_check_key(krl, k);
                 printf("%s%s%s%s: %s\n", argv[i],                  printf("%s%s%s%s: %s\n", argv[i],
                     *comment ? " (" : "", comment, *comment ? ")" : "",                      *comment ? " (" : "", comment, *comment ? ")" : "",
Line 2465 
Line 2450 
                     strcmp(privpath + plen - slen, suffixes[i]) != 0)                      strcmp(privpath + plen - slen, suffixes[i]) != 0)
                         continue;                          continue;
                 privpath[plen - slen] = '\0';                  privpath[plen - slen] = '\0';
                 debug("%s: %s looks like a public key, using private key "                  debug_f("%s looks like a public key, using private key "
                     "path %s instead", __func__, keypath, privpath);                      "path %s instead", keypath, privpath);
         }          }
         if ((privkey = load_identity(privpath, NULL)) == NULL) {          if ((privkey = load_identity(privpath, NULL)) == NULL) {
                 error("Couldn't load identity %s", keypath);                  error("Couldn't load identity %s", keypath);
Line 2483 
Line 2468 
                  * it capable of signing.                   * it capable of signing.
                  */                   */
                 if ((r = sshkey_to_certified(privkey)) != 0) {                  if ((r = sshkey_to_certified(privkey)) != 0) {
                         error("%s: sshkey_to_certified: %s", __func__,                          error_fr(r, "sshkey_to_certified");
                             ssh_err(r));  
                         goto done;                          goto done;
                 }                  }
                 if ((r = sshkey_cert_copy(pubkey, privkey)) != 0) {                  if ((r = sshkey_cert_copy(pubkey, privkey)) != 0) {
                         error("%s: sshkey_cert_copy: %s", __func__, ssh_err(r));                          error_fr(r, "sshkey_cert_copy");
                         goto done;                          goto done;
                 }                  }
         }          }
Line 2508 
Line 2492 
         struct sshbuf *sigbuf = NULL, *abuf = NULL;          struct sshbuf *sigbuf = NULL, *abuf = NULL;
         int r = SSH_ERR_INTERNAL_ERROR, wfd = -1, oerrno;          int r = SSH_ERR_INTERNAL_ERROR, wfd = -1, oerrno;
         char *wfile = NULL, *asig = NULL, *fp = NULL;          char *wfile = NULL, *asig = NULL, *fp = NULL;
           char *pin = NULL, *prompt = NULL;
   
         if (!quiet) {          if (!quiet) {
                 if (fd == STDIN_FILENO)                  if (fd == STDIN_FILENO)
Line 2515 
Line 2500 
                 else                  else
                         fprintf(stderr, "Signing file %s\n", filename);                          fprintf(stderr, "Signing file %s\n", filename);
         }          }
         if (signer == NULL && sshkey_is_sk(signkey) &&          if (signer == NULL && sshkey_is_sk(signkey)) {
             (signkey->sk_flags & SSH_SK_USER_PRESENCE_REQD)) {                  if ((signkey->sk_flags & SSH_SK_USER_VERIFICATION_REQD)) {
                 if ((fp = sshkey_fingerprint(signkey, fingerprint_hash,                          xasprintf(&prompt, "Enter PIN for %s key: ",
                     SSH_FP_DEFAULT)) == NULL)                              sshkey_type(signkey));
                         fatal("%s: sshkey_fingerprint failed", __func__);                          if ((pin = read_passphrase(prompt,
                 fprintf(stderr, "Confirm user presence for key %s %s\n",                              RP_ALLOW_STDIN)) == NULL)
                     sshkey_type(signkey), fp);                                  fatal_f("couldn't read PIN");
                 free(fp);                  }
                   if ((signkey->sk_flags & SSH_SK_USER_PRESENCE_REQD)) {
                           if ((fp = sshkey_fingerprint(signkey, fingerprint_hash,
                               SSH_FP_DEFAULT)) == NULL)
                                   fatal_f("fingerprint failed");
                           fprintf(stderr, "Confirm user presence for key %s %s\n",
                               sshkey_type(signkey), fp);
                           free(fp);
                   }
         }          }
         if ((r = sshsig_sign_fd(signkey, NULL, sk_provider, fd, sig_namespace,          if ((r = sshsig_sign_fd(signkey, NULL, sk_provider, pin,
             &sigbuf, signer, signer_ctx)) != 0) {              fd, sig_namespace, &sigbuf, signer, signer_ctx)) != 0) {
                 error("Signing %s failed: %s", filename, ssh_err(r));                  error_r(r, "Signing %s failed", filename);
                 goto out;                  goto out;
         }          }
         if ((r = sshsig_armor(sigbuf, &abuf)) != 0) {          if ((r = sshsig_armor(sigbuf, &abuf)) != 0) {
                 error("%s: sshsig_armor: %s", __func__, ssh_err(r));                  error_fr(r, "sshsig_armor");
                 goto out;                  goto out;
         }          }
         if ((asig = sshbuf_dup_string(abuf)) == NULL) {          if ((asig = sshbuf_dup_string(abuf)) == NULL) {
                 error("%s: buffer error", __func__);                  error_f("buffer error");
                 r = SSH_ERR_ALLOC_FAIL;                  r = SSH_ERR_ALLOC_FAIL;
                 goto out;                  goto out;
         }          }
Line 2573 
Line 2566 
         r = 0;          r = 0;
  out:   out:
         free(wfile);          free(wfile);
           free(prompt);
         free(asig);          free(asig);
           if (pin != NULL)
                   freezero(pin, strlen(pin));
         sshbuf_free(abuf);          sshbuf_free(abuf);
         sshbuf_free(sigbuf);          sshbuf_free(sigbuf);
         if (wfd != -1)          if (wfd != -1)
Line 2598 
Line 2594 
         }          }
   
         if ((r = sshkey_load_public(keypath, &pubkey, NULL)) != 0) {          if ((r = sshkey_load_public(keypath, &pubkey, NULL)) != 0) {
                 error("Couldn't load public key %s: %s", keypath, ssh_err(r));                  error_r(r, "Couldn't load public key %s", keypath);
                 goto done;                  goto done;
         }          }
   
         if ((r = ssh_get_authentication_socket(&agent_fd)) != 0)          if ((r = ssh_get_authentication_socket(&agent_fd)) != 0)
                 debug("Couldn't get agent socket: %s", ssh_err(r));                  debug_r(r, "Couldn't get agent socket");
         else {          else {
                 if ((r = ssh_agent_has_key(agent_fd, pubkey)) == 0)                  if ((r = ssh_agent_has_key(agent_fd, pubkey)) == 0)
                         signer = agent_signer;                          signer = agent_signer;
                 else                  else
                         debug("Couldn't find key in agent: %s", ssh_err(r));                          debug_r(r, "Couldn't find key in agent");
         }          }
   
         if (signer == NULL) {          if (signer == NULL) {
Line 2664 
Line 2660 
   
         memset(&sig_details, 0, sizeof(sig_details));          memset(&sig_details, 0, sizeof(sig_details));
         if ((r = sshbuf_load_file(signature, &abuf)) != 0) {          if ((r = sshbuf_load_file(signature, &abuf)) != 0) {
                 error("Couldn't read signature file: %s", ssh_err(r));                  error_r(r, "Couldn't read signature file");
                 goto done;                  goto done;
         }          }
   
         if ((r = sshsig_dearmor(abuf, &sigbuf)) != 0) {          if ((r = sshsig_dearmor(abuf, &sigbuf)) != 0) {
                 error("%s: sshsig_armor: %s", __func__, ssh_err(r));                  error_fr(r, "sshsig_armor");
                 goto done;                  goto done;
         }          }
         if ((r = sshsig_verify_fd(sigbuf, STDIN_FILENO, sig_namespace,          if ((r = sshsig_verify_fd(sigbuf, STDIN_FILENO, sig_namespace,
Line 2678 
Line 2674 
   
         if ((fp = sshkey_fingerprint(sign_key, fingerprint_hash,          if ((fp = sshkey_fingerprint(sign_key, fingerprint_hash,
             SSH_FP_DEFAULT)) == NULL)              SSH_FP_DEFAULT)) == NULL)
                 fatal("%s: sshkey_fingerprint failed", __func__);                  fatal_f("sshkey_fingerprint failed");
         debug("Valid (unverified) signature from key %s", fp);          debug("Valid (unverified) signature from key %s", fp);
         if (sig_details != NULL) {          if (sig_details != NULL) {
                 debug2("%s: signature details: counter = %u, flags = 0x%02x",                  debug2_f("signature details: counter = %u, flags = 0x%02x",
                     __func__, sig_details->sk_counter, sig_details->sk_flags);                      sig_details->sk_counter, sig_details->sk_flags);
         }          }
         free(fp);          free(fp);
         fp = NULL;          fp = NULL;
   
         if (revoked_keys != NULL) {          if (revoked_keys != NULL) {
                 if ((r = sshkey_check_revoked(sign_key, revoked_keys)) != 0) {                  if ((r = sshkey_check_revoked(sign_key, revoked_keys)) != 0) {
                         debug3("sshkey_check_revoked failed: %s", ssh_err(r));                          debug3_fr(r, "sshkey_check_revoked");
                         goto done;                          goto done;
                 }                  }
         }          }
   
         if (allowed_keys != NULL &&          if (allowed_keys != NULL && (r = sshsig_check_allowed_keys(allowed_keys,
             (r = sshsig_check_allowed_keys(allowed_keys, sign_key,              sign_key, principal, sig_namespace)) != 0) {
                                            principal, sig_namespace)) != 0) {                  debug3_fr(r, "sshsig_check_allowed_keys");
                 debug3("sshsig_check_allowed_keys failed: %s", ssh_err(r));  
                 goto done;                  goto done;
         }          }
         /* success */          /* success */
Line 2706 
Line 2701 
         if (!quiet) {          if (!quiet) {
                 if (ret == 0) {                  if (ret == 0) {
                         if ((fp = sshkey_fingerprint(sign_key, fingerprint_hash,                          if ((fp = sshkey_fingerprint(sign_key, fingerprint_hash,
                             SSH_FP_DEFAULT)) == NULL) {                              SSH_FP_DEFAULT)) == NULL)
                                 fatal("%s: sshkey_fingerprint failed",                                  fatal_f("sshkey_fingerprint failed");
                                     __func__);  
                         }  
                         if (principal == NULL) {                          if (principal == NULL) {
                                 printf("Good \"%s\" signature with %s key %s\n",                                  printf("Good \"%s\" signature with %s key %s\n",
                                        sig_namespace, sshkey_type(sign_key), fp);                                         sig_namespace, sshkey_type(sign_key), fp);
Line 2739 
Line 2732 
         char *principals = NULL, *cp, *tmp;          char *principals = NULL, *cp, *tmp;
   
         if ((r = sshbuf_load_file(signature, &abuf)) != 0) {          if ((r = sshbuf_load_file(signature, &abuf)) != 0) {
                 error("Couldn't read signature file: %s", ssh_err(r));                  error_r(r, "Couldn't read signature file");
                 goto done;                  goto done;
         }          }
         if ((r = sshsig_dearmor(abuf, &sigbuf)) != 0) {          if ((r = sshsig_dearmor(abuf, &sigbuf)) != 0) {
                 error("%s: sshsig_armor: %s", __func__, ssh_err(r));                  error_fr(r, "sshsig_armor");
                 goto done;                  goto done;
         }          }
         if ((r = sshsig_get_pubkey(sigbuf, &sign_key)) != 0) {          if ((r = sshsig_get_pubkey(sigbuf, &sign_key)) != 0) {
                 error("%s: sshsig_get_pubkey: %s",                  error_fr(r, "sshsig_get_pubkey");
                     __func__, ssh_err(r));  
                 goto done;                  goto done;
         }          }
         if ((r = sshsig_find_principals(allowed_keys, sign_key,          if ((r = sshsig_find_principals(allowed_keys, sign_key,
             &principals)) != 0) {              &principals)) != 0) {
                 error("%s: sshsig_get_principal: %s",                  error_fr(r, "sshsig_get_principal");
                       __func__, ssh_err(r));  
                 goto done;                  goto done;
         }          }
         ret = 0;          ret = 0;
Line 2938 
Line 2929 
 {  {
         struct sshkey **keys;          struct sshkey **keys;
         size_t nkeys, i;          size_t nkeys, i;
         int r, ok = -1;          int r, ret = -1;
         char *fp, *pin = NULL, *pass = NULL, *path, *pubpath;          char *fp, *pin = NULL, *pass = NULL, *path, *pubpath;
         const char *ext;          const char *ext;
   
         if (skprovider == NULL)          if (skprovider == NULL)
                 fatal("Cannot download keys without provider");                  fatal("Cannot download keys without provider");
   
         for (i = 0; i < 2; i++) {          pin = read_passphrase("Enter PIN for authenticator: ", RP_ALLOW_STDIN);
                 if (i == 1) {          if (!quiet) {
                         pin = read_passphrase("Enter PIN for authenticator: ",                  printf("You may need to touch your authenticator "
                             RP_ALLOW_STDIN);                      "to authorize key download.\n");
                 }          }
                 if ((r = sshsk_load_resident(skprovider, device, pin,          if ((r = sshsk_load_resident(skprovider, device, pin,
                     &keys, &nkeys)) != 0) {              &keys, &nkeys)) != 0) {
                         if (i == 0 && r == SSH_ERR_KEY_WRONG_PASSPHRASE)                  if (pin != NULL)
                                 continue;  
                         freezero(pin, strlen(pin));                          freezero(pin, strlen(pin));
                         error("Unable to load resident keys: %s", ssh_err(r));                  error_r(r, "Unable to load resident keys");
                         return -1;                  return -1;
                 }  
         }          }
         if (nkeys == 0)          if (nkeys == 0)
                 logit("No keys to download");                  logit("No keys to download");
         freezero(pin, strlen(pin));          if (pin != NULL)
                   freezero(pin, strlen(pin));
   
         for (i = 0; i < nkeys; i++) {          for (i = 0; i < nkeys; i++) {
                 if (keys[i]->type != KEY_ECDSA_SK &&                  if (keys[i]->type != KEY_ECDSA_SK &&
Line 2972 
Line 2962 
                 }                  }
                 if ((fp = sshkey_fingerprint(keys[i],                  if ((fp = sshkey_fingerprint(keys[i],
                     fingerprint_hash, SSH_FP_DEFAULT)) == NULL)                      fingerprint_hash, SSH_FP_DEFAULT)) == NULL)
                         fatal("%s: sshkey_fingerprint failed", __func__);                          fatal_f("sshkey_fingerprint failed");
                 debug("%s: key %zu: %s %s %s (flags 0x%02x)", __func__, i,                  debug_f("key %zu: %s %s %s (flags 0x%02x)", i,
                     sshkey_type(keys[i]), fp, keys[i]->sk_application,                      sshkey_type(keys[i]), fp, keys[i]->sk_application,
                     keys[i]->sk_flags);                      keys[i]->sk_flags);
                 ext = skip_ssh_url_preamble(keys[i]->sk_application);                  ext = skip_ssh_url_preamble(keys[i]->sk_application);
Line 2993 
Line 2983 
                 if ((r = sshkey_save_private(keys[i], path, pass,                  if ((r = sshkey_save_private(keys[i], path, pass,
                     keys[i]->sk_application, private_key_format,                      keys[i]->sk_application, private_key_format,
                     openssh_format_cipher, rounds)) != 0) {                      openssh_format_cipher, rounds)) != 0) {
                         error("Saving key \"%s\" failed: %s",                          error_r(r, "Saving key \"%s\" failed", path);
                             path, ssh_err(r));  
                         free(path);                          free(path);
                         break;                          break;
                 }                  }
Line 3011 
Line 3000 
                 free(path);                  free(path);
                 if ((r = sshkey_save_public(keys[i], pubpath,                  if ((r = sshkey_save_public(keys[i], pubpath,
                     keys[i]->sk_application)) != 0) {                      keys[i]->sk_application)) != 0) {
                         error("Saving public key \"%s\" failed: %s",                          error_r(r, "Saving public key \"%s\" failed", pubpath);
                             pubpath, ssh_err(r));  
                         free(pubpath);                          free(pubpath);
                         break;                          break;
                 }                  }
Line 3020 
Line 3008 
         }          }
   
         if (i >= nkeys)          if (i >= nkeys)
                 ok = 0; /* success */                  ret = 0; /* success */
         if (pass != NULL)          if (pass != NULL)
                 freezero(pass, strlen(pass));                  freezero(pass, strlen(pass));
         for (i = 0; i < nkeys; i++)          for (i = 0; i < nkeys; i++)
                 sshkey_free(keys[i]);                  sshkey_free(keys[i]);
         free(keys);          free(keys);
         return ok ? 0 : -1;          return ret;
 }  }
   
 static void  static void
   save_attestation(struct sshbuf *attest, const char *path)
   {
           mode_t omask;
           int r;
   
           if (path == NULL)
                   return; /* nothing to do */
           if (attest == NULL || sshbuf_len(attest) == 0)
                   fatal("Enrollment did not return attestation data");
           omask = umask(077);
           r = sshbuf_write_file(path, attest);
           umask(omask);
           if (r != 0)
                   fatal_r(r, "Unable to write attestation data \"%s\"", path);
           if (!quiet)
                   printf("Your FIDO attestation certificate has been saved in "
                       "%s\n", path);
   }
   
   static void
 usage(void)  usage(void)
 {  {
         fprintf(stderr,          fprintf(stderr,
             "usage: ssh-keygen [-q] [-b bits] [-C comment] [-f output_keyfile] [-m format]\n"              "usage: ssh-keygen [-q] [-a rounds] [-b bits] [-C comment] [-f output_keyfile]\n"
               "                  [-m format] [-N new_passphrase] [-O option]\n"
             "                  [-t dsa | ecdsa | ecdsa-sk | ed25519 | ed25519-sk | rsa]\n"              "                  [-t dsa | ecdsa | ecdsa-sk | ed25519 | ed25519-sk | rsa]\n"
             "                  [-N new_passphrase] [-O option] [-w provider]\n"              "                  [-w provider]\n"
             "       ssh-keygen -p [-f keyfile] [-m format] [-N new_passphrase]\n"              "       ssh-keygen -p [-a rounds] [-f keyfile] [-m format] [-N new_passphrase]\n"
             "                   [-P old_passphrase]\n"              "                   [-P old_passphrase]\n"
             "       ssh-keygen -i [-f input_keyfile] [-m key_format]\n"              "       ssh-keygen -i [-f input_keyfile] [-m key_format]\n"
             "       ssh-keygen -e [-f input_keyfile] [-m key_format]\n"              "       ssh-keygen -e [-f input_keyfile] [-m key_format]\n"
             "       ssh-keygen -y [-f input_keyfile]\n"              "       ssh-keygen -y [-f input_keyfile]\n"
             "       ssh-keygen -c [-C comment] [-f keyfile] [-P passphrase]\n"              "       ssh-keygen -c [-a rounds] [-C comment] [-f keyfile] [-P passphrase]\n"
             "       ssh-keygen -l [-v] [-E fingerprint_hash] [-f input_keyfile]\n"              "       ssh-keygen -l [-v] [-E fingerprint_hash] [-f input_keyfile]\n"
             "       ssh-keygen -B [-f input_keyfile]\n");              "       ssh-keygen -B [-f input_keyfile]\n");
 #ifdef ENABLE_PKCS11  #ifdef ENABLE_PKCS11
Line 3051 
Line 3060 
         fprintf(stderr,          fprintf(stderr,
             "       ssh-keygen -F hostname [-lv] [-f known_hosts_file]\n"              "       ssh-keygen -F hostname [-lv] [-f known_hosts_file]\n"
             "       ssh-keygen -H [-f known_hosts_file]\n"              "       ssh-keygen -H [-f known_hosts_file]\n"
             "       ssh-keygen -K [-w provider]\n"              "       ssh-keygen -K [-a rounds] [-w provider]\n"
             "       ssh-keygen -R hostname [-f known_hosts_file]\n"              "       ssh-keygen -R hostname [-f known_hosts_file]\n"
             "       ssh-keygen -r hostname [-g] [-f input_keyfile]\n"              "       ssh-keygen -r hostname [-g] [-f input_keyfile]\n"
 #ifdef WITH_OPENSSL  #ifdef WITH_OPENSSL
Line 3062 
Line 3071 
             "                  [-n principals] [-O option] [-V validity_interval]\n"              "                  [-n principals] [-O option] [-V validity_interval]\n"
             "                  [-z serial_number] file ...\n"              "                  [-z serial_number] file ...\n"
             "       ssh-keygen -L [-f input_keyfile]\n"              "       ssh-keygen -L [-f input_keyfile]\n"
             "       ssh-keygen -A [-f prefix_path]\n"              "       ssh-keygen -A [-a rounds] [-f prefix_path]\n"
             "       ssh-keygen -k -f krl_file [-u] [-s ca_public] [-z version_number]\n"              "       ssh-keygen -k -f krl_file [-u] [-s ca_public] [-z version_number]\n"
             "                  file ...\n"              "                  file ...\n"
             "       ssh-keygen -Q [-l] -f krl_file [file ...]\n"              "       ssh-keygen -Q [-l] -f krl_file [file ...]\n"
Line 3080 
Line 3089 
 int  int
 main(int argc, char **argv)  main(int argc, char **argv)
 {  {
         char dotsshdir[PATH_MAX], comment[1024], *passphrase;          char comment[1024], *passphrase;
         char *rr_hostname = NULL, *ep, *fp, *ra;          char *rr_hostname = NULL, *ep, *fp, *ra;
         struct sshkey *private, *public;          struct sshkey *private, *public;
         struct passwd *pw;          struct passwd *pw;
         struct stat st;  
         int r, opt, type;          int r, opt, type;
         int change_passphrase = 0, change_comment = 0, show_cert = 0;          int change_passphrase = 0, change_comment = 0, show_cert = 0;
         int find_host = 0, delete_host = 0, hash_hosts = 0;          int find_host = 0, delete_host = 0, hash_hosts = 0;
Line 3095 
Line 3103 
         unsigned long long cert_serial = 0;          unsigned long long cert_serial = 0;
         char *identity_comment = NULL, *ca_key_path = NULL, **opts = NULL;          char *identity_comment = NULL, *ca_key_path = NULL, **opts = NULL;
         char *sk_application = NULL, *sk_device = NULL, *sk_user = NULL;          char *sk_application = NULL, *sk_device = NULL, *sk_user = NULL;
         char *sk_attestaion_path = NULL;          char *sk_attestation_path = NULL;
         struct sshbuf *challenge = NULL, *attest = NULL;          struct sshbuf *challenge = NULL, *attest = NULL;
         size_t i, nopts = 0;          size_t i, nopts = 0;
         u_int32_t bits = 0;          u_int32_t bits = 0;
Line 3531 
Line 3539 
                 for (i = 0; i < nopts; i++) {                  for (i = 0; i < nopts; i++) {
                         if (strcasecmp(opts[i], "no-touch-required") == 0) {                          if (strcasecmp(opts[i], "no-touch-required") == 0) {
                                 sk_flags &= ~SSH_SK_USER_PRESENCE_REQD;                                  sk_flags &= ~SSH_SK_USER_PRESENCE_REQD;
                           } else if (strcasecmp(opts[i], "verify-required") == 0) {
                                   sk_flags |= SSH_SK_USER_VERIFICATION_REQD;
                         } else if (strcasecmp(opts[i], "resident") == 0) {                          } else if (strcasecmp(opts[i], "resident") == 0) {
                                 sk_flags |= SSH_SK_RESIDENT_KEY;                                  sk_flags |= SSH_SK_RESIDENT_KEY;
                         } else if (strncasecmp(opts[i], "device=", 7) == 0) {                          } else if (strncasecmp(opts[i], "device=", 7) == 0) {
Line 3540 
Line 3550 
                         } else if (strncasecmp(opts[i], "challenge=", 10) == 0) {                          } else if (strncasecmp(opts[i], "challenge=", 10) == 0) {
                                 if ((r = sshbuf_load_file(opts[i] + 10,                                  if ((r = sshbuf_load_file(opts[i] + 10,
                                     &challenge)) != 0) {                                      &challenge)) != 0) {
                                         fatal("Unable to load FIDO enrollment "                                          fatal_r(r, "Unable to load FIDO "
                                             "challenge \"%s\": %s",                                              "enrollment challenge \"%s\"",
                                             opts[i] + 10, ssh_err(r));                                              opts[i] + 10);
                                 }                                  }
                         } else if (strncasecmp(opts[i],                          } else if (strncasecmp(opts[i],
                             "write-attestation=", 18) == 0) {                              "write-attestation=", 18) == 0) {
                                 sk_attestaion_path = opts[i] + 18;                                  sk_attestation_path = opts[i] + 18;
                         } else if (strncasecmp(opts[i],                          } else if (strncasecmp(opts[i],
                             "application=", 12) == 0) {                              "application=", 12) == 0) {
                                 sk_application = xstrdup(opts[i] + 12);                                  sk_application = xstrdup(opts[i] + 12);
Line 3563 
Line 3573 
                         printf("You may need to touch your authenticator "                          printf("You may need to touch your authenticator "
                             "to authorize key generation.\n");                              "to authorize key generation.\n");
                 }                  }
                 passphrase = NULL;  
                 if ((attest = sshbuf_new()) == NULL)                  if ((attest = sshbuf_new()) == NULL)
                         fatal("sshbuf_new failed");                          fatal("sshbuf_new failed");
                   if ((sk_flags &
                       (SSH_SK_USER_VERIFICATION_REQD|SSH_SK_RESIDENT_KEY))) {
                           passphrase = read_passphrase("Enter PIN for "
                               "authenticator: ", RP_ALLOW_STDIN);
                   } else {
                           passphrase = NULL;
                   }
                 for (i = 0 ; ; i++) {                  for (i = 0 ; ; i++) {
                         fflush(stdout);                          fflush(stdout);
                         r = sshsk_enroll(type, sk_provider, sk_device,                          r = sshsk_enroll(type, sk_provider, sk_device,
Line 3575 
Line 3591 
                         if (r == 0)                          if (r == 0)
                                 break;                                  break;
                         if (r != SSH_ERR_KEY_WRONG_PASSPHRASE)                          if (r != SSH_ERR_KEY_WRONG_PASSPHRASE)
                                 fatal("Key enrollment failed: %s", ssh_err(r));                                  fatal_r(r, "Key enrollment failed");
                         else if (i > 0)                          else if (passphrase != NULL) {
                                 error("PIN incorrect");                                  error("PIN incorrect");
                         if (passphrase != NULL) {  
                                 freezero(passphrase, strlen(passphrase));                                  freezero(passphrase, strlen(passphrase));
                                 passphrase = NULL;                                  passphrase = NULL;
                         }                          }
Line 3586 
Line 3601 
                                 fatal("Too many incorrect PINs");                                  fatal("Too many incorrect PINs");
                         passphrase = read_passphrase("Enter PIN for "                          passphrase = read_passphrase("Enter PIN for "
                             "authenticator: ", RP_ALLOW_STDIN);                              "authenticator: ", RP_ALLOW_STDIN);
                           if (!quiet) {
                                   printf("You may need to touch your "
                                       "authenticator (again) to authorize "
                                       "key generation.\n");
                           }
                 }                  }
                 if (passphrase != NULL) {                  if (passphrase != NULL) {
                         freezero(passphrase, strlen(passphrase));                          freezero(passphrase, strlen(passphrase));
Line 3598 
Line 3618 
                 break;                  break;
         }          }
         if ((r = sshkey_from_private(private, &public)) != 0)          if ((r = sshkey_from_private(private, &public)) != 0)
                 fatal("sshkey_from_private failed: %s\n", ssh_err(r));                  fatal_r(r, "sshkey_from_private");
   
         if (!have_identity)          if (!have_identity)
                 ask_filename(pw, "Enter file in which to save the key");                  ask_filename(pw, "Enter file in which to save the key");
   
         /* Create ~/.ssh directory if it doesn't already exist. */          /* Create ~/.ssh directory if it doesn't already exist. */
         snprintf(dotsshdir, sizeof dotsshdir, "%s/%s",          hostfile_create_user_ssh_dir(identity_file, !quiet);
             pw->pw_dir, _PATH_SSH_USER_DIR);  
         if (strstr(identity_file, dotsshdir) != NULL) {  
                 if (stat(dotsshdir, &st) == -1) {  
                         if (errno != ENOENT) {  
                                 error("Could not stat %s: %s", dotsshdir,  
                                     strerror(errno));  
                         } else if (mkdir(dotsshdir, 0700) == -1) {  
                                 error("Could not create directory '%s': %s",  
                                     dotsshdir, strerror(errno));  
                         } else if (!quiet)  
                                 printf("Created directory '%s'.\n", dotsshdir);  
                 }  
         }  
         /* If the file already exists, ask the user to confirm. */          /* If the file already exists, ask the user to confirm. */
         if (!confirm_overwrite(identity_file))          if (!confirm_overwrite(identity_file))
                 exit(1);                  exit(1);
Line 3634 
Line 3642 
         /* Save the key with the given passphrase and comment. */          /* Save the key with the given passphrase and comment. */
         if ((r = sshkey_save_private(private, identity_file, passphrase,          if ((r = sshkey_save_private(private, identity_file, passphrase,
             comment, private_key_format, openssh_format_cipher, rounds)) != 0) {              comment, private_key_format, openssh_format_cipher, rounds)) != 0) {
                 error("Saving key \"%s\" failed: %s",                  error_r(r, "Saving key \"%s\" failed", identity_file);
                     identity_file, ssh_err(r));  
                 freezero(passphrase, strlen(passphrase));                  freezero(passphrase, strlen(passphrase));
                 exit(1);                  exit(1);
         }          }
Line 3648 
Line 3655 
         }          }
   
         strlcat(identity_file, ".pub", sizeof(identity_file));          strlcat(identity_file, ".pub", sizeof(identity_file));
         if ((r = sshkey_save_public(public, identity_file, comment)) != 0) {          if ((r = sshkey_save_public(public, identity_file, comment)) != 0)
                 fatal("Unable to save public key to %s: %s",                  fatal_r(r, "Unable to save public key to %s", identity_file);
                     identity_file, strerror(errno));  
         }  
   
         if (!quiet) {          if (!quiet) {
                 fp = sshkey_fingerprint(public, fingerprint_hash,                  fp = sshkey_fingerprint(public, fingerprint_hash,
Line 3670 
Line 3675 
                 free(fp);                  free(fp);
         }          }
   
         if (sk_attestaion_path != NULL) {          if (sk_attestation_path != NULL)
                 if (attest == NULL || sshbuf_len(attest) == 0) {                  save_attestation(attest, sk_attestation_path);
                         fatal("Enrollment did not return attestation "  
                             "certificate");  
                 }  
                 if ((r = sshbuf_write_file(sk_attestaion_path, attest)) != 0) {  
                         fatal("Unable to write attestation certificate "  
                             "\"%s\": %s", sk_attestaion_path, ssh_err(r));  
                 }  
                 if (!quiet) {  
                         printf("Your FIDO attestation certificate has been "  
                             "saved in %s\n", sk_attestaion_path);  
                 }  
         }  
         sshbuf_free(attest);          sshbuf_free(attest);
         sshkey_free(public);          sshkey_free(public);
   

Legend:
Removed from v.1.406  
changed lines
  Added in v.1.421