Changeset 269 for trunk/src/common/pkcs11_lib.c

Timestamp:

05/15/07 07:18:24 (5 years ago)

Author:

ludovic.rousseau

Message:

C_Initalize should be called after each fork

The pkcs11_eventmgr first C_Initialize then becomes a daemon... The
PKCS#11 standard states that after fork the user must call C_Initialize
again, in order to allow proper library initialization at new process.

Also, don't quit on major errors... try to reinitialize.

Closes ticket #14. Thanks to Alon Bar-Lev for the patch

File:

• trunk/src/common/pkcs11_lib.c (modified) (3 diffs)

trunk/src/common/pkcs11_lib.c

@@ -618 +618 @@
   void *module_handle;
   CK_FUNCTION_LIST_PTR fl;
+  int should_finalize;
   slot_t *slots;
   CK_ULONG slot_count;
@@ -715 +716 @@
   if (flag) rv = h->fl->C_Initialize((CK_VOID_PTR) &initArgs);
   else      rv = h->fl->C_Initialize(NULL);
-  if (rv != CKR_OK) {
+  if (rv == CKR_OK)
+    h->should_finalize = 1;
+  else if (rv != CKR_CRYPTOKI_ALREADY_INITIALIZED) {
     set_error("C_Initialize() failed: %x", rv);
     return -1;
@@ -804 +807 @@
   /* finalise pkcs #11 module */
   if (h->fl != NULL)
-    h->fl->C_Finalize(NULL);
+    if (h->should_finalize)
+      h->fl->C_Finalize(NULL);
   /* unload the module */
   if (h->module_handle != NULL)