]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/extra/m_ssl_openssl.cpp
Change all socketengine methods to be static
[user/henk/code/inspircd.git] / src / modules / extra / m_ssl_openssl.cpp
index 7acbcfc187534e39290bf380b43a8f03c98eea36..53a01724be9b6924ce6d6488edce3e567a515ba8 100644 (file)
@@ -60,26 +60,180 @@ char* get_error()
        return ERR_error_string(ERR_get_error(), NULL);
 }
 
-static int error_callback(const char *str, size_t len, void *u);
+static int OnVerify(int preverify_ok, X509_STORE_CTX* ctx);
 
-/** Represents an SSL user's extra data
- */
-class issl_session
+namespace OpenSSL
 {
-public:
-       SSL* sess;
-       issl_status status;
-       reference<ssl_cert> cert;
+       class Exception : public ModuleException
+       {
+        public:
+               Exception(const std::string& reason)
+                       : ModuleException(reason) { }
+       };
 
-       bool outbound;
-       bool data_to_write;
+       class DHParams
+       {
+               DH* dh;
 
-       issl_session()
+        public:
+               DHParams(const std::string& filename)
+               {
+                       FILE* dhpfile = fopen(filename.c_str(), "r");
+                       if (dhpfile == NULL)
+                               throw Exception("Couldn't open DH file " + filename + ": " + strerror(errno));
+
+                       dh = PEM_read_DHparams(dhpfile, NULL, NULL, NULL);
+                       fclose(dhpfile);
+                       if (!dh)
+                               throw Exception("Couldn't read DH params from file " + filename);
+               }
+
+               ~DHParams()
+               {
+                       DH_free(dh);
+               }
+
+               DH* get()
+               {
+                       return dh;
+               }
+       };
+
+       class Context
        {
-               outbound = false;
-               data_to_write = false;
-       }
-};
+               SSL_CTX* const ctx;
+
+        public:
+               Context(SSL_CTX* context)
+                       : ctx(context)
+               {
+                       SSL_CTX_set_mode(ctx, SSL_MODE_ENABLE_PARTIAL_WRITE | SSL_MODE_ACCEPT_MOVING_WRITE_BUFFER);
+                       SSL_CTX_set_verify(ctx, SSL_VERIFY_PEER | SSL_VERIFY_CLIENT_ONCE, OnVerify);
+               }
+
+               ~Context()
+               {
+                       SSL_CTX_free(ctx);
+               }
+
+               bool SetDH(DHParams& dh)
+               {
+                       return (SSL_CTX_set_tmp_dh(ctx, dh.get()) >= 0);
+               }
+
+               bool SetCiphers(const std::string& ciphers)
+               {
+                       return SSL_CTX_set_cipher_list(ctx, ciphers.c_str());
+               }
+
+               bool SetCerts(const std::string& filename)
+               {
+                       return SSL_CTX_use_certificate_chain_file(ctx, filename.c_str());
+               }
+
+               bool SetPrivateKey(const std::string& filename)
+               {
+                       return SSL_CTX_use_PrivateKey_file(ctx, filename.c_str(), SSL_FILETYPE_PEM);
+               }
+
+               bool SetCA(const std::string& filename)
+               {
+                       return SSL_CTX_load_verify_locations(ctx, filename.c_str(), 0);
+               }
+
+               SSL* CreateSession()
+               {
+                       return SSL_new(ctx);
+               }
+       };
+
+       class Profile : public refcountbase
+       {
+               /** Name of this profile
+                */
+               const std::string name;
+
+               /** DH parameters in use
+                */
+               DHParams dh;
+
+               /** OpenSSL makes us have two contexts, one for servers and one for clients
+                */
+               Context ctx;
+               Context clictx;
+
+               /** Digest to use when generating fingerprints
+                */
+               const EVP_MD* digest;
+
+               /** Last error, set by error_callback()
+                */
+               std::string lasterr;
+
+               static int error_callback(const char* str, size_t len, void* u)
+               {
+                       Profile* profile = reinterpret_cast<Profile*>(u);
+                       profile->lasterr = std::string(str, len - 1);
+                       return 0;
+               }
+
+        public:
+               Profile(const std::string& profilename, ConfigTag* tag)
+                       : name(profilename)
+                       , dh(ServerInstance->Config->Paths.PrependConfig(tag->getString("dhfile", "dh.pem")))
+                       , ctx(SSL_CTX_new(SSLv23_server_method()))
+                       , clictx(SSL_CTX_new(SSLv23_client_method()))
+               {
+                       if ((!ctx.SetDH(dh)) || (!clictx.SetDH(dh)))
+                               throw Exception("Couldn't set DH parameters");
+
+                       std::string hash = tag->getString("hash", "md5");
+                       digest = EVP_get_digestbyname(hash.c_str());
+                       if (digest == NULL)
+                               throw Exception("Unknown hash type " + hash);
+
+                       std::string ciphers = tag->getString("ciphers");
+                       if (!ciphers.empty())
+                       {
+                               if ((!ctx.SetCiphers(ciphers)) || (!clictx.SetCiphers(ciphers)))
+                               {
+                                       ERR_print_errors_cb(error_callback, this);
+                                       throw Exception("Can't set cipher list to \"" + ciphers + "\" " + lasterr);
+                               }
+                       }
+
+                       /* Load our keys and certificates
+                        * NOTE: OpenSSL's error logging API sucks, don't blame us for this clusterfuck.
+                        */
+                       std::string filename = ServerInstance->Config->Paths.PrependConfig(tag->getString("certfile", "cert.pem"));
+                       if ((!ctx.SetCerts(filename)) || (!clictx.SetCerts(filename)))
+                       {
+                               ERR_print_errors_cb(error_callback, this);
+                               throw Exception("Can't read certificate file: " + lasterr);
+                       }
+
+                       filename = ServerInstance->Config->Paths.PrependConfig(tag->getString("keyfile", "key.pem"));
+                       if ((!ctx.SetPrivateKey(filename)) || (!clictx.SetPrivateKey(filename)))
+                       {
+                               ERR_print_errors_cb(error_callback, this);
+                               throw Exception("Can't read key file: " + lasterr);
+                       }
+
+                       // Load the CAs we trust
+                       filename = ServerInstance->Config->Paths.PrependConfig(tag->getString("cafile", "ca.pem"));
+                       if ((!ctx.SetCA(filename)) || (!clictx.SetCA(filename)))
+                       {
+                               ERR_print_errors_cb(error_callback, this);
+                               ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Can't read CA list from %s. This is only a problem if you want to verify client certificates, otherwise it's safe to ignore this message. Error: %s", filename.c_str(), lasterr.c_str());
+                       }
+               }
+
+               const std::string& GetName() const { return name; }
+               SSL* CreateServerSession() { return ctx.CreateSession(); }
+               SSL* CreateClientSession() { return clictx.CreateSession(); }
+               const EVP_MD* GetDigest() { return digest; }
+       };
+}
 
 static int OnVerify(int preverify_ok, X509_STORE_CTX *ctx)
 {
@@ -98,34 +252,40 @@ static int OnVerify(int preverify_ok, X509_STORE_CTX *ctx)
 class OpenSSLIOHook : public SSLIOHook
 {
  private:
-       bool Handshake(StreamSocket* user, issl_session* session)
+       SSL* sess;
+       issl_status status;
+       const bool outbound;
+       bool data_to_write;
+       reference<OpenSSL::Profile> profile;
+
+       bool Handshake(StreamSocket* user)
        {
                int ret;
 
-               if (session->outbound)
-                       ret = SSL_connect(session->sess);
+               if (outbound)
+                       ret = SSL_connect(sess);
                else
-                       ret = SSL_accept(session->sess);
+                       ret = SSL_accept(sess);
 
                if (ret < 0)
                {
-                       int err = SSL_get_error(session->sess, ret);
+                       int err = SSL_get_error(sess, ret);
 
                        if (err == SSL_ERROR_WANT_READ)
                        {
-                               ServerInstance->SE->ChangeEventMask(user, FD_WANT_POLL_READ | FD_WANT_NO_WRITE);
-                               session->status = ISSL_HANDSHAKING;
+                               SocketEngine::ChangeEventMask(user, FD_WANT_POLL_READ | FD_WANT_NO_WRITE);
+                               this->status = ISSL_HANDSHAKING;
                                return true;
                        }
                        else if (err == SSL_ERROR_WANT_WRITE)
                        {
-                               ServerInstance->SE->ChangeEventMask(user, FD_WANT_NO_READ | FD_WANT_SINGLE_WRITE);
-                               session->status = ISSL_HANDSHAKING;
+                               SocketEngine::ChangeEventMask(user, FD_WANT_NO_READ | FD_WANT_SINGLE_WRITE);
+                               this->status = ISSL_HANDSHAKING;
                                return true;
                        }
                        else
                        {
-                               CloseSession(session);
+                               CloseSession();
                        }
 
                        return false;
@@ -133,48 +293,45 @@ class OpenSSLIOHook : public SSLIOHook
                else if (ret > 0)
                {
                        // Handshake complete.
-                       VerifyCertificate(session, user);
+                       VerifyCertificate();
 
-                       session->status = ISSL_OPEN;
+                       status = ISSL_OPEN;
 
-                       ServerInstance->SE->ChangeEventMask(user, FD_WANT_POLL_READ | FD_WANT_NO_WRITE | FD_ADD_TRIAL_WRITE);
+                       SocketEngine::ChangeEventMask(user, FD_WANT_POLL_READ | FD_WANT_NO_WRITE | FD_ADD_TRIAL_WRITE);
 
                        return true;
                }
                else if (ret == 0)
                {
-                       CloseSession(session);
+                       CloseSession();
                        return true;
                }
 
                return true;
        }
 
-       void CloseSession(issl_session* session)
+       void CloseSession()
        {
-               if (session->sess)
+               if (sess)
                {
-                       SSL_shutdown(session->sess);
-                       SSL_free(session->sess);
+                       SSL_shutdown(sess);
+                       SSL_free(sess);
                }
-
-               session->sess = NULL;
-               session->status = ISSL_NONE;
+               sess = NULL;
+               certificate = NULL;
+               status = ISSL_NONE;
                errno = EIO;
        }
 
-       void VerifyCertificate(issl_session* session, StreamSocket* user)
+       void VerifyCertificate()
        {
-               if (!session->sess || !user)
-                       return;
-
                X509* cert;
                ssl_cert* certinfo = new ssl_cert;
-               session->cert = certinfo;
+               this->certificate = certinfo;
                unsigned int n;
                unsigned char md[EVP_MAX_MD_SIZE];
 
-               cert = SSL_get_peer_certificate((SSL*)session->sess);
+               cert = SSL_get_peer_certificate(sess);
 
                if (!cert)
                {
@@ -182,7 +339,7 @@ class OpenSSLIOHook : public SSLIOHook
                        return;
                }
 
-               certinfo->invalid = (SSL_get_verify_result(session->sess) != X509_V_OK);
+               certinfo->invalid = (SSL_get_verify_result(sess) != X509_V_OK);
 
                if (!SelfSigned)
                {
@@ -198,7 +355,7 @@ class OpenSSLIOHook : public SSLIOHook
                certinfo->dn = X509_NAME_oneline(X509_get_subject_name(cert),0,0);
                certinfo->issuer = X509_NAME_oneline(X509_get_issuer_name(cert),0,0);
 
-               if (!X509_digest(cert, digest, md, &n))
+               if (!X509_digest(cert, profile->GetDigest(), md, &n))
                {
                        certinfo->error = "Out of memory generating fingerprint";
                }
@@ -216,146 +373,87 @@ class OpenSSLIOHook : public SSLIOHook
        }
 
  public:
-       issl_session* sessions;
-       SSL_CTX* ctx;
-       SSL_CTX* clictx;
-       const EVP_MD *digest;
-
-       OpenSSLIOHook(Module* mod)
-               : SSLIOHook(mod, "ssl/openssl")
-       {
-               sessions = new issl_session[ServerInstance->SE->GetMaxFds()];
-       }
-
-       ~OpenSSLIOHook()
-       {
-               delete[] sessions;
-       }
-
-       void OnStreamSocketAccept(StreamSocket* user, irc::sockets::sockaddrs* client, irc::sockets::sockaddrs* server) CXX11_OVERRIDE
+       OpenSSLIOHook(IOHookProvider* hookprov, StreamSocket* sock, bool is_outbound, SSL* session, const reference<OpenSSL::Profile>& sslprofile)
+               : SSLIOHook(hookprov)
+               , sess(session)
+               , status(ISSL_NONE)
+               , outbound(is_outbound)
+               , data_to_write(false)
+               , profile(sslprofile)
        {
-               int fd = user->GetFd();
-
-               issl_session* session = &sessions[fd];
-
-               session->sess = SSL_new(ctx);
-               session->status = ISSL_NONE;
-               session->outbound = false;
-               session->cert = NULL;
-
-               if (session->sess == NULL)
-                       return;
-
-               if (SSL_set_fd(session->sess, fd) == 0)
-               {
-                       ServerInstance->Logs->Log(MODNAME, LOG_DEBUG, "BUG: Can't set fd with SSL_set_fd: %d", fd);
+               if (sess == NULL)
                        return;
-               }
+               if (SSL_set_fd(sess, sock->GetFd()) == 0)
+                       throw ModuleException("Can't set fd with SSL_set_fd: " + ConvToStr(sock->GetFd()));
 
-               Handshake(user, session);
-       }
-
-       void OnStreamSocketConnect(StreamSocket* user) CXX11_OVERRIDE
-       {
-               int fd = user->GetFd();
-               /* Are there any possibilities of an out of range fd? Hope not, but lets be paranoid */
-               if ((fd < 0) || (fd > ServerInstance->SE->GetMaxFds() -1))
-                       return;
-
-               issl_session* session = &sessions[fd];
-
-               session->sess = SSL_new(clictx);
-               session->status = ISSL_NONE;
-               session->outbound = true;
-
-               if (session->sess == NULL)
-                       return;
-
-               if (SSL_set_fd(session->sess, fd) == 0)
-               {
-                       ServerInstance->Logs->Log(MODNAME, LOG_DEBUG, "BUG: Can't set fd with SSL_set_fd: %d", fd);
-                       return;
-               }
-
-               Handshake(user, session);
+               sock->AddIOHook(this);
+               Handshake(sock);
        }
 
        void OnStreamSocketClose(StreamSocket* user) CXX11_OVERRIDE
        {
-               int fd = user->GetFd();
-               /* Are there any possibilities of an out of range fd? Hope not, but lets be paranoid */
-               if ((fd < 0) || (fd > ServerInstance->SE->GetMaxFds() - 1))
-                       return;
-
-               CloseSession(&sessions[fd]);
+               CloseSession();
        }
 
        int OnStreamSocketRead(StreamSocket* user, std::string& recvq) CXX11_OVERRIDE
        {
-               int fd = user->GetFd();
-               /* Are there any possibilities of an out of range fd? Hope not, but lets be paranoid */
-               if ((fd < 0) || (fd > ServerInstance->SE->GetMaxFds() - 1))
-                       return -1;
-
-               issl_session* session = &sessions[fd];
-
-               if (!session->sess)
+               if (!sess)
                {
-                       CloseSession(session);
+                       CloseSession();
                        return -1;
                }
 
-               if (session->status == ISSL_HANDSHAKING)
+               if (status == ISSL_HANDSHAKING)
                {
                        // The handshake isn't finished and it wants to read, try to finish it.
-                       if (!Handshake(user, session))
+                       if (!Handshake(user))
                        {
                                // Couldn't resume handshake.
-                               if (session->status == ISSL_NONE)
+                               if (status == ISSL_NONE)
                                        return -1;
                                return 0;
                        }
                }
 
-               // If we resumed the handshake then session->status will be ISSL_OPEN
+               // If we resumed the handshake then this->status will be ISSL_OPEN
 
-               if (session->status == ISSL_OPEN)
+               if (status == ISSL_OPEN)
                {
                        char* buffer = ServerInstance->GetReadBuffer();
                        size_t bufsiz = ServerInstance->Config->NetBufferSize;
-                       int ret = SSL_read(session->sess, buffer, bufsiz);
+                       int ret = SSL_read(sess, buffer, bufsiz);
 
                        if (ret > 0)
                        {
                                recvq.append(buffer, ret);
-                               if (session->data_to_write)
-                                       ServerInstance->SE->ChangeEventMask(user, FD_WANT_POLL_READ | FD_WANT_SINGLE_WRITE);
+                               if (data_to_write)
+                                       SocketEngine::ChangeEventMask(user, FD_WANT_POLL_READ | FD_WANT_SINGLE_WRITE);
                                return 1;
                        }
                        else if (ret == 0)
                        {
                                // Client closed connection.
-                               CloseSession(session);
+                               CloseSession();
                                user->SetError("Connection closed");
                                return -1;
                        }
                        else if (ret < 0)
                        {
-                               int err = SSL_get_error(session->sess, ret);
+                               int err = SSL_get_error(sess, ret);
 
                                if (err == SSL_ERROR_WANT_READ)
                                {
-                                       ServerInstance->SE->ChangeEventMask(user, FD_WANT_POLL_READ);
+                                       SocketEngine::ChangeEventMask(user, FD_WANT_POLL_READ);
                                        return 0;
                                }
                                else if (err == SSL_ERROR_WANT_WRITE)
                                {
-                                       ServerInstance->SE->ChangeEventMask(user, FD_WANT_NO_READ | FD_WANT_SINGLE_WRITE);
+                                       SocketEngine::ChangeEventMask(user, FD_WANT_NO_READ | FD_WANT_SINGLE_WRITE);
                                        return 0;
                                }
                                else
                                {
-                                       CloseSession(session);
+                                       CloseSession();
                                        return -1;
                                }
                        }
@@ -366,66 +464,62 @@ class OpenSSLIOHook : public SSLIOHook
 
        int OnStreamSocketWrite(StreamSocket* user, std::string& buffer) CXX11_OVERRIDE
        {
-               int fd = user->GetFd();
-
-               issl_session* session = &sessions[fd];
-
-               if (!session->sess)
+               if (!sess)
                {
-                       CloseSession(session);
+                       CloseSession();
                        return -1;
                }
 
-               session->data_to_write = true;
+               data_to_write = true;
 
-               if (session->status == ISSL_HANDSHAKING)
+               if (status == ISSL_HANDSHAKING)
                {
-                       if (!Handshake(user, session))
+                       if (!Handshake(user))
                        {
                                // Couldn't resume handshake.
-                               if (session->status == ISSL_NONE)
+                               if (status == ISSL_NONE)
                                        return -1;
                                return 0;
                        }
                }
 
-               if (session->status == ISSL_OPEN)
+               if (status == ISSL_OPEN)
                {
-                       int ret = SSL_write(session->sess, buffer.data(), buffer.size());
+                       int ret = SSL_write(sess, buffer.data(), buffer.size());
                        if (ret == (int)buffer.length())
                        {
-                               session->data_to_write = false;
-                               ServerInstance->SE->ChangeEventMask(user, FD_WANT_POLL_READ | FD_WANT_NO_WRITE);
+                               data_to_write = false;
+                               SocketEngine::ChangeEventMask(user, FD_WANT_POLL_READ | FD_WANT_NO_WRITE);
                                return 1;
                        }
                        else if (ret > 0)
                        {
                                buffer = buffer.substr(ret);
-                               ServerInstance->SE->ChangeEventMask(user, FD_WANT_SINGLE_WRITE);
+                               SocketEngine::ChangeEventMask(user, FD_WANT_SINGLE_WRITE);
                                return 0;
                        }
                        else if (ret == 0)
                        {
-                               CloseSession(session);
+                               CloseSession();
                                return -1;
                        }
                        else if (ret < 0)
                        {
-                               int err = SSL_get_error(session->sess, ret);
+                               int err = SSL_get_error(sess, ret);
 
                                if (err == SSL_ERROR_WANT_WRITE)
                                {
-                                       ServerInstance->SE->ChangeEventMask(user, FD_WANT_SINGLE_WRITE);
+                                       SocketEngine::ChangeEventMask(user, FD_WANT_SINGLE_WRITE);
                                        return 0;
                                }
                                else if (err == SSL_ERROR_WANT_READ)
                                {
-                                       ServerInstance->SE->ChangeEventMask(user, FD_WANT_POLL_READ);
+                                       SocketEngine::ChangeEventMask(user, FD_WANT_POLL_READ);
                                        return 0;
                                }
                                else
                                {
-                                       CloseSession(session);
+                                       CloseSession();
                                        return -1;
                                }
                        }
@@ -433,20 +527,12 @@ class OpenSSLIOHook : public SSLIOHook
                return 0;
        }
 
-       ssl_cert* GetCertificate(StreamSocket* sock) CXX11_OVERRIDE
-       {
-               int fd = sock->GetFd();
-               issl_session* session = &sessions[fd];
-               return session->cert;
-       }
-
        void TellCiphersAndFingerprint(LocalUser* user)
        {
-               issl_session& s = sessions[user->eh.GetFd()];
-               if (s.sess)
+               if (sess)
                {
-                       std::string text = "*** You are connected using SSL cipher '" + std::string(SSL_get_cipher(s.sess)) + "'";
-                       const std::string& fingerprint = s.cert->fingerprint;
+                       std::string text = "*** You are connected using SSL cipher '" + std::string(SSL_get_cipher(sess)) + "'";
+                       const std::string& fingerprint = certificate->fingerprint;
                        if (!fingerprint.empty())
                                text += " and your SSL fingerprint is " + fingerprint;
 
@@ -455,50 +541,103 @@ class OpenSSLIOHook : public SSLIOHook
        }
 };
 
-class ModuleSSLOpenSSL : public Module
+class OpenSSLIOHookProvider : public refcountbase, public IOHookProvider
 {
-       std::string sslports;
-       OpenSSLIOHook iohook;
+       reference<OpenSSL::Profile> profile;
 
  public:
-       ModuleSSLOpenSSL() : iohook(this)
+       OpenSSLIOHookProvider(Module* mod, reference<OpenSSL::Profile>& prof)
+               : IOHookProvider(mod, "ssl/" + prof->GetName(), IOHookProvider::IOH_SSL)
+               , profile(prof)
        {
-               /* Global SSL library initialization*/
-               SSL_library_init();
-               SSL_load_error_strings();
-
-               /* Build our SSL contexts:
-                * NOTE: OpenSSL makes us have two contexts, one for servers and one for clients. ICK.
-                */
-               iohook.ctx = SSL_CTX_new( SSLv23_server_method() );
-               iohook.clictx = SSL_CTX_new( SSLv23_client_method() );
-
-               SSL_CTX_set_mode(iohook.ctx, SSL_MODE_ENABLE_PARTIAL_WRITE | SSL_MODE_ACCEPT_MOVING_WRITE_BUFFER);
-               SSL_CTX_set_mode(iohook.clictx, SSL_MODE_ENABLE_PARTIAL_WRITE | SSL_MODE_ACCEPT_MOVING_WRITE_BUFFER);
+               ServerInstance->Modules->AddService(*this);
+       }
 
-               SSL_CTX_set_verify(iohook.ctx, SSL_VERIFY_PEER | SSL_VERIFY_CLIENT_ONCE, OnVerify);
-               SSL_CTX_set_verify(iohook.clictx, SSL_VERIFY_PEER | SSL_VERIFY_CLIENT_ONCE, OnVerify);
+       ~OpenSSLIOHookProvider()
+       {
+               ServerInstance->Modules->DelService(*this);
        }
 
-       ~ModuleSSLOpenSSL()
+       void OnAccept(StreamSocket* sock, irc::sockets::sockaddrs* client, irc::sockets::sockaddrs* server) CXX11_OVERRIDE
        {
-               SSL_CTX_free(iohook.ctx);
-               SSL_CTX_free(iohook.clictx);
+               new OpenSSLIOHook(this, sock, false, profile->CreateServerSession(), profile);
        }
 
-       void init() CXX11_OVERRIDE
+       void OnConnect(StreamSocket* sock) CXX11_OVERRIDE
        {
-               // Needs the flag as it ignores a plain /rehash
-               OnModuleRehash(NULL,"ssl");
+               new OpenSSLIOHook(this, sock, true, profile->CreateClientSession(), profile);
        }
+};
+
+class ModuleSSLOpenSSL : public Module
+{
+       typedef std::vector<reference<OpenSSLIOHookProvider> > ProfileList;
 
-       void OnHookIO(StreamSocket* user, ListenSocket* lsb) CXX11_OVERRIDE
+       std::string sslports;
+       ProfileList profiles;
+
+       void ReadProfiles()
        {
-               if (!user->GetIOHook() && lsb->bind_tag->getString("ssl") == "openssl")
+               ProfileList newprofiles;
+               ConfigTagList tags = ServerInstance->Config->ConfTags("sslprofile");
+               if (tags.first == tags.second)
+               {
+                       // Create a default profile named "openssl"
+                       const std::string defname = "openssl";
+                       ConfigTag* tag = ServerInstance->Config->ConfValue(defname);
+                       ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "No <sslprofile> tags found, using settings from the <openssl> tag");
+
+                       try
+                       {
+                               reference<OpenSSL::Profile> profile(new OpenSSL::Profile(defname, tag));
+                               newprofiles.push_back(new OpenSSLIOHookProvider(this, profile));
+                       }
+                       catch (OpenSSL::Exception& ex)
+                       {
+                               throw ModuleException("Error while initializing the default SSL profile - " + ex.GetReason());
+                       }
+               }
+
+               for (ConfigIter i = tags.first; i != tags.second; ++i)
                {
-                       /* Hook the user with our module */
-                       user->AddIOHook(&iohook);
+                       ConfigTag* tag = i->second;
+                       if (tag->getString("provider") != "openssl")
+                               continue;
+
+                       std::string name = tag->getString("name");
+                       if (name.empty())
+                       {
+                               ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Ignoring <sslprofile> tag without name at " + tag->getTagLocation());
+                               continue;
+                       }
+
+                       reference<OpenSSL::Profile> profile;
+                       try
+                       {
+                               profile = new OpenSSL::Profile(name, tag);
+                       }
+                       catch (CoreException& ex)
+                       {
+                               throw ModuleException("Error while initializing SSL profile \"" + name + "\" at " + tag->getTagLocation() + " - " + ex.GetReason());
+                       }
+
+                       newprofiles.push_back(new OpenSSLIOHookProvider(this, profile));
                }
+
+               profiles.swap(newprofiles);
+       }
+
+ public:
+       ModuleSSLOpenSSL()
+       {
+               // Initialize OpenSSL
+               SSL_library_init();
+               SSL_load_error_strings();
+       }
+
+       void init() CXX11_OVERRIDE
+       {
+               ReadProfiles();
        }
 
        void ReadConfig(ConfigStatus& status) CXX11_OVERRIDE
@@ -545,78 +684,14 @@ class ModuleSSLOpenSSL : public Module
                if (param != "ssl")
                        return;
 
-               std::string keyfile;
-               std::string certfile;
-               std::string cafile;
-               std::string dhfile;
-
-               ConfigTag* conf = ServerInstance->Config->ConfValue("openssl");
-
-               cafile   = ServerInstance->Config->Paths.PrependConfig(conf->getString("cafile", "ca.pem"));
-               certfile = ServerInstance->Config->Paths.PrependConfig(conf->getString("certfile", "cert.pem"));
-               keyfile  = ServerInstance->Config->Paths.PrependConfig(conf->getString("keyfile", "key.pem"));
-               dhfile   = ServerInstance->Config->Paths.PrependConfig(conf->getString("dhfile", "dhparams.pem"));
-               std::string hash = conf->getString("hash", "md5");
-
-               iohook.digest = EVP_get_digestbyname(hash.c_str());
-               if (iohook.digest == NULL)
-                       throw ModuleException("Unknown hash type " + hash);
-
-               std::string ciphers = conf->getString("ciphers", "");
-
-               SSL_CTX* ctx = iohook.ctx;
-               SSL_CTX* clictx = iohook.clictx;
-
-               if (!ciphers.empty())
+               try
                {
-                       if ((!SSL_CTX_set_cipher_list(ctx, ciphers.c_str())) || (!SSL_CTX_set_cipher_list(clictx, ciphers.c_str())))
-                       {
-                               ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Can't set cipher list to %s.", ciphers.c_str());
-                               ERR_print_errors_cb(error_callback, this);
-                       }
-               }
-
-               /* Load our keys and certificates
-                * NOTE: OpenSSL's error logging API sucks, don't blame us for this clusterfuck.
-                */
-               if ((!SSL_CTX_use_certificate_chain_file(ctx, certfile.c_str())) || (!SSL_CTX_use_certificate_chain_file(clictx, certfile.c_str())))
-               {
-                       ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Can't read certificate file %s. %s", certfile.c_str(), strerror(errno));
-                       ERR_print_errors_cb(error_callback, this);
-               }
-
-               if (((!SSL_CTX_use_PrivateKey_file(ctx, keyfile.c_str(), SSL_FILETYPE_PEM))) || (!SSL_CTX_use_PrivateKey_file(clictx, keyfile.c_str(), SSL_FILETYPE_PEM)))
-               {
-                       ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Can't read key file %s. %s", keyfile.c_str(), strerror(errno));
-                       ERR_print_errors_cb(error_callback, this);
-               }
-
-               /* Load the CAs we trust*/
-               if (((!SSL_CTX_load_verify_locations(ctx, cafile.c_str(), 0))) || (!SSL_CTX_load_verify_locations(clictx, cafile.c_str(), 0)))
-               {
-                       ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Can't read CA list from %s. This is only a problem if you want to verify client certificates, otherwise it's safe to ignore this message. Error: %s", cafile.c_str(), strerror(errno));
-                       ERR_print_errors_cb(error_callback, this);
+                       ReadProfiles();
                }
-
-               FILE* dhpfile = fopen(dhfile.c_str(), "r");
-               DH* ret;
-
-               if (dhpfile == NULL)
-               {
-                       ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Couldn't open DH file %s: %s", dhfile.c_str(), strerror(errno));
-                       throw ModuleException("Couldn't open DH file " + dhfile + ": " + strerror(errno));
-               }
-               else
+               catch (ModuleException& ex)
                {
-                       ret = PEM_read_DHparams(dhpfile, NULL, NULL, NULL);
-                       if ((SSL_CTX_set_tmp_dh(ctx, ret) < 0) || (SSL_CTX_set_tmp_dh(clictx, ret) < 0))
-                       {
-                               ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Couldn't set DH parameters %s. SSL errors follow:", dhfile.c_str());
-                               ERR_print_errors_cb(error_callback, this);
-                       }
+                       ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, ex.GetReason() + " Not applying settings.");
                }
-
-               fclose(dhpfile);
        }
 
        void On005Numeric(std::map<std::string, std::string>& tokens) CXX11_OVERRIDE
@@ -627,8 +702,9 @@ class ModuleSSLOpenSSL : public Module
 
        void OnUserConnect(LocalUser* user) CXX11_OVERRIDE
        {
-               if (user->eh.GetIOHook() == &iohook)
-                       iohook.TellCiphersAndFingerprint(user);
+               IOHook* hook = user->eh.GetIOHook();
+               if (hook && hook->prov->creator == this)
+                       static_cast<OpenSSLIOHook*>(hook)->TellCiphersAndFingerprint(user);
        }
 
        void OnCleanup(int target_type, void* item) CXX11_OVERRIDE
@@ -637,7 +713,7 @@ class ModuleSSLOpenSSL : public Module
                {
                        LocalUser* user = IS_LOCAL((User*)item);
 
-                       if (user && user->eh.GetIOHook() == &iohook)
+                       if (user && user->eh.GetIOHook() && user->eh.GetIOHook()->prov->creator == this)
                        {
                                // User is using SSL, they're a local user, and they're using one of *our* SSL ports.
                                // Potentially there could be multiple SSL modules loaded at once on different ports.
@@ -652,20 +728,4 @@ class ModuleSSLOpenSSL : public Module
        }
 };
 
-static int error_callback(const char *str, size_t len, void *u)
-{
-       ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "SSL error: " + std::string(str, len - 1));
-
-       //
-       // XXX: Remove this line, it causes valgrind warnings...
-       //
-       // MD_update(&m, buf, j);
-       //
-       //
-       // ... ONLY JOKING! :-)
-       //
-
-       return 0;
-}
-
 MODULE_INIT(ModuleSSLOpenSSL)