[Libreoffice-commits] online.git: Branch 'libreoffice-6-0' - net/Ssl.cpp

[Pranav Kant]

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

New commits:
commit 139a7565bf2c4305c4f915cbff80373a8eff5d40
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
    Reviewed-on: https://gerrit.libreoffice.org/45375
    Reviewed-by: pranavk <pranavk at collabora.co.uk>
    Tested-by: Rene Engelhard <rene at debian.org>

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();