aboutsummaryrefslogtreecommitdiffstats
path: root/scd/app.c
diff options
context:
space:
mode:
authorWerner Koch <[email protected]>2020-11-26 11:36:44 +0000
committerWerner Koch <[email protected]>2020-11-26 14:56:37 +0000
commit7113263a00d8c9b09f0dfdb9590bfe2bab1bc776 (patch)
treec0fca9544130109763875b1175da38feffcb1af0 /scd/app.c
parentscd: Do not try to use a non-enabled app after card switching. (diff)
downloadgnupg-7113263a00d8c9b09f0dfdb9590bfe2bab1bc776.tar.gz
gnupg-7113263a00d8c9b09f0dfdb9590bfe2bab1bc776.zip
agent: Fix YK s/n and prettify the request card prompt for Yubikeys
* agent/divert-scd.c (ask_for_card): Detect and re-format the Yubikey prompt. * scd/app.c (app_munge_serialno): Fix Yubikey s/n munging. (card_get_dispserialno): Ditto. * scd/app-openpgp.c (get_disp_serialno): Remove. (get_prompt_info): Use app_get_dispserialno.-- --
Diffstat (limited to 'scd/app.c')
-rw-r--r--scd/app.c28
1 files changed, 19 insertions, 9 deletions
diff --git a/scd/app.c b/scd/app.c
index 933ad46d6..8a809e888 100644
--- a/scd/app.c
+++ b/scd/app.c
@@ -1208,10 +1208,22 @@ app_munge_serialno (card_t card)
buf[7] = 0; /* thus we use 0.0 and don't use this directly. */
buf[8] = 0; /* Manufacturer: Yubico (0x0006). */
buf[9] = 6;
- buf[10] = (sn >> 24);
- buf[11] = (sn >> 16);
- buf[12] = (sn >> 8);
- buf[13] = sn;
+ buf[13] = (sn % 10);
+ sn /= 10;
+ buf[13] |= (sn % 10) << 4;
+ sn /= 10;
+ buf[12] = (sn % 10);
+ sn /= 10;
+ buf[12] |= (sn % 10) << 4;
+ sn /= 10;
+ buf[11] = (sn % 10);
+ sn /= 10;
+ buf[11] |= (sn % 10) << 4;
+ sn /= 10;
+ buf[10] = (sn % 10);
+ sn /= 10;
+ buf[10] |= (sn % 10) << 4;
+ sn /= 10;
buf[14] = 0; /* Last two bytes are RFU. */
buf[15] = 0;
xfree (card->serialno);
@@ -1311,15 +1323,13 @@ card_get_dispserialno (card_t card, int nofallback)
else if (card && card->cardtype == CARDTYPE_YUBIKEY)
{
/* Get back the printed Yubikey number from the OpenPGP AID
- * Example: D2760001240100000006008A77C10000
+ * Example: D2760001240100000006120808620000
*/
result = card_get_serialno (card);
if (result && strlen (result) >= 28 && !strncmp (result+16, "0006", 4))
{
- sn = xtoi_2 (result+20) * 16777216;
- sn += xtoi_2 (result+22) * 65536;
- sn += xtoi_2 (result+24) * 256;
- sn += xtoi_2 (result+26);
+ sn = atoi_4 (result+20) * 10000;
+ sn += atoi_4 (result+24);
if ((card->cardversion >> 16) >= 5)
p = xtryasprintf ("%lu %03lu %03lu",
(sn/1000000ul),