aboutsummaryrefslogtreecommitdiffstats
path: root/g10/getkey.c
diff options
context:
space:
mode:
authorWerner Koch <[email protected]>2023-05-25 14:43:37 +0000
committerWerner Koch <[email protected]>2023-05-25 14:43:44 +0000
commit9f2f7a51b2430056168363828722afc6c7488946 (patch)
tree463ad4f56832eadc2b58dcf000aaf67d21a83266 /g10/getkey.c
parentgpg: Fix searching for the ADSK key when adding an ADSK. (diff)
downloadgnupg-9f2f7a51b2430056168363828722afc6c7488946.tar.gz
gnupg-9f2f7a51b2430056168363828722afc6c7488946.zip
gpg: Skip keys found via ADSKs.
* g10/encrypt.c (write_pubkey_enc): Indicate encryption to an ADSK. * g10/getkey.c (finish_lookup): Skip ADKS keys. -- If a key is searched by fingerprint or keyid and it happens that this is an ADSK (subkey with the RENC usage), we need to skip this key because it is not the key we actually want to encrypt to. The actual ADSK key is taken later by looking at all subkeys of the actual selected key. This is related to GnuPG-bug-id: 6504
Diffstat (limited to 'g10/getkey.c')
-rw-r--r--g10/getkey.c37
1 files changed, 22 insertions, 15 deletions
diff --git a/g10/getkey.c b/g10/getkey.c
index 68d7ee61d..21ffd5cfa 100644
--- a/g10/getkey.c
+++ b/g10/getkey.c
@@ -3640,24 +3640,31 @@ finish_lookup (kbnode_t keyblock, unsigned int req_usage, int want_exact,
log_assert (keyblock->pkt->pkttype == PKT_PUBLIC_KEY);
/* For an exact match mark the primary or subkey that matched the
- low-level search criteria. */
- if (want_exact)
+ * low-level search criteria. Use this loop also to sort our keys
+ * found using an ADSK fingerprint. */
+ for (k = keyblock; k; k = k->next)
{
- for (k = keyblock; k; k = k->next)
- {
- if ((k->flag & 1))
- {
- log_assert (k->pkt->pkttype == PKT_PUBLIC_KEY
- || k->pkt->pkttype == PKT_PUBLIC_SUBKEY);
- foundk = k;
+ if ((k->flag & 1) && (k->pkt->pkttype == PKT_PUBLIC_KEY
+ || k->pkt->pkttype == PKT_PUBLIC_SUBKEY))
+ {
+ if (want_exact)
+ {
+ if (DBG_LOOKUP)
+ log_debug ("finish_lookup: exact search requested and found\n");
+ foundk = k;
pk = k->pkt->pkt.public_key;
pk->flags.exact = 1;
- break;
- }
- }
- if (DBG_LOOKUP)
- log_debug ("finish_lookup: exact search requested: %sfound\n",
- foundk? "":"not ");
+ break;
+ }
+ else if ((k->pkt->pkt.public_key->pubkey_usage == PUBKEY_USAGE_RENC))
+ {
+ if (DBG_LOOKUP)
+ log_debug ("finish_lookup: found via ADSK - not selected\n");
+ if (r_flags)
+ *r_flags |= LOOKUP_NOT_SELECTED;
+ return NULL; /* Not found. */
+ }
+ }
}
/* Get the user id that matched that low-level search criteria. */