net/Ssl.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) New commits: commit bd817b2e60dfd049d25a539d25861ed5b0cf5498 Author: Pranav Kant <pran...@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 (cherry picked from commit 483a5d68e5f7e6b145d957c50120cd05c436387b) Reviewed-on: https://gerrit.libreoffice.org/46704 Reviewed-by: Jan Holesovsky <ke...@collabora.com> Tested-by: Jan Holesovsky <ke...@collabora.com> 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(); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits