security-and-privacy/gnupg
security-and-privacy/gnupg
1
0
Fork 0
mirror of git://git.gnupg.org/gnupg.git synced 2024-06-02 22:38:02 +02:00
Code Activity

gpg: Put the first key in candidates correctly.

Browse Source 
* g10/getkey.c (get_best_pubkey_byname): After the call of
get_pubkey_byname, set up CTX with KEYDB_SEARCH_MODE_LONG_KID to enter
the loop.

--

Fixes-commit: 7535f1d47a
GnuPG-bug-id: 4713
Signed-off-by: NIIBE Yutaka <gniibe@fsij.org>
This commit is contained in:
NIIBE Yutaka 2019-10-15 19:18:26 +09:00
parent c933c15d58
commit 44604209c1
1 changed files with 18 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
g10/getkey.c
View File

@ -1353,6 +1353,7 @@ get_best_pubkey_byname (ctrl_t ctrl, enum get_pubkey_modes mode,
struct getkey_ctx_s *ctx = NULL; struct getkey_ctx_s *ctx = NULL;
int is_mbox = is_valid_mailbox (name); int is_mbox = is_valid_mailbox (name);
int wkd_tried = 0; int wkd_tried = 0;
PKT_public_key *pk2;
log_assert (ret_keyblock != NULL); log_assert (ret_keyblock != NULL);
@ -1362,11 +1363,8 @@ get_best_pubkey_byname (ctrl_t ctrl, enum get_pubkey_modes mode,
start_over: start_over:
if (ctx) /* Clear in case of a start over. */ if (ctx) /* Clear in case of a start over. */
{ {
if (ret_keyblock) release_kbnode (*ret_keyblock);
{ *ret_keyblock = NULL;
release_kbnode (*ret_keyblock);
*ret_keyblock = NULL;
}
getkey_end (ctrl, ctx); getkey_end (ctrl, ctx);
ctx = NULL; ctx = NULL;
} }
@ -1379,13 +1377,14 @@ get_best_pubkey_byname (ctrl_t ctrl, enum get_pubkey_modes mode,
return err; return err;
} }
pk2 = (*ret_keyblock)->pkt->pkt.public_key;
/* If the keyblock was retrieved from the local database and the key /* If the keyblock was retrieved from the local database and the key
* has expired, do further checks. However, we can do this only if * has expired, do further checks. However, we can do this only if
* the caller requested a keyblock. */ * the caller requested a keyblock. */
if (is_mbox && ctx && ctx->found_via_akl == AKL_LOCAL && ret_keyblock) if (is_mbox && ctx && ctx->found_via_akl == AKL_LOCAL)
{ {
u32 now = make_timestamp (); u32 now = make_timestamp ();
PKT_public_key *pk2 = (*ret_keyblock)->pkt->pkt.public_key;
int found; int found;
/* If the key has expired and its origin was the WKD then try to /* If the key has expired and its origin was the WKD then try to
@ -1417,11 +1416,15 @@ get_best_pubkey_byname (ctrl_t ctrl, enum get_pubkey_modes mode,
struct pubkey_cmp_cookie best = { 0 }; struct pubkey_cmp_cookie best = { 0 };
struct pubkey_cmp_cookie new = { 0 }; struct pubkey_cmp_cookie new = { 0 };
kbnode_t new_keyblock; kbnode_t new_keyblock;
u32 *keyid = pk_keyid (pk2);
copy_public_key (&new.key, (*ret_keyblock)->pkt->pkt.public_key); ctx->exact = 1;
new_keyblock = clone_kbnode (*ret_keyblock); ctx->nitems = 1;
ctx->items[0].mode = KEYDB_SEARCH_MODE_LONG_KID;
ctx->items[0].u.kid[0] = keyid[0];
ctx->items[0].u.kid[1] = keyid[1];
do while (getkey_next (ctrl, ctx, &new.key, &new_keyblock) == 0)
{ {
int diff = pubkey_cmp (ctrl, name, &best, &new, new_keyblock); int diff = pubkey_cmp (ctrl, name, &best, &new, new_keyblock);
release_kbnode (new_keyblock); release_kbnode (new_keyblock);
@ -1446,7 +1449,6 @@ get_best_pubkey_byname (ctrl_t ctrl, enum get_pubkey_modes mode,
} }
new.uid = NULL; new.uid = NULL;
} }
while (getkey_next (ctrl, ctx, &new.key, &new_keyblock) == 0);
getkey_end (ctrl, ctx); getkey_end (ctrl, ctx);
ctx = NULL; ctx = NULL;
@ -1455,7 +1457,7 @@ get_best_pubkey_byname (ctrl_t ctrl, enum get_pubkey_modes mode,
if (best.valid) if (best.valid)
{ {
if (retctx || ret_keyblock) if (1)
{ {
ctx = xtrycalloc (1, sizeof **retctx); ctx = xtrycalloc (1, sizeof **retctx);
if (! ctx) if (! ctx)
@ -1473,19 +1475,16 @@ get_best_pubkey_byname (ctrl_t ctrl, enum get_pubkey_modes mode,
} }
else else
{ {
u32 *keyid = pk_keyid (&best.key); keyid = pk_keyid (&best.key);
ctx->exact = 1; ctx->exact = 1;
ctx->nitems = 1; ctx->nitems = 1;
ctx->items[0].mode = KEYDB_SEARCH_MODE_LONG_KID; ctx->items[0].mode = KEYDB_SEARCH_MODE_LONG_KID;
ctx->items[0].u.kid[0] = keyid[0]; ctx->items[0].u.kid[0] = keyid[0];
ctx->items[0].u.kid[1] = keyid[1]; ctx->items[0].u.kid[1] = keyid[1];
if (ret_keyblock) release_kbnode (*ret_keyblock);
{ *ret_keyblock = NULL;
release_kbnode (*ret_keyblock); err = getkey_next (ctrl, ctx, NULL, ret_keyblock);
*ret_keyblock = NULL;
err = getkey_next (ctrl, ctx, NULL, ret_keyblock);
}
} }
} }
} }