Added debug log messages for SCARD_E_NO_SERVICE
authorArmin Novak <armin.novak@thincast.com>
Thu, 9 Jan 2020 07:20:42 +0000 (08:20 +0100)
committerArmin Novak <armin.novak@thincast.com>
Thu, 9 Jan 2020 07:20:42 +0000 (08:20 +0100)
Log where and what caused SCARD_E_NO_SERVICE return in smartcard
wrapper

winpr/libwinpr/smartcard/smartcard.c
winpr/libwinpr/smartcard/smartcard_pcsc.c

index 78710dd..4b67ea6 100644 (file)
@@ -25,6 +25,9 @@
 #include <winpr/library.h>
 #include <winpr/smartcard.h>
 #include <winpr/synch.h>
+#include <winpr/wlog.h>
+
+#include "../log.h"
 
 #include "smartcard.h"
 
 static INIT_ONCE g_Initialized = INIT_ONCE_STATIC_INIT;
 static PSCardApiFunctionTable g_SCardApi = NULL;
 
-#define SCARDAPI_STUB_CALL_LONG(_name, ...)                                   \
-       InitOnceExecuteOnce(&g_Initialized, InitializeSCardApiStubs, NULL, NULL); \
-       if (!g_SCardApi || !g_SCardApi->pfn##_name)                               \
-               return SCARD_E_NO_SERVICE;                                            \
+#define TAG WINPR_TAG("smartcard")
+
+#define xstr(s) str(s)
+#define str(s) #s
+
+#define SCARDAPI_STUB_CALL_LONG(_name, ...)                                              \
+       InitOnceExecuteOnce(&g_Initialized, InitializeSCardApiStubs, NULL, NULL);            \
+       if (!g_SCardApi || !g_SCardApi->pfn##_name)                                          \
+       {                                                                                    \
+               WLog_DBG(TAG, "Missing function pointer g_SCardApi=%p->" xstr(pfn##_name) "=%p", \
+                        g_SCardApi, g_SCardApi ? g_SCardApi->pfn##_name : NULL);                \
+               return SCARD_E_NO_SERVICE;                                                       \
+       }                                                                                    \
        return g_SCardApi->pfn##_name(__VA_ARGS__)
 
 #define SCARDAPI_STUB_CALL_HANDLE(_name, ...)                                 \
@@ -775,8 +787,6 @@ WINSCARDAPI const char* WINAPI SCardGetErrorString(LONG errorCode)
                default:
                        return "SCARD_E_UNKNOWN";
        }
-
-       return "SCARD_E_UNKNOWN";
 }
 
 WINSCARDAPI const char* WINAPI SCardGetAttributeString(DWORD dwAttrId)
@@ -785,182 +795,136 @@ WINSCARDAPI const char* WINAPI SCardGetAttributeString(DWORD dwAttrId)
        {
                case SCARD_ATTR_VENDOR_NAME:
                        return "SCARD_ATTR_VENDOR_NAME";
-                       break;
 
                case SCARD_ATTR_VENDOR_IFD_TYPE:
                        return "SCARD_ATTR_VENDOR_IFD_TYPE";
-                       break;
 
                case SCARD_ATTR_VENDOR_IFD_VERSION:
                        return "SCARD_ATTR_VENDOR_IFD_VERSION";
-                       break;
 
                case SCARD_ATTR_VENDOR_IFD_SERIAL_NO:
                        return "SCARD_ATTR_VENDOR_IFD_SERIAL_NO";
-                       break;
 
                case SCARD_ATTR_CHANNEL_ID:
                        return "SCARD_ATTR_CHANNEL_ID";
-                       break;
 
                case SCARD_ATTR_PROTOCOL_TYPES:
                        return "SCARD_ATTR_PROTOCOL_TYPES";
-                       break;
 
                case SCARD_ATTR_DEFAULT_CLK:
                        return "SCARD_ATTR_DEFAULT_CLK";
-                       break;
 
                case SCARD_ATTR_MAX_CLK:
                        return "SCARD_ATTR_MAX_CLK";
-                       break;
 
                case SCARD_ATTR_DEFAULT_DATA_RATE:
                        return "SCARD_ATTR_DEFAULT_DATA_RATE";
-                       break;
 
                case SCARD_ATTR_MAX_DATA_RATE:
                        return "SCARD_ATTR_MAX_DATA_RATE";
-                       break;
 
                case SCARD_ATTR_MAX_IFSD:
                        return "SCARD_ATTR_MAX_IFSD";
-                       break;
 
                case SCARD_ATTR_POWER_MGMT_SUPPORT:
                        return "SCARD_ATTR_POWER_MGMT_SUPPORT";
-                       break;
 
                case SCARD_ATTR_USER_TO_CARD_AUTH_DEVICE:
                        return "SCARD_ATTR_USER_TO_CARD_AUTH_DEVICE";
-                       break;
 
                case SCARD_ATTR_USER_AUTH_INPUT_DEVICE:
                        return "SCARD_ATTR_USER_AUTH_INPUT_DEVICE";
-                       break;
 
                case SCARD_ATTR_CHARACTERISTICS:
                        return "SCARD_ATTR_CHARACTERISTICS";
-                       break;
 
                case SCARD_ATTR_CURRENT_PROTOCOL_TYPE:
                        return "SCARD_ATTR_CURRENT_PROTOCOL_TYPE";
-                       break;
 
                case SCARD_ATTR_CURRENT_CLK:
                        return "SCARD_ATTR_CURRENT_CLK";
-                       break;
 
                case SCARD_ATTR_CURRENT_F:
                        return "SCARD_ATTR_CURRENT_F";
-                       break;
 
                case SCARD_ATTR_CURRENT_D:
                        return "SCARD_ATTR_CURRENT_D";
-                       break;
 
                case SCARD_ATTR_CURRENT_N:
                        return "SCARD_ATTR_CURRENT_N";
-                       break;
 
                case SCARD_ATTR_CURRENT_W:
                        return "SCARD_ATTR_CURRENT_W";
-                       break;
 
                case SCARD_ATTR_CURRENT_IFSC:
                        return "SCARD_ATTR_CURRENT_IFSC";
-                       break;
 
                case SCARD_ATTR_CURRENT_IFSD:
                        return "SCARD_ATTR_CURRENT_IFSD";
-                       break;
 
                case SCARD_ATTR_CURRENT_BWT:
                        return "SCARD_ATTR_CURRENT_BWT";
-                       break;
 
                case SCARD_ATTR_CURRENT_CWT:
                        return "SCARD_ATTR_CURRENT_CWT";
-                       break;
 
                case SCARD_ATTR_CURRENT_EBC_ENCODING:
                        return "SCARD_ATTR_CURRENT_EBC_ENCODING";
-                       break;
 
                case SCARD_ATTR_EXTENDED_BWT:
                        return "SCARD_ATTR_EXTENDED_BWT";
-                       break;
 
                case SCARD_ATTR_ICC_PRESENCE:
                        return "SCARD_ATTR_ICC_PRESENCE";
-                       break;
 
                case SCARD_ATTR_ICC_INTERFACE_STATUS:
                        return "SCARD_ATTR_ICC_INTERFACE_STATUS";
-                       break;
 
                case SCARD_ATTR_CURRENT_IO_STATE:
                        return "SCARD_ATTR_CURRENT_IO_STATE";
-                       break;
 
                case SCARD_ATTR_ATR_STRING:
                        return "SCARD_ATTR_ATR_STRING";
-                       break;
 
                case SCARD_ATTR_ICC_TYPE_PER_ATR:
                        return "SCARD_ATTR_ICC_TYPE_PER_ATR";
-                       break;
 
                case SCARD_ATTR_ESC_RESET:
                        return "SCARD_ATTR_ESC_RESET";
-                       break;
 
                case SCARD_ATTR_ESC_CANCEL:
                        return "SCARD_ATTR_ESC_CANCEL";
-                       break;
 
                case SCARD_ATTR_ESC_AUTHREQUEST:
                        return "SCARD_ATTR_ESC_AUTHREQUEST";
-                       break;
 
                case SCARD_ATTR_MAXINPUT:
                        return "SCARD_ATTR_MAXINPUT";
-                       break;
 
                case SCARD_ATTR_DEVICE_UNIT:
                        return "SCARD_ATTR_DEVICE_UNIT";
-                       break;
 
                case SCARD_ATTR_DEVICE_IN_USE:
                        return "SCARD_ATTR_DEVICE_IN_USE";
-                       break;
 
                case SCARD_ATTR_DEVICE_FRIENDLY_NAME_A:
                        return "SCARD_ATTR_DEVICE_FRIENDLY_NAME_A";
-                       break;
 
                case SCARD_ATTR_DEVICE_SYSTEM_NAME_A:
                        return "SCARD_ATTR_DEVICE_SYSTEM_NAME_A";
-                       break;
 
                case SCARD_ATTR_DEVICE_FRIENDLY_NAME_W:
                        return "SCARD_ATTR_DEVICE_FRIENDLY_NAME_W";
-                       break;
 
                case SCARD_ATTR_DEVICE_SYSTEM_NAME_W:
                        return "SCARD_ATTR_DEVICE_SYSTEM_NAME_W";
-                       break;
 
                case SCARD_ATTR_SUPRESS_T1_IFS_REQUEST:
                        return "SCARD_ATTR_SUPRESS_T1_IFS_REQUEST";
-                       break;
 
                default:
                        return "SCARD_ATTR_UNKNOWN";
-                       break;
        }
-
-       return "SCARD_ATTR_UNKNOWN";
 }
 
 WINSCARDAPI const char* WINAPI SCardGetProtocolString(DWORD dwProtocols)
@@ -1001,22 +965,16 @@ WINSCARDAPI const char* WINAPI SCardGetShareModeString(DWORD dwShareMode)
        {
                case SCARD_SHARE_EXCLUSIVE:
                        return "SCARD_SHARE_EXCLUSIVE";
-                       break;
 
                case SCARD_SHARE_SHARED:
                        return "SCARD_SHARE_SHARED";
-                       break;
 
                case SCARD_SHARE_DIRECT:
                        return "SCARD_SHARE_DIRECT";
-                       break;
 
                default:
                        return "SCARD_SHARE_UNKNOWN";
-                       break;
        }
-
-       return "SCARD_SHARE_UNKNOWN";
 }
 
 WINSCARDAPI const char* WINAPI SCardGetDispositionString(DWORD dwDisposition)
