From 2e8f2c693d4dac19abda47655a3e385a40dd91ea Mon Sep 17 00:00:00 2001 From: Jukka-Pekka Virtanen Date: Thu, 11 Jan 2018 16:19:02 +0200 Subject: [PATCH] [MS-RDPESC] 2.2.2.18 Status_Call.cbAtrLen: Unused. MUST be ignored upon receipt. The [MS-RDPESC] clearly states in section 2.2.2.18 that Status_Call's cbAtrLen must not be used when generating Status_Return response. This is also how FreeRDP 1.1 behaves. --- channels/smartcard/client/smartcard_operations.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/channels/smartcard/client/smartcard_operations.c b/channels/smartcard/client/smartcard_operations.c index 05d4f2fcb..80dd4d007 100644 --- a/channels/smartcard/client/smartcard_operations.c +++ b/channels/smartcard/client/smartcard_operations.c @@ -1054,9 +1054,7 @@ static LONG smartcard_StatusA_Call(SMARTCARD_DEVICE* smartcard, SMARTCARD_OPERAT Status_Call* call = operation->call; ZeroMemory(ret.pbAtr, 32); - if (call->cbAtrLen > 32) - call->cbAtrLen = 32; - + call->cbAtrLen = 32; cbAtrLen = call->cbAtrLen; if (call->fmszReaderNamesIsNULL)