]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_chanprotect.cpp
Annotations
[user/henk/code/inspircd.git] / src / modules / m_chanprotect.cpp
index 99d219d24dbf7277e054a26c255660786c21307c..2f384f0012d8115ec92f228d66bd24b5b7d2c758 100644 (file)
-#include <stdio.h>
+/*       +------------------------------------+
+ *       | Inspire Internet Relay Chat Daemon |
+ *       +------------------------------------+
+ *
+ *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
+ *                    E-mail:
+ *             <brain@chatspike.net>
+ *               <Craig@chatspike.net>
+ *     
+ * Written by Craig Edwards, Craig McLure, and others.
+ * This program is free but copyrighted software; see
+ *         the file COPYING for details.
+ *
+ * ---------------------------------------------------
+ */
 
 #include "users.h"
 #include "channels.h"
 #include "modules.h"
+#include "inspircd.h"
 
 /* $ModDesc: Provides channel modes +a and +q */
 
-char dummyvalue[] = "on";
+#define PROTECT_VALUE 40000
+#define FOUNDER_VALUE 50000
+
+const char* fakevalue = "on";
+
+/* When this is set to true, no restrictions apply to setting or
+ * removal of +qa. This is used while unloading so that the server
+ * can freely clear all of its users of the modes.
+ */
+bool unload_kludge = false;
+
+/** Handles basic operation of +qa channel modes
+ */
+class FounderProtectBase
+{
+ private:
+       InspIRCd* MyInstance;
+       std::string extend;
+       std::string type;
+       int list;
+       int end;
+       char* dummyptr;
+ public:
+       FounderProtectBase(InspIRCd* Instance, const std::string &ext, const std::string &mtype, int l, int e) : MyInstance(Instance), extend(ext), type(mtype), list(l), end(e)
+       {
+       }
+
+       ModePair ModeSet(userrec* source, userrec* dest, chanrec* channel, const std::string &parameter)
+       {
+               userrec* x = MyInstance->FindNick(parameter);
+               if (x)
+               {
+                       if (!channel->HasUser(x))
+                       {
+                               return std::make_pair(false, parameter);
+                       }
+                       else
+                       {
+                               std::string item = extend+std::string(channel->name);
+                               if (x->GetExt(item,dummyptr))
+                               {
+                                       return std::make_pair(true, x->nick);
+                               }
+                               else
+                               {
+                                       return std::make_pair(false, parameter);
+                               }
+                       }
+               }
+               return std::make_pair(false, parameter);
+       }
+
+       void RemoveMode(chanrec* channel, char mc)
+       {
+               unload_kludge = true;
+               CUList* cl = channel->GetUsers();
+               std::string item = extend+std::string(channel->name);
+               char moderemove[MAXBUF];
+               userrec* n = new userrec(MyInstance);
+               n->SetFd(FD_MAGIC_NUMBER);
+               for (CUList::iterator i = cl->begin(); i != cl->end(); i++)
+               {
+                       if (i->second->GetExt(item, dummyptr))
+                       {
+                               sprintf(moderemove,"-%c",mc);
+                               const char* parameters[] = { channel->name, moderemove, i->second->nick };
+                               MyInstance->SendMode(parameters, 3, n);
+                       }
+               }
+               delete n;
+               unload_kludge = false;
+       }
+
+        void DisplayList(userrec* user, chanrec* channel)
+       {
+               CUList* cl = channel->GetUsers();
+               std::string item = extend+std::string(channel->name);
+               for (CUList::iterator i = cl->begin(); i != cl->end(); i++)
+               {
+                       if (i->second->GetExt(item, dummyptr))
+                       {
+                               user->WriteServ("%d %s %s %s", list, user->nick, channel->name,i->second->nick);
+                       }
+               }
+               user->WriteServ("%d %s %s :End of channel %s list", end, user->nick, channel->name, type.c_str());
+       }
+
+       userrec* FindAndVerify(std::string &parameter, chanrec* channel)
+       {
+               userrec* theuser = MyInstance->FindNick(parameter);
+               if ((!theuser) || (!channel->HasUser(theuser)))
+               {
+                       parameter = "";
+                       return NULL;
+               }
+               return theuser;
+       }
+
+       ModeAction HandleChange(userrec* source, userrec* theuser, bool adding, chanrec* channel, std::string &parameter)
+       {
+               std::string item = extend+std::string(channel->name);
+
+               if (adding)
+               {
+                       if (!theuser->GetExt(item, dummyptr))
+                       {
+                               theuser->Extend(item, fakevalue);
+                               parameter = theuser->nick;
+                               return MODEACTION_ALLOW;
+                       }
+               }
+               else
+               {
+                       if (theuser->GetExt(item, dummyptr))
+                       {
+                               theuser->Shrink(item);
+                               parameter = theuser->nick;
+                               return MODEACTION_ALLOW;
+                       }
+               }
+               return MODEACTION_DENY;
+       }
+};
+
+/** Abstraction of FounderProtectBase for channel mode +q
+ */
+class ChanFounder : public ModeHandler, public FounderProtectBase
+{
+       char* dummyptr;
+ public:
+       ChanFounder(InspIRCd* Instance, bool using_prefixes)
+               : ModeHandler(Instance, 'q', 1, 1, true, MODETYPE_CHANNEL, false, using_prefixes ? '~' : 0),
+                 FounderProtectBase(Instance, "cm_founder_", "founder", 386, 387) { }
+
+       unsigned int GetPrefixRank()
+       {
+               return FOUNDER_VALUE;
+       }
+
+       ModePair ModeSet(userrec* source, userrec* dest, chanrec* channel, const std::string &parameter)
+       {
+               return FounderProtectBase::ModeSet(source, dest, channel, parameter);
+       }
+
+       void RemoveMode(chanrec* channel)
+       {
+               FounderProtectBase::RemoveMode(channel, this->GetModeChar());
+       }
+
+       void RemoveMode(userrec* user)
+       {
+       }
+
+       ModeAction OnModeChange(userrec* source, userrec* dest, chanrec* channel, std::string &parameter, bool adding)
+       {
+               userrec* theuser = FounderProtectBase::FindAndVerify(parameter, channel);
+
+               if (!theuser)
+               {
+                       return MODEACTION_DENY;
+               }
+
+                // source is a server, or ulined, we'll let them +-q the user.
+               if ((unload_kludge) || (ServerInstance->ULine(source->nick)) || (ServerInstance->ULine(source->server)) || (!*source->server) || (!IS_LOCAL(source)))
+               {
+                       return FounderProtectBase::HandleChange(source, theuser, adding, channel, parameter);
+               }
+               else
+               {
+                       // whoops, someones being naughty!
+                       source->WriteServ("468 %s %s :Only servers may set channel mode +q",source->nick, channel->name);
+                       parameter = "";
+                       return MODEACTION_DENY;
+               }
+       }
+
+       void DisplayList(userrec* user, chanrec* channel)
+       {
+               FounderProtectBase::DisplayList(user,channel);
+       }
+};
+
+/** Abstraction of FounderProtectBase for channel mode +a
+ */
+class ChanProtect : public ModeHandler, public FounderProtectBase
+{
+       char* dummyptr;
+ public:
+       ChanProtect(InspIRCd* Instance, bool using_prefixes)
+               : ModeHandler(Instance, 'a', 1, 1, true, MODETYPE_CHANNEL, false, using_prefixes ? '&' : 0),
+                 FounderProtectBase(Instance,"cm_protect_","protected user", 388, 389) { }
+
+       unsigned int GetPrefixRank()
+       {
+               return PROTECT_VALUE;
+       }
+
+       ModePair ModeSet(userrec* source, userrec* dest, chanrec* channel, const std::string &parameter)
+       {
+               return FounderProtectBase::ModeSet(source, dest, channel, parameter);
+       }
+
+       void RemoveMode(chanrec* channel)
+       {
+               FounderProtectBase::RemoveMode(channel, this->GetModeChar());
+       }
+
+       void RemoveMode(userrec* user)
+       {
+       }
+
+       ModeAction OnModeChange(userrec* source, userrec* dest, chanrec* channel, std::string &parameter, bool adding)
+       {
+               userrec* theuser = FounderProtectBase::FindAndVerify(parameter, channel);
+
+               if (!theuser)
+                       return MODEACTION_DENY;
+
+               std::string founder = "cm_founder_"+std::string(channel->name);
+
+               // source has +q, is a server, or ulined, we'll let them +-a the user.
+               if ((unload_kludge) || (ServerInstance->ULine(source->nick)) || (ServerInstance->ULine(source->server)) || (!*source->server) || (source->GetExt(founder,dummyptr)) || (!IS_LOCAL(source)))
+               {
+                       return FounderProtectBase::HandleChange(source, theuser, adding, channel, parameter);
+               }
+               else
+               {
+                       // bzzzt, wrong answer!
+                       source->WriteServ("482 %s %s :You are not a channel founder",source->nick, channel->name);
+                       return MODEACTION_DENY;
+               }
+       }
+
+       virtual void DisplayList(userrec* user, chanrec* channel)
+       {
+               FounderProtectBase::DisplayList(user, channel);
+       }
+
+};
 
 class ModuleChanProtect : public Module
 {
-       Server *Srv;
+       
        bool FirstInGetsFounder;
-       ConfigReader *Conf;
+       bool QAPrefixes;
+       ChanProtect* cp;
+       ChanFounder* cf;
+       char* dummyptr;
        
  public:
  
-       ModuleChanProtect()
+       ModuleChanProtect(InspIRCd* Me) : Module::Module(Me)
+       {       
+               /* Load config stuff */
+               OnRehash("");
+
+               /* Initialise module variables */
+
+               cp = new ChanProtect(ServerInstance,QAPrefixes);
+               cf = new ChanFounder(ServerInstance,QAPrefixes);
+
+               ServerInstance->AddMode(cp, 'a');
+               ServerInstance->AddMode(cf, 'q');
+       }
+
+       void Implements(char* List)
        {
-       
-               // here we initialise our module. Use new to create new instances of the required
-               // classes.
-               
-               Srv = new Server;
-               Conf = new ConfigReader;
-               
-               // set up our modes. We're using listmodes and not normal extmodes here.
-               // listmodes only need one parameter as everything else is assumed by the
-               // nature of the mode thats being created.
-               Srv->AddExtendedListMode('a');
-               Srv->AddExtendedListMode('q');
-               
-               // read our config options (main config file)
-               FirstInGetsFounder = Conf->ReadFlag("options","noservices",0);
+               List[I_OnUserKick] = List[I_OnUserPart] = List[I_OnRehash] = List[I_OnUserJoin] = List[I_OnAccessCheck] = List[I_OnSyncChannel] = 1;
        }
-       
-       virtual void OnRehash()
+
+       virtual void OnUserKick(userrec* source, userrec* user, chanrec* chan, const std::string &reason)
+       {
+               // FIX: when someone gets kicked from a channel we must remove their Extensibles!
+               user->Shrink("cm_founder_"+std::string(chan->name));
+               user->Shrink("cm_protect_"+std::string(chan->name));
+       }
+
+       virtual void OnUserPart(userrec* user, chanrec* channel, const std::string &partreason)
+       {
+               // FIX: when someone parts a channel we must remove their Extensibles!
+               user->Shrink("cm_founder_"+std::string(channel->name));
+               user->Shrink("cm_protect_"+std::string(channel->name));
+       }
+
+       virtual void OnRehash(const std::string &parameter)
        {
-               // on a rehash we delete our classes for good measure and create them again.
-               delete Conf;
-               Conf = new ConfigReader;
-               // re-read our config options on a rehash
-               FirstInGetsFounder = Conf->ReadFlag("options","noservices",0);
+               /* Create a configreader class and read our flag,
+                * in old versions this was heap-allocated and the
+                * object was kept between rehashes...now we just
+                * stack-allocate it locally.
+                */
+               ConfigReader Conf(ServerInstance);
+               
+               FirstInGetsFounder = Conf.ReadFlag("options","noservices",0);
+               QAPrefixes = Conf.ReadFlag("options","qaprefixes",0);
        }
        
        virtual void OnUserJoin(userrec* user, chanrec* channel)
@@ -50,15 +324,20 @@ class ModuleChanProtect : public Module
                // the config option for it is set
                if (FirstInGetsFounder)
                {
-                       if (Srv->CountUsers(channel) == 1)
+                       if (channel->GetUserCounter() == 1)
                        {
                                // we're using Extensible::Extend to add data into user objects.
                                // this way is best as it adds data thats accessible to other modules
                                // (so long as you document your code properly) without breaking anything
                                // because its encapsulated neatly in a map.
-                               if (user->Extend("cm_founder_"+std::string(channel->name),dummyvalue))
+
+                               // Change requested by katsklaw... when the first in is set to get founder,
+                               // to make it clearer that +q has been given, send that one user the +q notice
+                               // so that their client's syncronization and their sanity are left intact.
+                               user->WriteServ("MODE %s +q %s",channel->name,user->nick);
+                               if (user->Extend("cm_founder_"+std::string(channel->name),fakevalue))
                                {
-                                       Srv->Log(DEBUG,"Marked user "+std::string(user->nick)+" as founder for "+std::string(channel->name));
+                                       ServerInstance->Log(DEBUG,"Marked user "+std::string(user->nick)+" as founder for "+std::string(channel->name));
                                }
                        }
                }
@@ -70,6 +349,7 @@ class ModuleChanProtect : public Module
                // etc of protected users. There are many types of access check, we're going to handle
                // a relatively small number of them relevent to our module using a switch statement.
        
+               ServerInstance->Log(DEBUG,"chanprotect OnAccessCheck %d",access_type);
                // don't allow action if:
                // (A) Theyre founder (no matter what)
                // (B) Theyre protected, and you're not
@@ -79,196 +359,119 @@ class ModuleChanProtect : public Module
                
                // firstly, if a ulined nick, or a server, is setting the mode, then allow them to set the mode
                // without any access checks, we're not worthy :p
-               if ((Srv->IsUlined(source->nick)) || (Srv->IsUlined(source->server)) || (!strcmp(source->server,"")))
+               if ((ServerInstance->ULine(source->nick)) || (ServerInstance->ULine(source->server)) || (!*source->server))
                {
+                       ServerInstance->Log(DEBUG,"chanprotect OnAccessCheck returns ALLOW");
                        return ACR_ALLOW;
                }
 
+               std::string founder = "cm_founder_"+std::string(channel->name);
+               std::string protect = "cm_protect_"+std::string(channel->name);
+
                switch (access_type)
                {
                        // a user has been deopped. Do we let them? hmmm...
                        case AC_DEOP:
-                               if (dest->GetExt("cm_founder_"+std::string(channel->name)))
+                               ServerInstance->Log(DEBUG,"OnAccessCheck AC_DEOP");
+                               if (dest->GetExt(founder,dummyptr))
                                {
-                                       Srv->SendServ(source->fd,"482 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't deop "+std::string(dest->nick)+" as the're a channel founder");
+                                       ServerInstance->Log(DEBUG,"Has %s",founder.c_str());
+                                       source->WriteServ("484 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't deop "+std::string(dest->nick)+" as they're a channel founder");
                                        return ACR_DENY;
                                }
-                               if ((dest->GetExt("cm_protect_"+std::string(channel->name))) && (!source->GetExt("cm_protect_"+std::string(channel->name))))
+                               else
+                               {
+                                       ServerInstance->Log(DEBUG,"Doesnt have %s",founder.c_str());
+                               }
+                               if ((dest->GetExt(protect,dummyptr)) && (!source->GetExt(protect,dummyptr)))
                                {
-                                       Srv->SendServ(source->fd,"482 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't deop "+std::string(dest->nick)+" as the're protected (+a)");
+                                       source->WriteServ("484 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't deop "+std::string(dest->nick)+" as they're protected (+a)");
                                        return ACR_DENY;
                                }
                        break;
 
                        // a user is being kicked. do we chop off the end of the army boot?
                        case AC_KICK:
-                               if (dest->GetExt("cm_founder_"+std::string(channel->name)))
+                               ServerInstance->Log(DEBUG,"OnAccessCheck AC_KICK");
+                               if (dest->GetExt(founder,dummyptr))
                                {
-                                       Srv->SendServ(source->fd,"482 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't kick "+std::string(dest->nick)+" as the're a channel founder");
+                                       source->WriteServ("484 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't kick "+std::string(dest->nick)+" as they're a channel founder");
                                        return ACR_DENY;
                                }
-                               if ((dest->GetExt("cm_protect_"+std::string(channel->name))) && (!source->GetExt("cm_protect_"+std::string(channel->name))))
+                               if ((dest->GetExt(protect,dummyptr)) && (!source->GetExt(protect,dummyptr)))
                                {
-                                       Srv->SendServ(source->fd,"482 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't kick "+std::string(dest->nick)+" as the're protected (+a)");
+                                       source->WriteServ("484 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't kick "+std::string(dest->nick)+" as they're protected (+a)");
                                        return ACR_DENY;
                                }
                        break;
 
                        // a user is being dehalfopped. Yes, we do disallow -h of a +ha user
                        case AC_DEHALFOP:
-                               if (dest->GetExt("cm_founder_"+std::string(channel->name)))
+                               if (dest->GetExt(founder,dummyptr))
                                {
-                                       Srv->SendServ(source->fd,"482 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't de-halfop "+std::string(dest->nick)+" as the're a channel founder");
+                                       source->WriteServ("484 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't de-halfop "+std::string(dest->nick)+" as they're a channel founder");
                                        return ACR_DENY;
                                }
-                               if ((dest->GetExt("cm_protect_"+std::string(channel->name))) && (!source->GetExt("cm_protect_"+std::string(channel->name))))
+                               if ((dest->GetExt(protect,dummyptr)) && (!source->GetExt(protect,dummyptr)))
                                {
-                                       Srv->SendServ(source->fd,"482 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't de-halfop "+std::string(dest->nick)+" as the're protected (+a)");
+                                       source->WriteServ("484 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't de-halfop "+std::string(dest->nick)+" as they're protected (+a)");
                                        return ACR_DENY;
                                }
                        break;
 
                        // same with devoice.
                        case AC_DEVOICE:
-                               if (dest->GetExt("cm_founder_"+std::string(channel->name)))
+                               if (dest->GetExt(founder,dummyptr))
                                {
-                                       Srv->SendServ(source->fd,"482 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't devoice "+std::string(dest->nick)+" as the're a channel founder");
+                                       source->WriteServ("484 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't devoice "+std::string(dest->nick)+" as they're a channel founder");
                                        return ACR_DENY;
                                }
-                               if ((dest->GetExt("cm_protect_"+std::string(channel->name))) && (!source->GetExt("cm_protect_"+std::string(channel->name))))
+                               if ((dest->GetExt(protect,dummyptr)) && (!source->GetExt(protect,dummyptr)))
                                {
-                                       Srv->SendServ(source->fd,"482 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't devoice "+std::string(dest->nick)+" as the're protected (+a)");
+                                       source->WriteServ("484 "+std::string(source->nick)+" "+std::string(channel->name)+" :Can't devoice "+std::string(dest->nick)+" as they're protected (+a)");
                                        return ACR_DENY;
                                }
                        break;
                }
                
                // we dont know what this access check is, or dont care. just carry on, nothing to see here.
+               ServerInstance->Log(DEBUG,"chanprotect OnAccessCheck returns DEFAULT");
                return ACR_DEFAULT;
        }
        
