[openssl-commits] [openssl] OpenSSL_1_1_1-stable update

[Richard Levitte]

The branch OpenSSL_1_1_1-stable has been updated
       via  eaacc2475044b624a7b3b1ecd9ebe5953a6ff45d (commit)
      from  e1cce612a6520555805c25be2539f231c22696d9 (commit)


- Log -----------------------------------------------------------------
commit eaacc2475044b624a7b3b1ecd9ebe5953a6ff45d
Author: Richard Levitte <levitte at openssl.org>
Date:   Wed Feb 6 20:51:47 2019 +0100

    test/drbgtest.c: call OPENSSL_thread_stop() explicitly
    
    The manual says this in its notes:
    
        ... and therefore applications using static linking should also call
        OPENSSL_thread_stop() on each thread. ...
    
    Fixes #8171
    
    Reviewed-by: Matthias St. Pierre <Matthias.St.Pierre at ncp-e.com>
    (Merged from https://github.com/openssl/openssl/pull/8173)
    
    (cherry picked from commit 03cdfe1efaf2a3b5192b8cb3ef331939af7bfeb8)

-----------------------------------------------------------------------

Summary of changes:
 test/drbgtest.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/test/drbgtest.c b/test/drbgtest.c
index b690475..67fc1aa 100644
--- a/test/drbgtest.c
+++ b/test/drbgtest.c
@@ -830,6 +830,11 @@ typedef HANDLE thread_t;
 static DWORD WINAPI thread_run(LPVOID arg)
 {
     run_multi_thread_test();
+    /*
+     * Because we're linking with a static library, we must stop each
+     * thread explicitly, or so says OPENSSL_thread_stop(3)
+     */
+    OPENSSL_thread_stop();
     return 0;
 }
 
@@ -851,6 +856,11 @@ typedef pthread_t thread_t;
 static void *thread_run(void *arg)
 {
     run_multi_thread_test();
+    /*
+     * Because we're linking with a static library, we must stop each
+     * thread explicitly, or so says OPENSSL_thread_stop(3)
+     */
+    OPENSSL_thread_stop();
     return NULL;
 }