]> granicus.if.org Git - p11-kit/commitdiff
test: Remove /proxy/deinit-after-fork test
authorDaiki Ueno <dueno@redhat.com>
Wed, 21 Sep 2016 12:50:38 +0000 (14:50 +0200)
committerDaiki Ueno <ueno@gnu.org>
Mon, 31 Oct 2016 14:24:17 +0000 (15:24 +0100)
This test hasn't been working since the removal of the pthread_atfork()
deinit code.  To properly clean up, the child process needs to call
C_Initialize() and C_Finalize(), and it is already tested by
/proxy/initialize-child.

p11-kit/test-proxy.c

index 0fb270b73a30db3debcdb73cc0fdcbb3e60ce2f4..9d894bf61f35f0a13afe502f00bc090016bfed2a 100644 (file)
@@ -114,42 +114,6 @@ test_initialize_multiple (void)
 }
 
 #ifndef _WIN32
-static void
-test_deinit_after_fork (void)
-{
-       CK_FUNCTION_LIST_PTR proxy;
-       CK_RV rv;
-       pid_t pid;
-       int st;
-
-       rv = C_GetFunctionList (&proxy);
-       assert (rv == CKR_OK);
-
-       assert (p11_proxy_module_check (proxy));
-
-       rv = proxy->C_Initialize(NULL);
-       assert_num_eq (rv, CKR_OK);
-
-       pid = fork ();
-       if (!pid) {
-               exit(0);
-       }
-       assert (pid != -1);
-       waitpid(pid, &st, 0);
-
-       rv = proxy->C_Finalize (NULL);
-       assert_num_eq (rv, CKR_OK);
-
-       p11_proxy_module_cleanup ();
-
-       /* If the assertion fails, p11_kit_failed() doesn't return. So make
-        * sure we do all the cleanup before the (expected) failure, or it
-        * causes all the *later* tests to fail too! */
-       if (!WIFEXITED (st) || WEXITSTATUS(st) != 0)
-               assert_fail("Child failed to C_Initialize() and C_Finalize()", NULL);
-
-}
-
 static void
 test_initialize_child (void)
 {
@@ -286,7 +250,6 @@ main (int argc,
        p11_test (test_initialize_finalize, "/proxy/initialize-finalize");
        p11_test (test_initialize_multiple, "/proxy/initialize-multiple");
 #ifndef _WIN32
-       p11_test (test_deinit_after_fork, "/proxy/deinit-after-fork");
        p11_test (test_initialize_child, "/proxy/initialize-child");
 #endif