@@ -1025,22 +983,16 @@ WINSCARDAPI const char* WINAPI SCardGetDispositionString(DWORD dwDisposition)
        {
                case SCARD_LEAVE_CARD:
                        return "SCARD_LEAVE_CARD";
-                       break;
 
                case SCARD_RESET_CARD:
                        return "SCARD_RESET_CARD";
-                       break;
 
                case SCARD_UNPOWER_CARD:
                        return "SCARD_UNPOWER_CARD";
-                       break;
 
                default:
                        return "SCARD_UNKNOWN_CARD";
-                       break;
        }
-
-       return "SCARD_UNKNOWN_CARD";
 }
 
 WINSCARDAPI const char* WINAPI SCardGetScopeString(DWORD dwScope)
@@ -1049,22 +1001,16 @@ WINSCARDAPI const char* WINAPI SCardGetScopeString(DWORD dwScope)
        {
                case SCARD_SCOPE_USER:
                        return "SCARD_SCOPE_USER";
-                       break;
 
                case SCARD_SCOPE_TERMINAL:
                        return "SCARD_SCOPE_TERMINAL";
-                       break;
 
                case SCARD_SCOPE_SYSTEM:
                        return "SCARD_SCOPE_SYSTEM";
-                       break;
 
                default:
                        return "SCARD_SCOPE_UNKNOWN";
-                       break;
        }
