* trustdb.c (update_min_ownertrust, validate_keys): Do not use keystr
authorDavid Shaw <dshaw@jabberwocky.com>
Fri, 19 Mar 2004 23:15:27 +0000 (23:15 +0000)
committerDavid Shaw <dshaw@jabberwocky.com>
Fri, 19 Mar 2004 23:15:27 +0000 (23:15 +0000)
functions in log_debug.

* import.c (import_one): Try and collapse user IDs when importing a key
for the first time.

* keyedit.c (menu_addrevoker): Allow appointing a subkey as a designated
revoker if the user forces it via keyid!, so long as the subkey can
certify.  Also use the proper date string when prompting for confirmation.

* g10.c (main): Maintain ordering of multiple Comment lines. Requested by
Peter Hyman.

g10/ChangeLog
g10/g10.c
g10/import.c
g10/keyedit.c
g10/trustdb.c

index 017150c..466bd76 100644 (file)
@@ -1,3 +1,19 @@
+2004-03-19  David Shaw  <dshaw@jabberwocky.com>
+
+       * trustdb.c (update_min_ownertrust, validate_keys): Do not use
+       keystr functions in log_debug.
+
+       * import.c (import_one): Try and collapse user IDs when importing
+       a key for the first time.
+
+       * keyedit.c (menu_addrevoker): Allow appointing a subkey as a
+       designated revoker if the user forces it via keyid!, so long as
+       the subkey can certify.  Also use the proper date string when
+       prompting for confirmation.
+
+       * g10.c (main): Maintain ordering of multiple Comment lines.
+       Requested by Peter Hyman.
+
 2004-03-17  David Shaw  <dshaw@jabberwocky.com>
 
        * mainproc.c (proc_pubkey_enc, print_pkenc_list, list_node):
index 3e5dd29..4e1c36b 100644 (file)
--- a/g10/g10.c
+++ b/g10/g10.c
@@ -1929,7 +1929,7 @@ main( int argc, char **argv )
          case oUseEmbeddedFilename: opt.use_embedded_filename = 1; break;
          case oComment:
            if(pargs.r.ret_str[0])
-             add_to_strlist(&opt.comments,pargs.r.ret_str);
+             append_to_strlist(&opt.comments,pargs.r.ret_str);
            break;
          case oDefaultComment:
            deprecated_warning(configname,configlineno,
index b5a9339..c1110bf 100644 (file)
@@ -758,6 +758,9 @@ import_one( const char *fname, KBNODE keyblock,
        }
        if( opt.verbose > 1 )
            log_info (_("writing to `%s'\n"), keydb_get_resource_name (hd) );
+
+       collapse_uids(&keyblock);
+
        rc = keydb_insert_keyblock (hd, keyblock );
         if (rc)
           log_error (_("error writing keyring `%s': %s\n"),
index 0b648dc..9b488b0 100644 (file)
@@ -2685,8 +2685,10 @@ menu_addrevoker( KBNODE pub_keyblock, KBNODE sec_keyblock, int sensitive )
          goto fail;
        }
 
+      /* Note that I'm requesting SIG here and not CERT.  We're making
+        a certification, but it is okay to be a subkey. */
+      revoker_pk->req_usage=PUBKEY_USAGE_SIG;
       rc=get_pubkey_byname(revoker_pk,answer,NULL,NULL,1);
-
       if(rc)
        {
          log_error (_("key `%s' not found: %s\n"),answer,g10_errstr(rc));
@@ -2756,7 +2758,7 @@ menu_addrevoker( KBNODE pub_keyblock, KBNODE sec_keyblock, int sensitive )
       tty_printf("\npub   %4u%c/%08lX %s   ",
                 nbits_from_pk( revoker_pk ),
                 pubkey_letter( revoker_pk->pubkey_algo ),
-                (ulong)keyid[1], datestr_from_pk(pk) );
+                (ulong)keyid[1], datestr_from_pk(revoker_pk) );
 
       p = get_user_id( keyid, &n );
       tty_print_utf8_string( p, n );
index 67f9211..5cb69cc 100644 (file)
@@ -797,8 +797,9 @@ update_min_ownertrust (u32 *kid, unsigned int new_trust )
   if (!rc)
     {
       if (DBG_TRUST)
-        log_debug ("key %s: update min_ownertrust from %u to %u\n",
-                   keystr(kid),(unsigned int)rec.r.trust.min_ownertrust,
+        log_debug ("key %08lX%08lX: update min_ownertrust from %u to %u\n",
+                   (ulong)kid[0],(ulong)kid[1],
+                  (unsigned int)rec.r.trust.min_ownertrust,
                   new_trust );
       if (rec.r.trust.min_ownertrust != new_trust)
         {
@@ -1985,9 +1986,9 @@ validate_keys (int interactive)
          if(k->ownertrust<min)
            {
              if(DBG_TRUST)
-               log_debug("key %s:"
+               log_debug("key %08lX%08lX:"
                          " overriding ownertrust \"%s\" with \"%s\"\n",
-                         keystr(k->kid),
+                         (ulong)k->kid[0],(ulong)k->kid[1],
                          trust_value_to_string(k->ownertrust),
                          trust_value_to_string(min));