On Friday 08 December 2006 20:20, Alon Bar-Lev wrote:
> Hi!
>
> I've modified pkcs11-helper to use the new free pkcs11.h header.
> I was surprised that it went relatively OK, good work! :)
>
> Some modifications:

One more:
6. Cast CKR_VENDOR_DEFINED to ck_rv_t so compiler will not
complain about signess.

I think the same should be done for:
CKO_VENDOR_DEFINED
CKH_VENDOR_DEFINED
CKK_VENDOR_DEFINED
etc...

Best Regards,
Alon Bar-Lev.

--- include/pkcs11-helper-1.0/pkcs11.h  (revision 28)
+++ include/pkcs11-helper-1.0/pkcs11.h  (working copy)
@@ -1061,7 +1061,7 @@
 #define CKR_MUTEX_BAD                          (0x1a0)
 #define CKR_MUTEX_NOT_LOCKED                   (0x1a1)
 #define CKR_FUNCTION_REJECTED                   (0x200)
-#define CKR_VENDOR_DEFINED                     (1 << 31)
+#define CKR_VENDOR_DEFINED                     (ck_rv_t)(1 << 31)
_______________________________________________
opensc-devel mailing list
opensc-devel@lists.opensc-project.org
http://www.opensc-project.org/mailman/listinfo/opensc-devel

Reply via email to