-
-       return "SCARD_SCOPE_UNKNOWN";
 }
 
 WINSCARDAPI const char* WINAPI SCardGetCardStateString(DWORD dwCardState)
@@ -1073,38 +1019,28 @@ WINSCARDAPI const char* WINAPI SCardGetCardStateString(DWORD dwCardState)
        {
                case SCARD_UNKNOWN:
                        return "SCARD_UNKNOWN";
-                       break;
 
                case SCARD_ABSENT:
                        return "SCARD_ABSENT";
-                       break;
 
                case SCARD_PRESENT:
                        return "SCARD_PRESENT";
-                       break;
 
                case SCARD_SWALLOWED:
                        return "SCARD_SWALLOWED";
-                       break;
 
                case SCARD_POWERED:
                        return "SCARD_POWERED";
-                       break;
 
                case SCARD_NEGOTIABLE:
                        return "SCARD_NEGOTIABLE";
-                       break;
 
                case SCARD_SPECIFIC:
                        return "SCARD_SPECIFIC";
-                       break;
 
                default:
                        return "SCARD_UNKNOWN";
-                       break;
        }
-
-       return "SCARD_UNKNOWN";
 }
 
 WINSCARDAPI char* WINAPI SCardGetReaderStateString(DWORD dwReaderState)
