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

Diff for /src/usr.bin/ssh/ssh-sk-helper.c between version 1.10 and 1.11

version 1.10, 2020/05/26 01:59:46 version 1.11, 2020/10/18 11:32:02
Line 64 
Line 64 
         free(msg);          free(msg);
   
         if (r >= 0)          if (r >= 0)
                 fatal("%s: invalid error code %d", __func__, r);                  fatal_f("invalid error code %d", r);
   
         if ((resp = sshbuf_new()) == NULL)          if ((resp = sshbuf_new()) == NULL)
                 fatal("%s: sshbuf_new failed", __progname);                  fatal("%s: sshbuf_new failed", __progname);
Line 103 
Line 103 
             (r = sshbuf_get_cstring(req, NULL, NULL)) != 0 || /* alg */              (r = sshbuf_get_cstring(req, NULL, NULL)) != 0 || /* alg */
             (r = sshbuf_get_u32(req, &compat)) != 0 ||              (r = sshbuf_get_u32(req, &compat)) != 0 ||
             (r = sshbuf_get_cstring(req, &pin, NULL)) != 0)              (r = sshbuf_get_cstring(req, &pin, NULL)) != 0)
                 fatal("%s: buffer error: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: parse", __progname);
         if (sshbuf_len(req) != 0)          if (sshbuf_len(req) != 0)
                 fatal("%s: trailing data in request", __progname);                  fatal("%s: trailing data in request", __progname);
   
         if ((r = sshkey_private_deserialize(kbuf, &key)) != 0)          if ((r = sshkey_private_deserialize(kbuf, &key)) != 0)
                 fatal("Unable to parse private key: %s", ssh_err(r));                  fatal_r(r, "%s: Unable to parse private key", __progname);
         if (!sshkey_is_sk(key))          if (!sshkey_is_sk(key)) {
                 fatal("Unsupported key type %s", sshkey_ssh_name(key));                  fatal("%s: Unsupported key type %s",
                       __progname, sshkey_ssh_name(key));
           }
   
         debug("%s: ready to sign with key %s, provider %s: "          debug_f("ready to sign with key %s, provider %s: "
             "msg len %zu, compat 0x%lx", __progname, sshkey_type(key),              "msg len %zu, compat 0x%lx", sshkey_type(key),
             provider, msglen, (u_long)compat);              provider, msglen, (u_long)compat);
   
         null_empty(&pin);          null_empty(&pin);
Line 129 
Line 131 
   
         if ((r = sshbuf_put_u32(resp, SSH_SK_HELPER_SIGN)) != 0 ||          if ((r = sshbuf_put_u32(resp, SSH_SK_HELPER_SIGN)) != 0 ||
             (r = sshbuf_put_string(resp, sig, siglen)) != 0)              (r = sshbuf_put_string(resp, sig, siglen)) != 0)
                 fatal("%s: buffer error: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: compose", __progname);
  out:   out:
         sshkey_free(key);          sshkey_free(key);
         sshbuf_free(kbuf);          sshbuf_free(kbuf);
Line 163 
Line 165 
             (r = sshbuf_get_u8(req, &flags)) != 0 ||              (r = sshbuf_get_u8(req, &flags)) != 0 ||
             (r = sshbuf_get_cstring(req, &pin, NULL)) != 0 ||              (r = sshbuf_get_cstring(req, &pin, NULL)) != 0 ||
             (r = sshbuf_froms(req, &challenge)) != 0)              (r = sshbuf_froms(req, &challenge)) != 0)
                 fatal("%s: buffer error: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: parse", __progname);
         if (sshbuf_len(req) != 0)          if (sshbuf_len(req) != 0)
                 fatal("%s: trailing data in request", __progname);                  fatal("%s: trailing data in request", __progname);
   
Line 186 
Line 188 
         if ((resp = sshbuf_new()) == NULL)          if ((resp = sshbuf_new()) == NULL)
                 fatal("%s: sshbuf_new failed", __progname);                  fatal("%s: sshbuf_new failed", __progname);
         if ((r = sshkey_private_serialize(key, kbuf)) != 0)          if ((r = sshkey_private_serialize(key, kbuf)) != 0)
                 fatal("%s: serialize private key: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: encode key", __progname);
         if ((r = sshbuf_put_u32(resp, SSH_SK_HELPER_ENROLL)) != 0 ||          if ((r = sshbuf_put_u32(resp, SSH_SK_HELPER_ENROLL)) != 0 ||
             (r = sshbuf_put_stringb(resp, kbuf)) != 0 ||              (r = sshbuf_put_stringb(resp, kbuf)) != 0 ||
             (r = sshbuf_put_stringb(resp, attest)) != 0)              (r = sshbuf_put_stringb(resp, attest)) != 0)
                 fatal("%s: buffer error: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: compose", __progname);
   
  out:   out:
         sshkey_free(key);          sshkey_free(key);