-       virtual int OnExtendedMode(userrec* user, void* target, char modechar, int type, bool mode_on, string_list &params)
-       {
-               // not out mode, bail
-               if ((modechar == 'q') && (type == MT_CHANNEL))
-               {
-                       // set up parameters
-                       chanrec* chan = (chanrec*)target;
-                       userrec* theuser = Srv->FindNick(params[0]);
-               
-                       // cant find the user given as the parameter, eat the mode change.
-                       if (!theuser)
-                               return -1;
-                       
-                       // given user isnt even on the channel, eat the mode change
-                       if (!Srv->IsOnChannel(theuser,chan))
-                               return -1;
-                       
-                       // source is a server, or ulined, we'll let them +-q the user.
-                       if ((Srv->IsUlined(user->nick)) || (Srv->IsUlined(user->server)) || (!strcmp(user->server,"")))
-                       {
-                               if (mode_on)
-                               {
-                                       if (!theuser->GetExt("cm_founder_"+std::string(chan->name)))
-                                       {
-                                               theuser->Extend("cm_founder_"+std::string(chan->name),dummyvalue);
-                                               return 1;
-                                       }
-                               }
-                               else
-                               {
-                                       if (theuser->GetExt("cm_founder_"+std::string(chan->name)))
-                                       {
-                                               theuser->Shrink("cm_founder_"+std::string(chan->name));
-                                               return 1;
-                                       }
-                               }       
-
-                               return -1;
-                       }
-                       else
-                       {
-                               // whoops, someones being naughty!
-                               WriteServ(user->fd,"468 %s %s :Only servers may set channel mode +q",user->nick, chan->name);
-                               return -1;
-                       }
-               }
-               if ((modechar == 'a') && (type == MT_CHANNEL))
-               {
-                       // set up parameters
-                       chanrec* chan = (chanrec*)target;
-                       userrec* theuser = Srv->FindNick(params[0]);
-               
-                       // cant find the user given as the parameter, eat the mode change.
-                       if (!theuser)
-                               return -1;
-                       
-                       // given user isnt even on the channel, eat the mode change
-                       if (!Srv->IsOnChannel(theuser,chan))
-                               return -1;
-
-                       // source has +q, is a server, or ulined, we'll let them +-a the user.
-                       if ((Srv->IsUlined(user->nick)) || (Srv->IsUlined(user->server)) || (!strcmp(user->server,"")) || (user->GetExt("cm_founder_"+std::string(chan->name))))
-                       {
-                               if (mode_on)
-                               {
-                                       if (!theuser->GetExt("cm_protect_"+std::string(chan->name)))
-                                       {
-                                               theuser->Extend("cm_protect_"+std::string(chan->name),dummyvalue);
-                                               return 1;
-                                       }
-                               }
-                               else
-                               {
-                                       if (theuser->GetExt("cm_protect_"+std::string(chan->name)))
-                                       {
-                                               theuser->Shrink("cm_protect_"+std::string(chan->name));
-                                               return 1;
-                                       }
-                               }       
-
-                               return -1;
-                       }
-                       else
-                       {
-                               // bzzzt, wrong answer!
-                               WriteServ(user->fd,"482 %s %s :You are not a channel founder",user->nick, chan->name);
-                               return -1;
-                       }
-               }
-               return 0;
-       }
-       
        virtual ~ModuleChanProtect()
        {
-               delete Conf;
-               delete Srv;
+               ServerInstance->Modes->DelMode(cp);
+               ServerInstance->Modes->DelMode(cf);
+               DELETE(cp);
+               DELETE(cf);
        }
        
        virtual Version GetVersion()
        {
-               return Version(1,0,0,0);
+               return Version(1, 0, 0, 0, VF_COMMON | VF_VENDOR);
        }
        
