test/shlibloadtest: small fixes
authorRichard Levitte <levitte@openssl.org>
Thu, 3 Nov 2016 15:21:36 +0000 (16:21 +0100)
committerRichard Levitte <levitte@openssl.org>
Thu, 3 Nov 2016 15:21:36 +0000 (16:21 +0100)
- Make sure to initialise SHLIB variables
- Make sure to make local variables static

Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/1838)

test/shlibloadtest.c

index be1e78449e3d55231ffd7c0ba81ca51dddc839ba..20030d94a8fafdb96d87e62cb8ba9ce87b7c089b 100644 (file)
@@ -28,12 +28,12 @@ typedef void (*SSL_CTX_free_t)(SSL_CTX *);
 typedef unsigned long (*ERR_get_error_t)(void);
 typedef unsigned long (*OpenSSL_version_num_t)(void);
 
 typedef unsigned long (*ERR_get_error_t)(void);
 typedef unsigned long (*OpenSSL_version_num_t)(void);
 
-TLS_method_t TLS_method;
-SSL_CTX_new_t SSL_CTX_new;
-SSL_CTX_free_t SSL_CTX_free;
+static TLS_method_t TLS_method;
+static SSL_CTX_new_t SSL_CTX_new;
+static SSL_CTX_free_t SSL_CTX_free;
 
 
-ERR_get_error_t ERR_get_error;
-OpenSSL_version_num_t OpenSSL_version_num;
+static ERR_get_error_t ERR_get_error;
+static OpenSSL_version_num_t OpenSSL_version_num;
 
 
 #ifdef DSO_DLFCN
 
 
 #ifdef DSO_DLFCN
@@ -42,6 +42,7 @@ OpenSSL_version_num_t OpenSSL_version_num;
 
 typedef void * SHLIB;
 typedef void * SHLIB_SYM;
 
 typedef void * SHLIB;
 typedef void * SHLIB_SYM;
+# define SHLIB_INIT NULL
 
 # define SHARED_LIBRARY_SUFFIX ".so"
 
 
 # define SHARED_LIBRARY_SUFFIX ".so"
 
@@ -87,6 +88,7 @@ static int shlib_close(SHLIB lib)
 
 typedef HINSTANCE SHLIB;
 typedef void * SHLIB_SYM;
 
 typedef HINSTANCE SHLIB;
 typedef void * SHLIB_SYM;
+# define SHLIB_INIT 0
 
 static int shlib_load(char *filename, SHLIB *lib)
 {
 
 static int shlib_load(char *filename, SHLIB *lib)
 {
@@ -129,7 +131,7 @@ enum test_types_en {
 
 int main(int argc, char **argv)
 {
 
 int main(int argc, char **argv)
 {
-    SHLIB ssllib, cryptolib;
+    SHLIB ssllib = SHLIB_INIT, cryptolib = SHLIB_INIT;
     SSL_CTX *ctx;
     union {
         void (*func) (void);
     SSL_CTX *ctx;
     union {
         void (*func) (void);