Line 220 
Line 222 
         if ((r = sshbuf_get_cstring(req, &provider, NULL)) != 0 ||          if ((r = sshbuf_get_cstring(req, &provider, NULL)) != 0 ||
             (r = sshbuf_get_cstring(req, &device, NULL)) != 0 ||              (r = sshbuf_get_cstring(req, &device, NULL)) != 0 ||
             (r = sshbuf_get_cstring(req, &pin, NULL)) != 0)              (r = sshbuf_get_cstring(req, &pin, NULL)) != 0)
                 fatal("%s: buffer error: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: parse", __progname);
         if (sshbuf_len(req) != 0)          if (sshbuf_len(req) != 0)
                 fatal("%s: trailing data in request", __progname);                  fatal("%s: trailing data in request", __progname);
   
Line 238 
Line 240 
                 fatal("%s: sshbuf_new failed", __progname);                  fatal("%s: sshbuf_new failed", __progname);
   
         if ((r = sshbuf_put_u32(resp, SSH_SK_HELPER_LOAD_RESIDENT)) != 0)          if ((r = sshbuf_put_u32(resp, SSH_SK_HELPER_LOAD_RESIDENT)) != 0)
                 fatal("%s: buffer error: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: compose", __progname);
   
         for (i = 0; i < nkeys; i++) {          for (i = 0; i < nkeys; i++) {
                 debug("%s: key %zu %s %s", __func__, i,                  debug_f("key %zu %s %s", i, sshkey_type(keys[i]),
                     sshkey_type(keys[i]), keys[i]->sk_application);                      keys[i]->sk_application);
                 sshbuf_reset(kbuf);                  sshbuf_reset(kbuf);
                 if ((r = sshkey_private_serialize(keys[i], kbuf)) != 0)                  if ((r = sshkey_private_serialize(keys[i], kbuf)) != 0)
                         fatal("%s: serialize private key: %s",                          fatal_r(r, "%s: encode key", __progname);
                             __progname, ssh_err(r));  
                 if ((r = sshbuf_put_stringb(resp, kbuf)) != 0 ||                  if ((r = sshbuf_put_stringb(resp, kbuf)) != 0 ||
                     (r = sshbuf_put_cstring(resp, "")) != 0) /* comment */                      (r = sshbuf_put_cstring(resp, "")) != 0) /* comment */
                         fatal("%s: buffer error: %s", __progname, ssh_err(r));                          fatal_r(r, "%s: compose key", __progname);
         }          }
   
  out:   out:
Line 308 
Line 309 
         if (ssh_msg_recv(in, req) < 0)          if (ssh_msg_recv(in, req) < 0)
                 fatal("ssh_msg_recv failed");                  fatal("ssh_msg_recv failed");
         close(in);          close(in);
         debug("%s: received message len %zu", __progname, sshbuf_len(req));          debug_f("received message len %zu", sshbuf_len(req));
   
         if ((r = sshbuf_get_u8(req, &version)) != 0)          if ((r = sshbuf_get_u8(req, &version)) != 0)
                 fatal("%s: buffer error: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: parse version", __progname);
         if (version != SSH_SK_HELPER_VERSION) {          if (version != SSH_SK_HELPER_VERSION) {
                 fatal("unsupported version: received %d, expected %d",                  fatal("unsupported version: received %d, expected %d",
                     version, SSH_SK_HELPER_VERSION);                      version, SSH_SK_HELPER_VERSION);
Line 320 
Line 321 
         if ((r = sshbuf_get_u32(req, &rtype)) != 0 ||          if ((r = sshbuf_get_u32(req, &rtype)) != 0 ||
             (r = sshbuf_get_u8(req, &log_stderr)) != 0 ||              (r = sshbuf_get_u8(req, &log_stderr)) != 0 ||
             (r = sshbuf_get_u32(req, &ll)) != 0)              (r = sshbuf_get_u32(req, &ll)) != 0)
                 fatal("%s: buffer error: %s", __progname, ssh_err(r));                  fatal_r(r, "%s: parse", __progname);
   
         if (!vflag && log_level_name((LogLevel)ll) != NULL)          if (!vflag && log_level_name((LogLevel)ll) != NULL)
                 log_init(__progname, (LogLevel)ll, log_facility, log_stderr);                  log_init(__progname, (LogLevel)ll, log_facility, log_stderr);
Line 339 
Line 340 
                 fatal("%s: unsupported request type %u", __progname, rtype);                  fatal("%s: unsupported request type %u", __progname, rtype);
         }          }
         sshbuf_free(req);          sshbuf_free(req);
         debug("%s: reply len %zu", __progname, sshbuf_len(resp));          debug_f("reply len %zu", sshbuf_len(resp));
   
         if (ssh_msg_send(out, SSH_SK_HELPER_VERSION, resp) == -1)          if (ssh_msg_send(out, SSH_SK_HELPER_VERSION, resp) == -1)
                 fatal("ssh_msg_send failed");                  fatal("ssh_msg_send failed");
Legend:
Removed from v.1.10  
changed lines
  Added in v.1.11