[Pcsclite-cvs-commit] r7040 - trunk/Drivers/ccid/src

[ludovic.rousseau at free.fr]

Author: rousseau
Date: 2014-11-09 20:35:19 +0000 (Sun, 09 Nov 2014)
New Revision: 7040

Modified:
   trunk/Drivers/ccid/src/ccid.c
Log:
Use CmdEscapeCheck() in set_gemalto_firmware_features()

Not all the Gemplus or Gemalto readers do suppoort the proprietary
GET_FIRMWARE_FEATURES command.

In case the reader is too old to support the command it is annoying and
worrying to get an error message in the log.

Closes Alioth bug [#314871] "commands.c:979:CmdEscape error on byte 10"
message
https://alioth.debian.org/tracker/index.php?func=detail&aid=314871&group_id=30105&atid=410086


Modified: trunk/Drivers/ccid/src/ccid.c
===================================================================
--- trunk/Drivers/ccid/src/ccid.c	2014-11-09 20:32:32 UTC (rev 7039)
+++ trunk/Drivers/ccid/src/ccid.c	2014-11-09 20:35:19 UTC (rev 7040)
@@ -195,8 +195,8 @@
 		unsigned int len_features = sizeof *gf_features;
 		RESPONSECODE ret;
 
-		ret = CmdEscape(reader_index, cmd, sizeof cmd,
-			(unsigned char*)gf_features, &len_features, 0);
+		ret = CmdEscapeCheck(reader_index, cmd, sizeof cmd,
+			(unsigned char*)gf_features, &len_features, 0, TRUE);
 		if ((IFD_SUCCESS == ret) &&
 		    (len_features == sizeof *gf_features))
 		{