Fix memory leak and invalid vtable location on unload of m_sslinfo

[user/henk/code/inspircd.git] / src / modules / m_sslinfo.cpp

diff --git a/src/modules/m_sslinfo.cpp b/src/modules/m_sslinfo.cpp
index 93f2a71dbdeb152d40c939351c4d7ff26cfad0a4..0f1f04e99cfe99f8676e30ef5ad6c807fe1ee891 100644 (file)
--- a/src/modules/m_sslinfo.cpp
+++ b/src/modules/m_sslinfo.cpp
@@ -83,6 +83,14 @@ class ModuleSSLInfo : public Module
                return Version("$Id$", VF_VENDOR, API_VERSION);
        }
 
+       virtual void OnCleanup(int target_type, void* item)
+       {
+               if (target_type != TYPE_USER)
+                       return;
+               User* user = static_cast<User*>(item);
+               user->Shrink("ssl_cert");
+       }
+
        virtual void OnWhois(User* source, User* dest)
        {
                if(dest->GetExt("ssl"))
@@ -156,7 +164,7 @@ class ModuleSSLInfo : public Module
                return false;
        }
 
-       virtual int OnPreCommand(std::string &command, std::vector<std::string> &parameters, User *user, bool validated, const std::string &original_line)
+       virtual ModResult OnPreCommand(std::string &command, std::vector<std::string> &parameters, User *user, bool validated, const std::string &original_line)
        {
                irc::string pcmd = command.c_str();
 
@@ -204,7 +212,7 @@ class ModuleSSLInfo : public Module
                                if (SSLOnly && !user->GetExt("ssl"))
                                {
                                        user->WriteNumeric(491, "%s :This oper login name requires an SSL connection.", user->nick.c_str());
-                                       return 1;
+                                       return MOD_RES_DENY;
                                }
 
                                /*
@@ -215,13 +223,13 @@ class ModuleSSLInfo : public Module
                                        user->WriteNumeric(491, "%s :This oper login name requires a matching key fingerprint.",user->nick.c_str());
                                        ServerInstance->SNO->WriteToSnoMask('o',"'%s' cannot oper, does not match fingerprint", user->nick.c_str());
                                        ServerInstance->Logs->Log("m_ssl_oper_cert",DEFAULT,"OPER: Failed oper attempt by %s!%s@%s: credentials valid, but wrong fingerprint.", user->nick.c_str(), user->ident.c_str(), user->host.c_str());
-                                       return 1;
+                                       return MOD_RES_DENY;
                                }
                        }
                }
 
                // Let core handle it for extra stuff
-               return 0;
+               return MOD_RES_PASSTHRU;
        }