]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/extra/m_ssl_gnutls.cpp
MetaData rework
[user/henk/code/inspircd.git] / src / modules / extra / m_ssl_gnutls.cpp
index 1b9139867e94752a6fd63d76a8ffc84088770f83..a769bd12bfae55aadb200b1b2ce5775974e7c3ab 100644 (file)
@@ -22,8 +22,8 @@
 #endif
 
 /* $ModDesc: Provides SSL support for clients */
-/* $CompileFlags: exec("libgnutls-config --cflags") */
-/* $LinkerFlags: rpath("libgnutls-config --libs") exec("libgnutls-config --libs") */
+/* $CompileFlags: pkgconfincludes("gnutls","/gnutls/gnutls.h","") */
+/* $LinkerFlags: rpath("pkg-config --libs gnutls") pkgconflibs("gnutls","/libgnutls.so","-lgnutls") */
 /* $ModDep: transport.h */
 /* $CopyInstall: conf/key.pem $(CONPATH) */
 /* $CopyInstall: conf/cert.pem $(CONPATH) */
@@ -73,7 +73,7 @@ class CommandStartTLS : public Command
                 */
                if (user->registered != REG_NONE)
                {
-                       ServerInstance->Users->QuitUser(user, "STARTTLS is not permitted after client registration has started");
+                       user->WriteNumeric(691, "%s :STARTTLS is not permitted after client registration has started", user->nick.c_str());
                }
                else
                {
@@ -81,7 +81,7 @@ class CommandStartTLS : public Command
                        {
                                user->WriteNumeric(670, "%s :STARTTLS successful, go ahead with TLS handshake", user->nick.c_str());
                                user->AddIOHook(Caller);
-                               Caller->OnRawSocketAccept(user->GetFd(), user->GetIPString(), user->GetPort());
+                               Caller->OnRawSocketAccept(user->GetFd(), NULL, NULL);
                        }
                        else
                                user->WriteNumeric(691, "%s :STARTTLS failure", user->nick.c_str());
@@ -93,11 +93,6 @@ class CommandStartTLS : public Command
 
 class ModuleSSLGnuTLS : public Module
 {
-
-       ConfigReader* Conf;
-
-       char* dummy;
-
        std::vector<std::string> listenports;
 
        issl_session* sessions;
@@ -115,12 +110,12 @@ class ModuleSSLGnuTLS : public Module
        int clientactive;
        bool cred_alloc;
 
-       CommandStartTLS* starttls;
+       CommandStartTLS starttls;
 
  public:
 
        ModuleSSLGnuTLS(InspIRCd* Me)
-               : Module(Me)
+               : Module(Me), starttls(Me, this)
        {
                ServerInstance->Modules->PublishInterface("BufferedSocketHook", this);
 
@@ -130,35 +125,36 @@ class ModuleSSLGnuTLS : public Module
 
                cred_alloc = false;
                // Needs the flag as it ignores a plain /rehash
-               OnRehash(NULL,"ssl");
+               OnModuleRehash(NULL,"ssl");
 
                // Void return, guess we assume success
                gnutls_certificate_set_dh_params(x509_cred, dh_params);
-               Implementation eventlist[] = { I_On005Numeric, I_OnRawSocketConnect, I_OnRawSocketAccept, I_OnRawSocketClose, I_OnRawSocketRead, I_OnRawSocketWrite, I_OnCleanup,
-                       I_OnBufferFlushed, I_OnRequest, I_OnSyncUserMetaData, I_OnDecodeMetaData, I_OnUnloadModule, I_OnRehash, I_OnWhois, I_OnPostConnect, I_OnEvent, I_OnHookUserIO };
-               ServerInstance->Modules->Attach(eventlist, this, 17);
+               Implementation eventlist[] = { I_On005Numeric, I_OnRawSocketConnect, I_OnRawSocketAccept,
+                       I_OnRawSocketClose, I_OnRawSocketRead, I_OnRawSocketWrite, I_OnCleanup,
+                       I_OnBufferFlushed, I_OnRequest, I_OnUnloadModule, I_OnRehash, I_OnModuleRehash,
+                       I_OnPostConnect, I_OnEvent, I_OnHookUserIO };
+               ServerInstance->Modules->Attach(eventlist, this, sizeof(eventlist)/sizeof(Implementation));
 
-               starttls = new CommandStartTLS(ServerInstance, this);
-               ServerInstance->AddCommand(starttls);
+               ServerInstance->AddCommand(&starttls);
        }
 
-       virtual void OnRehash(User* user, const std::string &param)
+       virtual void OnRehash(User* user)
        {
-               Conf = new ConfigReader(ServerInstance);
+               ConfigReader Conf(ServerInstance);
 
                listenports.clear();
                clientactive = 0;
                sslports.clear();
 
-               for(int index = 0; index < Conf->Enumerate("bind"); index++)
+               for(int index = 0; index < Conf.Enumerate("bind"); index++)
                {
                        // For each <bind> tag
-                       std::string x = Conf->ReadValue("bind", "type", index);
-                       if(((x.empty()) || (x == "clients")) && (Conf->ReadValue("bind", "ssl", index) == "gnutls"))
+                       std::string x = Conf.ReadValue("bind", "type", index);
+                       if(((x.empty()) || (x == "clients")) && (Conf.ReadValue("bind", "ssl", index) == "gnutls"))
                        {
                                // Get the port we're meant to be listening on with SSL
-                               std::string port = Conf->ReadValue("bind", "port", index);
-                               std::string addr = Conf->ReadValue("bind", "address", index);
+                               std::string port = Conf.ReadValue("bind", "port", index);
+                               std::string addr = Conf.ReadValue("bind", "address", index);
 
                                if (!addr.empty())
                                {
@@ -166,7 +162,7 @@ class ModuleSSLGnuTLS : public Module
                                        int portint = 0;
                                        irc::sockets::sockaddrs bin;
                                        if (irc::sockets::aptosa(addr.c_str(), portint, &bin))
-                                               irc::sockets::satoap(addr, portint, &bin);
+                                               irc::sockets::satoap(&bin, addr, portint);
                                }
 
                                irc::portparser portrange(port, false);
@@ -178,12 +174,13 @@ class ModuleSSLGnuTLS : public Module
                                        {
                                                listenports.push_back(addr + ":" + ConvToStr(portno));
 
-                                               for (size_t i = 0; i < ServerInstance->Config->ports.size(); i++)
-                                                       if ((ServerInstance->Config->ports[i]->GetPort() == portno) && (ServerInstance->Config->ports[i]->GetIP() == addr))
-                                                               ServerInstance->Config->ports[i]->SetDescription("ssl");
+                                               for (size_t i = 0; i < ServerInstance->ports.size(); i++)
+                                                       if ((ServerInstance->ports[i]->GetPort() == portno) && (ServerInstance->ports[i]->GetIP() == addr))
+                                                               ServerInstance->ports[i]->SetDescription("ssl");
                                                ServerInstance->Logs->Log("m_ssl_gnutls",DEFAULT, "m_ssl_gnutls.so: Enabling SSL for port %ld", portno);
 
-                                               sslports.append((addr.empty() ? "*" : addr)).append(":").append(ConvToStr(portno)).append(";");
+                                               if (addr != "127.0.0.1")
+                                                       sslports.append((addr.empty() ? "*" : addr)).append(":").append(ConvToStr(portno)).append(";");
                                        }
                                        catch (ModuleException &e)
                                        {
@@ -195,22 +192,26 @@ class ModuleSSLGnuTLS : public Module
 
                if (!sslports.empty())
                        sslports.erase(sslports.end() - 1);
+       }
 
+       virtual void OnModuleRehash(User* user, const std::string &param)
+       {
                if(param != "ssl")
-               {
-                       delete Conf;
                        return;
-               }
+
+               OnRehash(user);
+
+               ConfigReader Conf(ServerInstance);
 
                std::string confdir(ServerInstance->ConfigFileName);
                // +1 so we the path ends with a /
                confdir = confdir.substr(0, confdir.find_last_of('/') + 1);
 
-               cafile  = Conf->ReadValue("gnutls", "cafile", 0);
-               crlfile = Conf->ReadValue("gnutls", "crlfile", 0);
-               certfile        = Conf->ReadValue("gnutls", "certfile", 0);
-               keyfile = Conf->ReadValue("gnutls", "keyfile", 0);
-               dh_bits = Conf->ReadInteger("gnutls", "dhbits", 0, false);
+               cafile = Conf.ReadValue("gnutls", "cafile", 0);
+               crlfile = Conf.ReadValue("gnutls", "crlfile", 0);
+               certfile = Conf.ReadValue("gnutls", "certfile", 0);
+               keyfile = Conf.ReadValue("gnutls", "keyfile", 0);
+               dh_bits = Conf.ReadInteger("gnutls", "dhbits", 0, false);
 
                // Set all the default values needed.
                if (cafile.empty())
@@ -253,16 +254,16 @@ class ModuleSSLGnuTLS : public Module
                        cred_alloc = true;
 
                if((ret = gnutls_certificate_allocate_credentials(&x509_cred)) < 0)
-                       ServerInstance->Logs->Log("m_ssl_gnutls",DEFAULT, "m_ssl_gnutls.so: Failed to allocate certificate credentials: %s", gnutls_strerror(ret));
+                       ServerInstance->Logs->Log("m_ssl_gnutls",DEBUG, "m_ssl_gnutls.so: Failed to allocate certificate credentials: %s", gnutls_strerror(ret));
 
                if((ret = gnutls_dh_params_init(&dh_params)) < 0)
-                       ServerInstance->Logs->Log("m_ssl_gnutls",DEFAULT, "m_ssl_gnutls.so: Failed to initialise DH parameters: %s", gnutls_strerror(ret));
+                       ServerInstance->Logs->Log("m_ssl_gnutls",DEBUG, "m_ssl_gnutls.so: Failed to initialise DH parameters: %s", gnutls_strerror(ret));
 
                if((ret =gnutls_certificate_set_x509_trust_file(x509_cred, cafile.c_str(), GNUTLS_X509_FMT_PEM)) < 0)
-                       ServerInstance->Logs->Log("m_ssl_gnutls",DEFAULT, "m_ssl_gnutls.so: Failed to set X.509 trust file '%s': %s", cafile.c_str(), gnutls_strerror(ret));
+                       ServerInstance->Logs->Log("m_ssl_gnutls",DEBUG, "m_ssl_gnutls.so: Failed to set X.509 trust file '%s': %s", cafile.c_str(), gnutls_strerror(ret));
 
                if((ret = gnutls_certificate_set_x509_crl_file (x509_cred, crlfile.c_str(), GNUTLS_X509_FMT_PEM)) < 0)
-                       ServerInstance->Logs->Log("m_ssl_gnutls",DEFAULT, "m_ssl_gnutls.so: Failed to set X.509 CRL file '%s': %s", crlfile.c_str(), gnutls_strerror(ret));
+                       ServerInstance->Logs->Log("m_ssl_gnutls",DEBUG, "m_ssl_gnutls.so: Failed to set X.509 CRL file '%s': %s", crlfile.c_str(), gnutls_strerror(ret));
 
                if((ret = gnutls_certificate_set_x509_key_file (x509_cred, certfile.c_str(), keyfile.c_str(), GNUTLS_X509_FMT_PEM)) < 0)
                {
@@ -272,8 +273,6 @@ class ModuleSSLGnuTLS : public Module
 
                // This may be on a large (once a day or week) timer eventually.
                GenerateDHParams();
-
-               delete Conf;
        }
 
        void GenerateDHParams()
@@ -302,7 +301,7 @@ class ModuleSSLGnuTLS : public Module
        {
                if(target_type == TYPE_USER)
                {
-                       User* user = (User*)item;
+                       User* user = static_cast<User*>(item);
 
                        if (user->GetIOHook() == this)
                        {
@@ -311,7 +310,7 @@ class ModuleSSLGnuTLS : public Module
                                ServerInstance->Users->QuitUser(user, "SSL module unloading");
                                user->DelIOHook();
                        }
-                       if (user->GetExt("ssl_cert", dummy))
+                       if (user->GetExt("ssl_cert"))
                        {
                                ssl_cert* tofree;
                                user->GetExt("ssl_cert", tofree);
@@ -327,9 +326,9 @@ class ModuleSSLGnuTLS : public Module
                {
                        for(unsigned int i = 0; i < listenports.size(); i++)
                        {
-                               for (size_t j = 0; j < ServerInstance->Config->ports.size(); j++)
-                                       if (listenports[i] == (ServerInstance->Config->ports[j]->GetIP()+":"+ConvToStr(ServerInstance->Config->ports[j]->GetPort())))
-                                               ServerInstance->Config->ports[j]->SetDescription("plaintext");
+                               for (size_t j = 0; j < ServerInstance->ports.size(); j++)
+                                       if (listenports[i] == (ServerInstance->ports[j]->GetIP()+":"+ConvToStr(ServerInstance->ports[j]->GetPort())))
+                                               ServerInstance->ports[j]->SetDescription("plaintext");
                        }
                }
        }
@@ -347,9 +346,9 @@ class ModuleSSLGnuTLS : public Module
                output.append(" STARTTLS");
        }
 
-       virtual void OnHookUserIO(User* user, const std::string &targetip)
+       virtual void OnHookUserIO(User* user)
        {
-               if (!user->GetIOHook() && isin(targetip,user->GetPort(),listenports))
+               if (!user->GetIOHook() && isin(user->GetServerIP(),user->GetServerPort(),listenports))
                {
                        /* Hook the user with our module */
                        user->AddIOHook(this);
@@ -358,7 +357,7 @@ class ModuleSSLGnuTLS : public Module
 
        virtual const char* OnRequest(Request* request)
        {
-               ISHRequest* ISR = (ISHRequest*)request;
+               ISHRequest* ISR = static_cast<ISHRequest*>(request);
                if (strcmp("IS_NAME", request->GetId()) == 0)
                {
                        return "gnutls";
@@ -368,7 +367,7 @@ class ModuleSSLGnuTLS : public Module
                        const char* ret = "OK";
                        try
                        {
-                               ret = ISR->Sock->AddIOHook((Module*)this) ? "OK" : NULL;
+                               ret = ISR->Sock->AddIOHook(this) ? "OK" : NULL;
                        }
                        catch (ModuleException &e)
                        {
@@ -395,19 +394,33 @@ class ModuleSSLGnuTLS : public Module
                                issl_session* session = &sessions[ISR->Sock->GetFd()];
                                if (session->sess)
                                {
-                                       if ((Extensible*)ServerInstance->FindDescriptor(ISR->Sock->GetFd()) == (Extensible*)(ISR->Sock))
+                                       if (static_cast<Extensible*>(ServerInstance->SE->GetRef(ISR->Sock->GetFd())) == static_cast<Extensible*>(ISR->Sock))
                                        {
-                                               VerifyCertificate(session, (BufferedSocket*)ISR->Sock);
+                                               VerifyCertificate(session, ISR->Sock);
                                                return "OK";
                                        }
                                }
                        }
                }
+               else if (strcmp("GET_FP", request->GetId()) == 0)
+               {
+                       if (ISR->Sock->GetFd() > -1)
+                       {
+                               issl_session* session = &sessions[ISR->Sock->GetFd()];
+                               if (session->sess)
+                               {
+                                       Extensible* ext = ISR->Sock;
+                                       ssl_cert* certinfo;
+                                       if (ext->GetExt("ssl_cert",certinfo))
+                                               return certinfo->GetFingerprint().c_str();
+                               }
+                       }
+               }
                return NULL;
        }
 
 
-       virtual void OnRawSocketAccept(int fd, const std::string &ip, int localport)
+       virtual void OnRawSocketAccept(int fd, irc::sockets::sockaddrs* client, irc::sockets::sockaddrs* server)
        {
                /* Are there any possibilities of an out of range fd? Hope not, but lets be paranoid */
                if ((fd < 0) || (fd > ServerInstance->SE->GetMaxFds() - 1))
@@ -425,7 +438,7 @@ class ModuleSSLGnuTLS : public Module
                gnutls_credentials_set(session->sess, GNUTLS_CRD_CERTIFICATE, x509_cred);
                gnutls_dh_set_prime_bits(session->sess, dh_bits);
 
-               gnutls_transport_set_ptr(session->sess, (gnutls_transport_ptr_t) fd); // Give gnutls the fd for the socket.
+               gnutls_transport_set_ptr(session->sess, reinterpret_cast<gnutls_transport_ptr_t>(fd)); // Give gnutls the fd for the socket.
 
                gnutls_certificate_server_set_request(session->sess, GNUTLS_CERT_REQUEST); // Request client certificate if any.
 
@@ -445,7 +458,7 @@ class ModuleSSLGnuTLS : public Module
                gnutls_set_default_priority(session->sess); // Avoid calling all the priority functions, defaults are adequate.
                gnutls_credentials_set(session->sess, GNUTLS_CRD_CERTIFICATE, x509_cred);
                gnutls_dh_set_prime_bits(session->sess, dh_bits);
-               gnutls_transport_set_ptr(session->sess, (gnutls_transport_ptr_t) fd); // Give gnutls the fd for the socket.
+               gnutls_transport_set_ptr(session->sess, reinterpret_cast<gnutls_transport_ptr_t>(fd)); // Give gnutls the fd for the socket.
 
                Handshake(session, fd);
        }
@@ -460,7 +473,7 @@ class ModuleSSLGnuTLS : public Module
 
                EventHandler* user = ServerInstance->SE->GetRef(fd);
 
-               if ((user) && (user->GetExt("ssl_cert", dummy)))
+               if ((user) && (user->GetExt("ssl_cert")))
                {
                        ssl_cert* tofree;
                        user->GetExt("ssl_cert", tofree);
@@ -490,6 +503,7 @@ class ModuleSSLGnuTLS : public Module
 
                        if(!Handshake(session, fd))
                        {
+                               errno = session->status == ISSL_CLOSING ? EIO : EAGAIN;
                                // Couldn't resume handshake.
                                return -1;
                        }
@@ -505,34 +519,43 @@ class ModuleSSLGnuTLS : public Module
 
                if (session->status == ISSL_HANDSHAKEN)
                {
-                       int ret = gnutls_record_recv(session->sess, buffer, count);
-
-                       if (ret > 0)
+                       unsigned int len = 0;
+                       while (len < count)
                        {
-                               readresult = ret;
+                               int ret = gnutls_record_recv(session->sess, buffer + len, count - len);
+                               if (ret > 0)
+                               {
+                                       len += ret;
+                               }
+                               else if (ret == GNUTLS_E_AGAIN || ret == GNUTLS_E_INTERRUPTED)
+                               {
+                                       break;
+                               }
+                               else
+                               {
+                                       if (ret != 0)
+                                               ServerInstance->Logs->Log("m_ssl_gnutls", DEFAULT,
+                                                       "m_ssl_gnutls.so: Error while reading on fd %d: %s",
+                                                       fd, gnutls_strerror(ret));
+
+                                       // if ret == 0, client closed connection.
+                                       readresult = 0;
+                                       CloseSession(session);
+                                       return 1;
+                               }
                        }
-                       else if (ret == 0)
+                       readresult = len;
+                       if (len)
                        {
-                               // Client closed connection.
-                               readresult = 0;
-                               CloseSession(session);
                                return 1;
                        }
-                       else if (ret == GNUTLS_E_AGAIN || ret == GNUTLS_E_INTERRUPTED)
+                       else
                        {
                                errno = EAGAIN;
                                return -1;
                        }
-                       else
-                       {
-                               ServerInstance->Logs->Log("m_ssl_gnutls", DEFAULT,
-                                               "m_ssl_gnutls.so: Error while reading on fd %d: %s",
-                                               fd, gnutls_strerror(ret));
-                               readresult = 0;
-                               CloseSession(session);
-                       }
                }
-               else if(session->status == ISSL_CLOSING)
+               else if (session->status == ISSL_CLOSING)
                        readresult = 0;
 
                return 1;
@@ -561,7 +584,7 @@ class ModuleSSLGnuTLS : public Module
                {
                        // The handshake isn't finished, try to finish it.
                        Handshake(session, fd);
-                       errno = EAGAIN;
+                       errno = session->status == ISSL_CLOSING ? EIO : EAGAIN;
                        return -1;
                }
 
@@ -595,7 +618,8 @@ class ModuleSSLGnuTLS : public Module
                        }
                }
 
-               MakePollWrite(fd);
+               if (!session->outbuf.empty())
+                       MakePollWrite(fd);
 
                /* Who's smart idea was it to return 1 when we havent written anything?
                 * This fucks the buffer up in BufferedSocket :p
@@ -603,48 +627,6 @@ class ModuleSSLGnuTLS : public Module
                return ret < 1 ? 0 : ret;
        }
 
-       // :kenny.chatspike.net 320 Om Epy|AFK :is a Secure Connection
-       virtual void OnWhois(User* source, User* dest)
-       {
-               if (!clientactive)
-                       return;
-
-               // Bugfix, only send this numeric for *our* SSL users
-               if (dest->GetExt("ssl", dummy))
-               {
-                       ServerInstance->SendWhoisLine(source, dest, 320, "%s %s :is using a secure connection", source->nick.c_str(), dest->nick.c_str());
-               }
-       }
-
-       virtual void OnSyncUserMetaData(User* user, Module* proto, void* opaque, const std::string &extname, bool displayable)
-       {
-               // check if the linking module wants to know about OUR metadata
-               if(extname == "ssl")
-               {
-                       // check if this user has an swhois field to send
-                       if(user->GetExt(extname, dummy))
-                       {
-                               // call this function in the linking module, let it format the data how it
-                               // sees fit, and send it on its way. We dont need or want to know how.
-                               proto->ProtoSendMetaData(opaque, TYPE_USER, user, extname, displayable ? "Enabled" : "ON");
-                       }
-               }
-       }
-
-       virtual void OnDecodeMetaData(int target_type, void* target, const std::string &extname, const std::string &extdata)
-       {
-               // check if its our metadata key, and its associated with a user
-               if ((target_type == TYPE_USER) && (extname == "ssl"))
-               {
-                       User* dest = (User*)target;
-                       // if they dont already have an ssl flag, accept the remote server's
-                       if (!dest->GetExt(extname, dummy))
-                       {
-                               dest->Extend(extname, "ON");
-                       }
-               }
-       }
-
        bool Handshake(issl_session* session, int fd)
        {
                int ret = gnutls_handshake(session->sess);
@@ -683,10 +665,10 @@ class ModuleSSLGnuTLS : public Module
                {
                        // Handshake complete.
                        // This will do for setting the ssl flag...it could be done earlier if it's needed. But this seems neater.
-                       User* extendme = ServerInstance->FindDescriptor(fd);
+                       EventHandler *extendme = ServerInstance->SE->GetRef(fd);
                        if (extendme)
                        {
-                               if (!extendme->GetExt("ssl", dummy))
+                               if (!extendme->GetExt("ssl"))
                                        extendme->Extend("ssl", "ON");
                        }
 
@@ -706,10 +688,7 @@ class ModuleSSLGnuTLS : public Module
                // protocol module has propagated the NICK message.
                if (user->GetIOHook() == this && (IS_LOCAL(user)))
                {
-                       // Tell whatever protocol module we're using that we need to inform other servers of this metadata NOW.
-                       ServerInstance->PI->SendMetaData(user, TYPE_USER, "ssl", "on");
-
-                       VerifyCertificate(&sessions[user->GetFd()],user);
+                       ssl_cert* certdata = VerifyCertificate(&sessions[user->GetFd()],user);
                        if (sessions[user->GetFd()].sess)
                        {
                                std::string cipher = gnutls_kx_get_name(gnutls_kx_get(sessions[user->GetFd()].sess));
@@ -717,13 +696,17 @@ class ModuleSSLGnuTLS : public Module
                                cipher.append(gnutls_mac_get_name(gnutls_mac_get(sessions[user->GetFd()].sess)));
                                user->WriteServ("NOTICE %s :*** You are connected using SSL cipher \"%s\"", user->nick.c_str(), cipher.c_str());
                        }
+
+                       ServerInstance->PI->SendMetaData(user, "ssl", "ON");
+                       if (certdata)
+                               ServerInstance->PI->SendMetaData(user, "ssl_cert", certdata->GetMetaLine().c_str());
                }
        }
 
        void MakePollWrite(int fd)
        {
                //OnRawSocketWrite(fd, NULL, 0);
-               EventHandler* eh = ServerInstance->FindDescriptor(fd);
+               EventHandler* eh = ServerInstance->SE->GetRef(fd);
                if (eh)
                        ServerInstance->SE->WantWrite(eh);
        }
@@ -751,10 +734,10 @@ class ModuleSSLGnuTLS : public Module
                session->status = ISSL_NONE;
        }
 
-       void VerifyCertificate(issl_session* session, Extensible* user)
+       ssl_cert* VerifyCertificate(issl_session* session, Extensible* user)
        {
                if (!session->sess || !user)
-                       return;
+                       return NULL;
 
                unsigned int status;
                const gnutls_datum_t* cert_list;
@@ -776,42 +759,14 @@ class ModuleSSLGnuTLS : public Module
 
                if (ret < 0)
                {
-                       certinfo->data.insert(std::make_pair("error",std::string(gnutls_strerror(ret))));
-                       return;
+                       certinfo->error = std::string(gnutls_strerror(ret));
+                       return certinfo;
                }
 
-               if (status & GNUTLS_CERT_INVALID)
-               {
-                       certinfo->data.insert(std::make_pair("invalid",ConvToStr(1)));
-               }
-               else
-               {
-                       certinfo->data.insert(std::make_pair("invalid",ConvToStr(0)));
-               }
-               if (status & GNUTLS_CERT_SIGNER_NOT_FOUND)
-               {
-                       certinfo->data.insert(std::make_pair("unknownsigner",ConvToStr(1)));
-               }
-               else
-               {
-                       certinfo->data.insert(std::make_pair("unknownsigner",ConvToStr(0)));
-               }
-               if (status & GNUTLS_CERT_REVOKED)
-               {
-                       certinfo->data.insert(std::make_pair("revoked",ConvToStr(1)));
-               }
-               else
-               {
-                       certinfo->data.insert(std::make_pair("revoked",ConvToStr(0)));
-               }
-               if (status & GNUTLS_CERT_SIGNER_NOT_CA)
-               {
-                       certinfo->data.insert(std::make_pair("trusted",ConvToStr(0)));
-               }
-               else
-               {
-                       certinfo->data.insert(std::make_pair("trusted",ConvToStr(1)));
-               }
+               certinfo->invalid = (status & GNUTLS_CERT_INVALID);
+               certinfo->unknownsigner = (status & GNUTLS_CERT_SIGNER_NOT_FOUND);
+               certinfo->revoked = (status & GNUTLS_CERT_REVOKED);
+               certinfo->trusted = !(status & GNUTLS_CERT_SIGNER_NOT_CA);
 
                /* Up to here the process is the same for X.509 certificates and
                 * OpenPGP keys. From now on X.509 certificates are assumed. This can
@@ -819,23 +774,23 @@ class ModuleSSLGnuTLS : public Module
                 */
                if (gnutls_certificate_type_get(session->sess) != GNUTLS_CRT_X509)
                {
-                       certinfo->data.insert(std::make_pair("error","No X509 keys sent"));
-                       return;
+                       certinfo->error = "No X509 keys sent";
+                       return certinfo;
                }
 
                ret = gnutls_x509_crt_init(&cert);
                if (ret < 0)
                {
-                       certinfo->data.insert(std::make_pair("error",gnutls_strerror(ret)));
-                       return;
+                       certinfo->error = gnutls_strerror(ret);
+                       return certinfo;
                }
 
                cert_list_size = 0;
                cert_list = gnutls_certificate_get_peers(session->sess, &cert_list_size);
                if (cert_list == NULL)
                {
-                       certinfo->data.insert(std::make_pair("error","No certificate was found"));
-                       return;
+                       certinfo->error = "No certificate was found";
+                       return certinfo;
                }
 
                /* This is not a real world example, since we only check the first
@@ -845,37 +800,35 @@ class ModuleSSLGnuTLS : public Module
                ret = gnutls_x509_crt_import(cert, &cert_list[0], GNUTLS_X509_FMT_DER);
                if (ret < 0)
                {
-                       certinfo->data.insert(std::make_pair("error",gnutls_strerror(ret)));
-                       return;
+                       certinfo->error = gnutls_strerror(ret);
+                       return certinfo;
                }
 
                gnutls_x509_crt_get_dn(cert, name, &name_size);
-
-               certinfo->data.insert(std::make_pair("dn",name));
+               certinfo->dn = name;
 
                gnutls_x509_crt_get_issuer_dn(cert, name, &name_size);
-
-               certinfo->data.insert(std::make_pair("issuer",name));
+               certinfo->issuer = name;
 
                if ((ret = gnutls_x509_crt_get_fingerprint(cert, GNUTLS_DIG_MD5, digest, &digest_size)) < 0)
                {
-                       certinfo->data.insert(std::make_pair("error",gnutls_strerror(ret)));
+                       certinfo->error = gnutls_strerror(ret);
                }
                else
                {
-                       certinfo->data.insert(std::make_pair("fingerprint",irc::hex(digest, digest_size)));
+                       certinfo->fingerprint = irc::hex(digest, digest_size);
                }
 
                /* Beware here we do not check for errors.
                 */
                if ((gnutls_x509_crt_get_expiration_time(cert) < ServerInstance->Time()) || (gnutls_x509_crt_get_activation_time(cert) > ServerInstance->Time()))
                {
-                       certinfo->data.insert(std::make_pair("error","Not activated, or expired certificate"));
+                       certinfo->error = "Not activated, or expired certificate";
                }
 
                gnutls_x509_crt_deinit(cert);
 
-               return;
+               return certinfo;
        }
 
        void OnEvent(Event* ev)