From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41330) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDDlp-0002x5-Hk for qemu-devel@nongnu.org; Mon, 19 Jan 2015 09:57:38 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YDDlm-0006Re-Cz for qemu-devel@nongnu.org; Mon, 19 Jan 2015 09:57:37 -0500 Received: from mail.codeweavers.com ([216.251.189.131]:48981) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDDlm-0006RZ-8n for qemu-devel@nongnu.org; Mon, 19 Jan 2015 09:57:34 -0500 Received: from jwhite.mn.codeweavers.com ([10.69.137.101] helo=jwhite.mn.codeweavers.com.) by mail.codeweavers.com with esmtpsa (TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80) (envelope-from ) id 1YDDll-00011x-Ss for qemu-devel@nongnu.org; Mon, 19 Jan 2015 08:57:33 -0600 From: Jeremy White Date: Mon, 19 Jan 2015 08:57:31 -0600 Message-Id: <1421679451-9096-1-git-send-email-jwhite@codeweavers.com> Subject: [Qemu-devel] [PATCH] Retrieve the correct TD byte when checking an ATR. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org A physical smartcard with an ATR of 3B 95 95 40 FF AE 01 0E 00 00 was parsed incorrectly. The '40' should have been the second TD; instead the FF is used, incorrectly. Signed-off-by: Jeremy White --- hw/usb/ccid-card-passthru.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/usb/ccid-card-passthru.c b/hw/usb/ccid-card-passthru.c index 10f1d30..2ae3b81 100644 --- a/hw/usb/ccid-card-passthru.c +++ b/hw/usb/ccid-card-passthru.c @@ -168,8 +168,8 @@ static int check_atr(PassthruState *card, uint8_t *data, int len) opt_bytes++; } if (td & 0x8) { - opt_bytes++; td = data[opt_bytes + 2] >> 4; + opt_bytes++; } } if (len < 2 + historical_length + opt_bytes) { -- 1.7.10.4