g10: Simplify get_seckey_byname: it was never called with NAME not NULL.
authorNeal H. Walfield <neal@g10code.com>
Mon, 14 Sep 2015 19:14:00 +0000 (21:14 +0200)
committerNeal H. Walfield <neal@g10code.com>
Wed, 16 Sep 2015 13:15:23 +0000 (15:15 +0200)
* g10/keydb.h (get_seckey_byname): Rename from this...
(get_seckey_default): ... to this.  Drop the parameter name.  Update
users.
* g10/getkey.c (get_seckey_byname): Rename from this...
(get_seckey_default): ... to this.  Drop the parameter name.  Drop the
code which assumed that NAME is not NULL.

--
Signed-off-by: Neal H. Walfield <neal@g10code.com>.
g10/getkey.c
g10/keydb.h
g10/pkclist.c

index 5cb0807..7096066 100644 (file)
@@ -1029,19 +1029,14 @@ get_pubkey_byfprint_fast (PKT_public_key * pk,
  * the default key.  This functions checks that a corresponding secret
  * key is available.  With no secret key it does not succeeed. */
 gpg_error_t
-get_seckey_byname (PKT_public_key *pk, const char *name)
+get_seckey_default (PKT_public_key *pk)
 {
   gpg_error_t err;
   strlist_t namelist = NULL;
   int include_unusable = 1;
 
-  /* If we have no name, try to use the default secret key.  If we
-     have no default, we'll use the first usable one. */
-
-  if (!name && opt.def_secret_key && *opt.def_secret_key)
+  if (opt.def_secret_key && *opt.def_secret_key)
     add_to_strlist (&namelist, opt.def_secret_key);
-  else if (name)
-    add_to_strlist (&namelist, name);
   else
     include_unusable = 0;
 
index 0bc60bf..e160a87 100644 (file)
@@ -419,7 +419,7 @@ int get_pubkey_byfprint_fast (PKT_public_key *pk,
    whether there is some secret key with the specified key id.  */
 int have_secret_key_with_kid (u32 *keyid);
 
-gpg_error_t get_seckey_byname (PKT_public_key *pk, const char *name);
+gpg_error_t get_seckey_default (PKT_public_key *pk);
 
 gpg_error_t get_seckey_byfprint (PKT_public_key *pk,
                                  const byte *fprint, size_t fprint_len);
index 68d2812..9996d18 100644 (file)
@@ -714,7 +714,7 @@ default_recipient(void)
     if( !opt.def_recipient_self )
        return NULL;
     pk = xmalloc_clear( sizeof *pk );
-    i = get_seckey_byname (pk, NULL);
+    i = get_seckey_default (pk);
     if( i ) {
        free_public_key( pk );
        return NULL;