diff options
| author | Werner Koch <[email protected]> | 2013-08-21 13:44:52 +0000 |
|---|---|---|
| committer | Werner Koch <[email protected]> | 2013-08-21 13:45:07 +0000 |
| commit | 7bde2bf3b0ddb5d3515a44879e1a7ddb581a5c0b (patch) | |
| tree | 63deab699772dee3b8da4dfdddd07f9449c7eed4 | |
| parent | agent: Extend cmd KEYINFO to return data from sshcontrol. (diff) | |
| download | gnupg-7bde2bf3b0ddb5d3515a44879e1a7ddb581a5c0b.tar.gz gnupg-7bde2bf3b0ddb5d3515a44879e1a7ddb581a5c0b.zip | |
scd: Improve --enable-pinpad-varlen.
* tools/gpgconf-comp.c (gc_options_scdaemon): Add
enable-pinpad-varlen.
* scd/apdu.c (check_pcsc_pinpad): Detect SPRx32 reader.
Signed-off-by: Werner Koch <[email protected]>
| -rw-r--r-- | doc/scdaemon.texi | 8 | ||||
| -rw-r--r-- | scd/apdu.c | 8 | ||||
| -rw-r--r-- | tools/gpgconf-comp.c | 4 |
3 files changed, 15 insertions, 5 deletions
diff --git a/doc/scdaemon.texi b/doc/scdaemon.texi index ed2cc5129..f454f1428 100644 --- a/doc/scdaemon.texi +++ b/doc/scdaemon.texi @@ -290,10 +290,10 @@ than 0. @item --enable-pinpad-varlen @opindex enable-pinpad-varlen Please specify this option when the card reader supports variable -length input for pinpad (default is no). For known readers listed in -ccid-driver, this option is not needed. Note that if your card reader -doesn't supports variable length input but you want to use it, you -need to specify your pinpad request on your card. +length input for pinpad (default is no). For known readers (listed in +ccid-driver.c and apdu.c), this option is not needed. Note that if +your card reader doesn't supports variable length input but you want +to use it, you need to specify your pinpad request on your card. @item --disable-pinpad diff --git a/scd/apdu.c b/scd/apdu.c index 133058d59..ca45fc18c 100644 --- a/scd/apdu.c +++ b/scd/apdu.c @@ -2002,7 +2002,13 @@ check_pcsc_pinpad (int slot, int command, pininfo_t *pininfo) pcsc_dword_t len = 256; int sw; - (void)pininfo; /* XXX: Identify reader and set pininfo->fixedlen. */ + /* Hack to identify the SCM SPR532 and SPR332 readers which support + variable length PIN input. + FIXME: Figure out whether there is a feature attribute for this. + Alternatively use the USB ids to detect known readers. */ + if (reader_table[slot].rdrname + && strstr (reader_table[slot].rdrname, "SPRx32")) + pininfo->fixedlen = 0; check_again: if (command == ISO7816_VERIFY) diff --git a/tools/gpgconf-comp.c b/tools/gpgconf-comp.c index 0b3c9c0e7..ffe7eab09 100644 --- a/tools/gpgconf-comp.c +++ b/tools/gpgconf-comp.c @@ -620,6 +620,10 @@ static gc_option_t gc_options_scdaemon[] = { "disable-pinpad", GC_OPT_FLAG_NONE|GC_OPT_FLAG_RUNTIME, GC_LEVEL_BASIC, "gnupg", "do not use a reader's pinpad", GC_ARG_TYPE_NONE, GC_BACKEND_SCDAEMON }, + { "enable-pinpad-varlen", + GC_OPT_FLAG_NONE|GC_OPT_FLAG_RUNTIME, GC_LEVEL_BASIC, + "gnupg", "use variable length input for pinpad", + GC_ARG_TYPE_NONE, GC_BACKEND_SCDAEMON }, { "card-timeout", GC_OPT_FLAG_NONE|GC_OPT_FLAG_RUNTIME, GC_LEVEL_BASIC, "gnupg", "|N|disconnect the card after N seconds of inactivity", GC_ARG_TYPE_UINT32, GC_BACKEND_SCDAEMON }, |