From e1e740e35edeccfc04d2808fc215c6a12698305f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?louiz=E2=80=99?= <louiz@louiz.org>
Date: Thu, 8 Dec 2016 21:25:20 +0100
Subject: =?UTF-8?q?Don=E2=80=99t=20use=20global=20static=20members=20but?=
 =?UTF-8?q?=20functions=20that=20return=20a=20reference=20to=20an=20intern?=
 =?UTF-8?q?al=20static=20object?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

See https://github.com/randombit/botan/issues/761
---
 louloulibs/network/tcp_socket_handler.cpp | 26 ++++++++++++++++++++------
 louloulibs/network/tcp_socket_handler.hpp |  7 -------
 2 files changed, 20 insertions(+), 13 deletions(-)

(limited to 'louloulibs')

diff --git a/louloulibs/network/tcp_socket_handler.cpp b/louloulibs/network/tcp_socket_handler.cpp
index 9d9a6aa..6aef2b1 100644
--- a/louloulibs/network/tcp_socket_handler.cpp
+++ b/louloulibs/network/tcp_socket_handler.cpp
@@ -15,10 +15,24 @@
 # include <botan/hex.h>
 # include <botan/tls_exceptn.h>
 
-Botan::AutoSeeded_RNG TCPSocketHandler::rng;
-BiboumiTLSPolicy TCPSocketHandler::policy;
-Botan::TLS::Session_Manager_In_Memory TCPSocketHandler::session_manager(TCPSocketHandler::rng);
-
+namespace
+{
+    Botan::AutoSeeded_RNG& get_rng()
+    {
+      static Botan::AutoSeeded_RNG rng{};
+      return rng;
+    }
+    BiboumiTLSPolicy& get_policy()
+    {
+      static BiboumiTLSPolicy policy{};
+      return policy;
+    }
+    Botan::TLS::Session_Manager_In_Memory& get_session_manager()
+    {
+      static Botan::TLS::Session_Manager_In_Memory session_manager{get_rng()};
+      return session_manager;
+    }
+}
 #endif
 
 #ifndef UIO_FASTIOV
@@ -229,8 +243,8 @@ void TCPSocketHandler::start_tls(const std::string& address, const std::string&
       [this](Botan::TLS::Alert alert, const Botan::byte*, size_t) { this->tls_alert(alert); },
       [this](const Botan::TLS::Session& session) { return this->tls_session_established(session); },
 # endif
-      session_manager, this->credential_manager, policy,
-      rng, server_info, Botan::TLS::Protocol_Version::latest_tls_version());
+      get_session_manager(), this->credential_manager, get_policy(),
+      get_rng(), server_info, Botan::TLS::Protocol_Version::latest_tls_version());
 }
 
 void TCPSocketHandler::tls_recv()
diff --git a/louloulibs/network/tcp_socket_handler.hpp b/louloulibs/network/tcp_socket_handler.hpp
index 3c6ff71..7fc40c2 100644
--- a/louloulibs/network/tcp_socket_handler.hpp
+++ b/louloulibs/network/tcp_socket_handler.hpp
@@ -222,14 +222,7 @@ protected:
   virtual void on_connection_close(const std::string&) {}
   virtual void on_connection_failed(const std::string&) {}
 
-private:
 #ifdef BOTAN_FOUND
-  /**
-   * Botan stuff to manipulate a TLS session.
-   */
-  static Botan::AutoSeeded_RNG rng;
-  static BiboumiTLSPolicy policy;
-  static Botan::TLS::Session_Manager_In_Memory session_manager;
 protected:
   BasicCredentialsManager credential_manager;
 private:
-- 
cgit v1.2.3