winscard: add pcsc-lite helpers for upcoming implementation
Mounir IDRASSI
mounir.idrassi at idrix.fr
Wed May 23 23:56:56 CDT 2007
Thanks for your comments Juan.
Actually, the upcoming implementation checks the used function pointer at
each call and returns an error if not assigned. Also in it, winscard
always succeed to load even if pcsc-lite is not running. FYI, my first
winscard patch contains an advanced stage of the whole implementation.
Concerning the BOOL return value, it's here for "historical" reasons. I'll
remove it.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://wwww.idrix.fr
> +BOOL InitializePCSCLite(void)
>
> You don't use the return value:
>
> @@ -42,9 +43,11 @@ BOOL WINAPI DllMain (HINSTANCE hinstDLL, DWORD
> fdwReason, LPVOID lpvReserved)
> {
> DisableThreadLibraryCalls(hinstDLL);
> WINSCARD_hModule = hinstDLL;
> + InitializePCSCLite();
> break;
> }
>
> I suspect it's safer not to fail loading winscard if PCSC-Lite isn't
> available. Instead, the functions such as SCardListCards should probably
> return empty lists. But knowing to do so relies on knowing that PCSC-Lite
> couldn't be initialized, so the return value shouldn't be thrown away.
> Or, remove the return value, and have every function check the function
> pointers it needs.
>
> --Juan
More information about the wine-devel
mailing list