test/shlibloadtest.c: make some variables block local
authorRichard Levitte <levitte@openssl.org>
Fri, 23 Mar 2018 13:18:16 +0000 (14:18 +0100)
committerRichard Levitte <levitte@openssl.org>
Fri, 23 Mar 2018 13:18:16 +0000 (14:18 +0100)
myDSO_dsobyaddr and myDSO_free are only used in a narrow block of
code, and can therefore be made local to that block.  Otherwise, some
compilers may warn that they are unused.

Reviewed-by: Andy Polyakov <appro@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5733)

test/shlibloadtest.c

index 07ed40dc6f9c6e041d5809ca21f7bd093180a7e7..a8cf8e7fde10e21a087c28a2e565526d04f7cdd3 100644 (file)
@@ -108,8 +108,6 @@ static int test_lib(void)
     SSL_CTX_free_t mySSL_CTX_free;
     ERR_get_error_t myERR_get_error;
     OpenSSL_version_num_t myOpenSSL_version_num;
-    DSO_dsobyaddr_t myDSO_dsobyaddr;
-    DSO_free_t myDSO_free;
     int result = 0;
 
     switch (test_type) {
@@ -171,6 +169,9 @@ static int test_lib(void)
 
     if (test_type == DSO_REFTEST) {
 # ifdef DSO_DLFCN
+        DSO_dsobyaddr_t myDSO_dsobyaddr;
+        DSO_free_t myDSO_free;
+
         /*
          * This is resembling the code used in ossl_init_base() and
          * OPENSSL_atexit() to block unloading the library after dlclose().