[Libreoffice-commits] online.git: net/Ssl.cpp

Pranav Kant pranavk at collabora.co.uk
Mon Nov 27 11:01:55 UTC 2017


 net/Ssl.cpp |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 483a5d68e5f7e6b145d957c50120cd05c436387b
Author: Pranav Kant <pranavk at collabora.co.uk>
Date:   Mon Nov 27 16:29:26 2017 +0530

    tdf#114073: Potential fix for crash with openssl1.1
    
    Using null ssl context to set options is surely not a good idea:
    
    unsigned long SSL_CTX_set_options(SSL_CTX *ctx, unsigned long op)
    {
        return ctx->options |= op;
    }
    
    Change-Id: I2700350e0c3928e372488c81b8111c9ab0b48e06

diff --git a/net/Ssl.cpp b/net/Ssl.cpp
index dd0094b8..a16caa25 100644
--- a/net/Ssl.cpp
+++ b/net/Ssl.cpp
@@ -50,8 +50,7 @@ SslContext::SslContext(const std::string& certFilePath,
 #endif
 
 #if OPENSSL_VERSION_NUMBER >= 0x10100003L
-    SSL_CTX_set_options(nullptr, 0);
-    OPENSSL_init_ssl(OPENSSL_INIT_LOAD_CONFIG, NULL);
+    OPENSSL_init_ssl(OPENSSL_INIT_LOAD_CONFIG, nullptr);
 #else
     SSL_library_init();
     SSL_load_error_strings();


More information about the Libreoffice-commits mailing list