Fix shlibloadtest to properly execute the dso_ref test
authorMatt Caswell <matt@openssl.org>
Thu, 15 Nov 2018 14:50:52 +0000 (14:50 +0000)
committerMatt Caswell <matt@openssl.org>
Fri, 4 Jan 2019 13:19:14 +0000 (13:19 +0000)
Reviewed-by: Tim Hudson <tjh@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/7647)

test/shlibloadtest.c

index 4c5d80106e12d653e41451318964bb71a3998064..dcb19752e8f4f9dd7ede2003c84070c80969a61b 100644 (file)
@@ -121,6 +121,7 @@ static int test_lib(void)
 
     switch (test_type) {
     case JUST_CRYPTO:
+    case DSO_REFTEST:
         if (!TEST_true(shlib_load(path_crypto, &cryptolib)))
             goto end;
         break;
@@ -134,10 +135,6 @@ static int test_lib(void)
                 || !TEST_true(shlib_load(path_crypto, &cryptolib)))
             goto end;
         break;
-    case DSO_REFTEST:
-        if (!TEST_true(shlib_load(path_crypto, &cryptolib)))
-            goto end;
-        break;
     }
 
     if (test_type != JUST_CRYPTO && test_type != DSO_REFTEST) {
@@ -210,6 +207,7 @@ static int test_lib(void)
 
     switch (test_type) {
     case JUST_CRYPTO:
+    case DSO_REFTEST:
         if (!TEST_true(shlib_close(cryptolib)))
             goto end;
         break;
@@ -223,10 +221,6 @@ static int test_lib(void)
                 || !TEST_true(shlib_close(cryptolib)))
             goto end;
         break;
-    case DSO_REFTEST:
-        if (!TEST_true(shlib_close(cryptolib)))
-            goto end;
-        break;
     }
 
     result = 1;
@@ -247,7 +241,7 @@ int setup_tests(void)
     } else if (strcmp(p, "-just_crypto") == 0) {
         test_type = JUST_CRYPTO;
     } else if (strcmp(p, "-dso_ref") == 0) {
-        test_type = JUST_CRYPTO;
+        test_type = DSO_REFTEST;
     } else {
         TEST_error("Unrecognised argument");
         return 0;