-       virtual string_list OnChannelSync(chanrec* chan)
+       virtual void OnSyncChannel(chanrec* chan, Module* proto, void* opaque)
        {
                // this is called when the server is linking into a net and wants to sync channel data.
                // we should send our mode changes for the channel here to ensure that other servers
                // know whos +q/+a on the channel.
-               chanuserlist cl = Srv->GetUsers(chan);
+               CUList* cl = chan->GetUsers();
                string_list commands;
-               for (int i = 0; i < cl.size(); i++)
+               std::string founder = "cm_founder_"+std::string(chan->name);
+               std::string protect = "cm_protect_"+std::string(chan->name);
+               for (CUList::iterator i = cl->begin(); i != cl->end(); i++)
                {
-                       if (cl[i]->GetExt("cm_founder_"+std::string(chan->name)))
+                       if (i->second->GetExt(founder,dummyptr))
                        {
-                               commands.push_back("M "+std::string(chan->name)+" +q "+std::string(cl[i]->nick));
+                               proto->ProtoSendMode(opaque,TYPE_CHANNEL,chan,"+q "+std::string(i->second->nick));
                        }
-                       if (cl[i]->GetExt("cm_protect_"+std::string(chan->name)))
+                       if (i->second->GetExt(protect,dummyptr))
                        {
-                               commands.push_back("M "+std::string(chan->name)+" +a "+std::string(cl[i]->nick));
+                               proto->ProtoSendMode(opaque,TYPE_CHANNEL,chan,"+a "+std::string(i->second->nick));
                        }
                }
-               return commands;
        }
 
 };
@@ -285,9 +488,9 @@ class ModuleChanProtectFactory : public ModuleFactory
        {
        }
        
-       virtual Module * CreateModule()
+       virtual Module * CreateModule(InspIRCd* Me)
        {
-               return new ModuleChanProtect;
+               return new ModuleChanProtect(Me);
        }
        
 };
@@ -297,4 +500,3 @@ extern "C" void * init_module( void )
 {
        return new ModuleChanProtectFactory;
 }
-