Fix segfault in m_chanprotect when OnAccessCheck is called with a null channel

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

diff --git a/src/modules/m_cap.cpp b/src/modules/m_cap.cpp
index 2e9a24811cbdd152c31a22bc83bf9c805c62de99..626b4c23c33169ee728300424f6998e62588a3a5 100644 (file)
--- a/src/modules/m_cap.cpp
+++ b/src/modules/m_cap.cpp
@@ -34,11 +34,9 @@ CAP END
  */
 class CommandCAP : public Command
 {
-       Module* Creator;
  public:
-       CommandCAP (InspIRCd* Instance, Module* mod) : Command(Instance,"CAP", 0, 1, true), Creator(mod)
+       CommandCAP (InspIRCd* Instance, Module* mod) : Command(Instance, mod, "CAP", 0, 1, true)
        {
-               this->source = "m_cap.so";
        }
 
        CmdResult Handle (const std::vector<std::string> &parameters, User *user)
@@ -51,7 +49,7 @@ class CommandCAP : public Command
 
                        Data.type = subcommand;
                        Data.user = user;
-                       Data.creator = this->Creator;
+                       Data.creator = this->creator;
 
                        if (parameters.size() < 2)
                                return CMD_FAILURE;
@@ -67,7 +65,7 @@ class CommandCAP : public Command
                        }
 
                        user->Extend("CAP_REGHOLD");
-                       Event event((char*) &Data, (Module*)this->Creator, "cap_req");
+                       Event event((char*) &Data, this->creator, "cap_req");
                        event.Send(this->ServerInstance);
 
                        if (Data.ack.size() > 0)
@@ -92,10 +90,10 @@ class CommandCAP : public Command
 
                        Data.type = subcommand;
                        Data.user = user;
-                       Data.creator = this->Creator;
+                       Data.creator = this->creator;
 
                        user->Extend("CAP_REGHOLD");
-                       Event event((char*) &Data, (Module*)this->Creator, subcommand == "LS" ? "cap_ls" : "cap_list");
+                       Event event((char*) &Data, this->creator, subcommand == "LS" ? "cap_ls" : "cap_list");
                        event.Send(this->ServerInstance);
 
                        std::string Result;
@@ -112,10 +110,10 @@ class CommandCAP : public Command
 
                        Data.type = subcommand;
                        Data.user = user;
-                       Data.creator = this->Creator;
+                       Data.creator = this->creator;
 
                        user->Extend("CAP_REGHOLD");
-                       Event event((char*) &Data, (Module*)this->Creator, "cap_clear");
+                       Event event((char*) &Data, this->creator, "cap_clear");
                        event.Send(this->ServerInstance);
 
                        std::string Result = irc::stringjoiner(" ", Data.ack, 0, Data.ack.size() - 1).GetJoined();
@@ -143,13 +141,13 @@ class ModuleCAP : public Module
                ServerInstance->Modules->Attach(eventlist, this, 1);
        }
 
-       virtual bool OnCheckReady(User* user)
+       virtual ModResult OnCheckReady(User* user)
        {
                /* Users in CAP state get held until CAP END */
                if (user->GetExt("CAP_REGHOLD"))
-                       return false;
+                       return MOD_RES_DENY;
 
-               return true;
+               return MOD_RES_PASSTHRU;
        }
 
        virtual ~ModuleCAP()