index c804f7c..d2d645d 100644 (file)
@@ -161,6 +161,12 @@ static LONG WINAPI PCSC_SCardEstablishContext_Internal(DWORD dwScope, LPCVOID pv
                                                        LPSCARDCONTEXT phContext);
 static LONG WINAPI PCSC_SCardReleaseContext_Internal(SCARDCONTEXT hContext);
 
+static LONG PCSC_SCard_LogError(const char* what)
+{
+       WLog_DBG(TAG, "Missing function pointer %s=NULL", what);
+       return SCARD_E_NO_SERVICE;
+}
+
 static LONG PCSC_MapErrorCodeToWinSCard(LONG errorCode)
 {
        /**
@@ -593,7 +599,7 @@ static LONG WINAPI PCSC_SCardEstablishContext_Internal(DWORD dwScope, LPCVOID pv
        LONG status = SCARD_S_SUCCESS;
 
        if (!g_PCSC.pfnSCardEstablishContext)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardEstablishContext");
 
        status = (LONG)g_PCSC.pfnSCardEstablishContext(SCARD_SCOPE_SYSTEM, pvReserved1, pvReserved2,
                                                       phContext);
@@ -618,7 +624,7 @@ static LONG WINAPI PCSC_SCardReleaseContext_Internal(SCARDCONTEXT hContext)
        LONG status = SCARD_S_SUCCESS;
 
        if (!g_PCSC.pfnSCardReleaseContext)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardReleaseContext");
 
        if (!hContext)
        {
@@ -647,7 +653,7 @@ static LONG WINAPI PCSC_SCardIsValidContext(SCARDCONTEXT hContext)
        LONG status = SCARD_S_SUCCESS;
 
        if (!g_PCSC.pfnSCardIsValidContext)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardIsValidContext");
 
        status = (LONG)g_PCSC.pfnSCardIsValidContext(hContext);
        status = PCSC_MapErrorCodeToWinSCard(status);
@@ -666,7 +672,7 @@ static LONG WINAPI PCSC_SCardListReaderGroups_Internal(SCARDCONTEXT hContext, LP
                return SCARD_E_INVALID_PARAMETER;
 
        if (!g_PCSC.pfnSCardListReaderGroups)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardListReaderGroups");
 
        if (*pcchGroups == SCARD_AUTOALLOCATE)
                pcchGroupsAlloc = TRUE;
@@ -709,7 +715,7 @@ static LONG WINAPI PCSC_SCardListReaderGroupsA(SCARDCONTEXT hContext, LPSTR mszG
        LONG status = SCARD_S_SUCCESS;
 
        if (!g_PCSC.pfnSCardListReaderGroups)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardListReaderGroups");
 
        if (!PCSC_LockCardContext(hContext))
                return SCARD_E_INVALID_HANDLE;
@@ -730,7 +736,7 @@ static LONG WINAPI PCSC_SCardListReaderGroupsW(SCARDCONTEXT hContext, LPWSTR msz
        LONG status = SCARD_S_SUCCESS;
 
        if (!g_PCSC.pfnSCardListReaderGroups)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardListReaderGroups");
 
        if (!PCSC_LockCardContext(hContext))
                return SCARD_E_INVALID_HANDLE;
@@ -763,7 +769,7 @@ static LONG WINAPI PCSC_SCardListReaders_Internal(SCARDCONTEXT hContext, LPCSTR
                return SCARD_E_INVALID_PARAMETER;
 
        if (!g_PCSC.pfnSCardListReaders)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardListReaders");
 
        mszGroups = NULL; /* mszGroups is not supported by pcsc-lite */
 
@@ -811,7 +817,7 @@ static LONG WINAPI PCSC_SCardListReadersA(SCARDCONTEXT hContext, LPCSTR mszGroup
        LONG status = SCARD_S_SUCCESS;
 
        if (!g_PCSC.pfnSCardListReaders)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardListReaders");
 
        if (!hContext)
        {
@@ -849,7 +855,7 @@ static LONG WINAPI PCSC_SCardListReadersW(SCARDCONTEXT hContext, LPCWSTR mszGrou
        BOOL nullCardContext = FALSE;
 
        if (!g_PCSC.pfnSCardListReaders)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardListReaders");
 
        if (!hContext)
        {
@@ -1173,7 +1179,7 @@ static LONG WINAPI PCSC_SCardGetStatusChange_Internal(SCARDCONTEXT hContext, DWO
        PCSC_DWORD pcsc_cReaders = (PCSC_DWORD)cReaders;
 
        if (!g_PCSC.pfnSCardGetStatusChange)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardGetStatusChange");
 
        if (!cReaders)
                return SCARD_S_SUCCESS;
@@ -1281,7 +1287,7 @@ static LONG WINAPI PCSC_SCardGetStatusChangeW(SCARDCONTEXT hContext, DWORD dwTim
        LONG status = SCARD_S_SUCCESS;
 
        if (!g_PCSC.pfnSCardGetStatusChange)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardGetStatusChange");
 
        if (!PCSC_LockCardContext(hContext))
                return SCARD_E_INVALID_HANDLE;
@@ -1331,7 +1337,7 @@ static LONG WINAPI PCSC_SCardCancel(SCARDCONTEXT hContext)
        LONG status = SCARD_S_SUCCESS;
 
        if (!g_PCSC.pfnSCardCancel)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardCancel");
 
        status = (LONG)g_PCSC.pfnSCardCancel(hContext);
        status = PCSC_MapErrorCodeToWinSCard(status);
@@ -1351,7 +1357,7 @@ static LONG WINAPI PCSC_SCardConnect_Internal(SCARDCONTEXT hContext, LPCSTR szRe
        PCSC_DWORD pcsc_dwActiveProtocol = 0;
 
        if (!g_PCSC.pfnSCardConnect)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardConnect");
 
        shared = (dwShareMode == SCARD_SHARE_DIRECT) ? TRUE : FALSE;
        PCSC_WaitForCardAccess(hContext, 0, shared);
@@ -1439,7 +1445,7 @@ static LONG WINAPI PCSC_SCardReconnect(SCARDHANDLE hCard, DWORD dwShareMode,
        PCSC_DWORD pcsc_dwActiveProtocol = 0;
 
        if (!g_PCSC.pfnSCardReconnect)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardReconnect");
 
        shared = (dwShareMode == SCARD_SHARE_DIRECT) ? TRUE : FALSE;
        PCSC_WaitForCardAccess(0, hCard, shared);
@@ -1457,7 +1463,7 @@ static LONG WINAPI PCSC_SCardDisconnect(SCARDHANDLE hCard, DWORD dwDisposition)
        PCSC_DWORD pcsc_dwDisposition = (PCSC_DWORD)dwDisposition;
 
        if (!g_PCSC.pfnSCardDisconnect)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardDisconnect");
 
        status = (LONG)g_PCSC.pfnSCardDisconnect(hCard, pcsc_dwDisposition);
        status = PCSC_MapErrorCodeToWinSCard(status);
@@ -1478,7 +1484,7 @@ static LONG WINAPI PCSC_SCardBeginTransaction(SCARDHANDLE hCard)
        PCSC_SCARDCONTEXT* pContext = NULL;
 
        if (!g_PCSC.pfnSCardBeginTransaction)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardBeginTransaction");
 
        pCard = PCSC_GetCardHandleData(hCard);
 
@@ -1507,7 +1513,7 @@ static LONG WINAPI PCSC_SCardEndTransaction(SCARDHANDLE hCard, DWORD dwDispositi
        PCSC_DWORD pcsc_dwDisposition = (PCSC_DWORD)dwDisposition;
 
        if (!g_PCSC.pfnSCardEndTransaction)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardEndTransaction");
 
        pCard = PCSC_GetCardHandleData(hCard);
 
@@ -1548,7 +1554,7 @@ static LONG WINAPI PCSC_SCardState(SCARDHANDLE hCard, LPDWORD pdwState, LPDWORD
        PCSC_DWORD pcsc_cbAtrLen = (PCSC_DWORD)*pcbAtrLen;
 
        if (!g_PCSC.pfnSCardStatus)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardStatus");
 
        pCard = PCSC_GetCardHandleData(hCard);
 
@@ -1599,7 +1605,7 @@ static LONG WINAPI PCSC_SCardStatus_Internal(SCARDHANDLE hCard, LPSTR mszReaderN
        LPBYTE tATR = NULL;
 
        if (!g_PCSC.pfnSCardStatus)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardStatus");
 
        pCard = PCSC_GetCardHandleData(hCard);
 
@@ -1787,7 +1793,7 @@ static LONG WINAPI PCSC_SCardTransmit(SCARDHANDLE hCard, LPCSCARD_IO_REQUEST pio
        PCSC_DWORD pcsc_cbRecvLength = 0;
 
        if (!g_PCSC.pfnSCardTransmit)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardTransmit");
 
        pCard = PCSC_GetCardHandleData(hCard);
 
@@ -1916,7 +1922,7 @@ static LONG WINAPI PCSC_SCardControl(SCARDHANDLE hCard, DWORD dwControlCode, LPC
        PCSC_DWORD pcsc_BytesReturned = 0;
 
        if (!g_PCSC.pfnSCardControl)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardControl");
 
        pCard = PCSC_GetCardHandleData(hCard);
 
@@ -1985,7 +1991,7 @@ static LONG WINAPI PCSC_SCardGetAttrib_Internal(SCARDHANDLE hCard, DWORD dwAttrI
        PCSC_DWORD pcsc_cbAttrLen = 0;
 
        if (!g_PCSC.pfnSCardGetAttrib)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardGetAttrib");
 
        pCard = PCSC_GetCardHandleData(hCard);
 
@@ -2330,7 +2336,7 @@ static LONG WINAPI PCSC_SCardSetAttrib(SCARDHANDLE hCard, DWORD dwAttrId, LPCBYT
        PCSC_DWORD pcsc_cbAttrLen = (PCSC_DWORD)cbAttrLen;
 
        if (!g_PCSC.pfnSCardSetAttrib)
-               return SCARD_E_NO_SERVICE;
+               return PCSC_SCard_LogError("g_PCSC.pfnSCardSetAttrib");
 
        pCard = PCSC_GetCardHandleData(hCard);