[openssl-commits] [openssl] OpenSSL_1_1_0-stable update

Rich Salz rsalz at openssl.org
Wed Oct 19 11:03:11 UTC 2016

The branch OpenSSL_1_1_0-stable has been updated
       via  3d040392ff68e71171cbd5750f1d76efdb0516ca (commit)
      from  c347c5a3b6d51cbd24cfab45ab2698c7757a3ffc (commit)

- Log -----------------------------------------------------------------
commit 3d040392ff68e71171cbd5750f1d76efdb0516ca
Author: Mat <mberchtold at gmail.com>
Date:   Mon Oct 3 05:40:32 2016 +0200

    Do not set load_crypto_strings_inited when OPENSSL_NO_ERR is defined
    Only set the load_crypto_strings_inited to 1 when err_load_crypto_strings_int was called.
    This solves the following issue:
    - openssl is built with no-err
    - load_crypto_strings_inited is set to 1 during the OPENSSL_init_crypto call
    - During the cleanup: OPENSSL_cleanup, err_free_strings_int is called because load_crypto_strings_inited == 1
    - err_free_strings_int calls do_err_strings_init because it has never been called
    - Now do_err_strings_init calls OPENSSL_init_crypto
    - But since we are in the cleanup (stopped == 1) this results in an error:
    - which then tries to initialize everything we are trying to clean up: ERR_get_state, ossl_init_thread_start, etc
    Reviewed-by: Tim Hudson <tjh at openssl.org>
    Reviewed-by: Rich Salz <rsalz at openssl.org>
    (Merged from https://github.com/openssl/openssl/pull/1654)
    (cherry picked from commit a1f2b0e6e07a53c0ae2c81cba319b90e54210cd6)


Summary of changes:
 crypto/init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/crypto/init.c b/crypto/init.c
index 9af857b..7423ecc 100644
--- a/crypto/init.c
+++ b/crypto/init.c
@@ -103,8 +103,8 @@ DEFINE_RUN_ONCE_STATIC(ossl_init_load_crypto_strings)
 # endif
     ret = err_load_crypto_strings_int();
     load_crypto_strings_inited = 1;
     return ret;

More information about the openssl-commits mailing list