Remove .c_str()'s in match() calls that are no longer needed as match() natively...

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

diff --git a/src/modules/m_denychans.cpp b/src/modules/m_denychans.cpp
index e0b3829bd07be092251ddbb8bed677c90a8bd550..8d242edce4ac8feff5218050d4ad5222f8f94a5b 100644 (file)
--- a/src/modules/m_denychans.cpp
+++ b/src/modules/m_denychans.cpp
@@ -2,7 +2,7 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2007 InspIRCd Development Team
+ *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
  * See: http://www.inspircd.org/wiki/index.php/Credits
  *
  * This program is free but copyrighted software; see
@@ -34,30 +34,64 @@ class ModuleDenyChannels : public Module
        
        virtual void OnRehash(User* user, const std::string &param)
        {
-               DELETE(Conf);
+               delete Conf;
                Conf = new ConfigReader(ServerInstance);
+               /* check for redirect validity and loops/chains */
+               for (int i =0; i < Conf->Enumerate("badchan"); i++)
+               {
+                       std::string name = Conf->ReadValue("badchan","name",i);
+                       std::string redirect = Conf->ReadValue("badchan","redirect",i);
+                       
+                       if (!redirect.empty())
+                       {
+                       
+                               if (!ServerInstance->IsChannel(redirect.c_str()))
+                               {
+                                       if (user)
+                                               user->WriteServ("Notice %s :Invalid badchan redirect '%s'", user->nick, redirect.c_str());
+                                       throw ModuleException("Invalid badchan redirect, not a channel");
+                               }
+       
+                               for (int j =0; j < Conf->Enumerate("badchan"); j++)
+                               {
+                                       if (match(redirect, Conf->ReadValue("badchan","name",j)))
+                                       {
+                                               bool goodchan = false;
+                                               for (int k =0; k < Conf->Enumerate("goodchan"); k++)
+                                               {
+                                                       if (match(redirect, Conf->ReadValue("goodchan","name",k)))
+                                                               goodchan = true;
+                                               }
+       
+                                               if (!goodchan)
+                                               {
+                                                       /* <badchan:redirect> is a badchan */
+                                                       if (user)
+                                                               user->WriteServ("NOTICE %s :Badchan %s redirects to badchan %s", user->nick, name.c_str(), redirect.c_str());
+                                                       throw ModuleException("Badchan redirect loop");
+                                               }
+                                       }
+                               }
+                       }
+               }
        }
 
        virtual ~ModuleDenyChannels()
        {
-               DELETE(Conf);
+               delete Conf;
        }
        
        virtual Version GetVersion()
        {
-               return Version(1,1,0,1,VF_VENDOR,API_VERSION);
+               return Version(1,2,0,1,VF_VENDOR,API_VERSION);
        }
 
-       void Implements(char* List)
-       {
-               List[I_OnUserPreJoin] = List[I_OnRehash] = 1;
-       }
 
-       virtual int OnUserPreJoin(User* user, Channel* chan, const char* cname, std::string &privs)
+       virtual int OnUserPreJoin(User* user, Channel* chan, const char* cname, std::string &privs, const std::string &keygiven)
        {
                for (int j =0; j < Conf->Enumerate("badchan"); j++)
                {
-                       if (match(cname, Conf->ReadValue("badchan","name",j).c_str()))
+                       if (match(cname, Conf->ReadValue("badchan","name",j)))
                        {
                                if (IS_OPER(user) && Conf->ReadFlag("badchan","allowopers",j))
                                {
@@ -66,7 +100,29 @@ class ModuleDenyChannels : public Module
                                else
                                {
                                        std::string reason = Conf->ReadValue("badchan","reason",j);
-                                       user->WriteServ("926 %s %s :Channel %s is forbidden: %s",user->nick,cname,cname,reason.c_str());
+                                       std::string redirect = Conf->ReadValue("badchan","redirect",j);
+
+                                       for (int i = 0; i < Conf->Enumerate("goodchan"); i++)
+                                       {
+                                               if (match(cname, Conf->ReadValue("goodchan", "name", i)))
+                                               {
+                                                       return 0;
+                                               }
+                                       }
+                                       
+                                       if (ServerInstance->IsChannel(redirect.c_str()))
+                                       {
+                                               /* simple way to avoid potential loops: don't redirect to +L channels */
+                                               Channel *newchan = ServerInstance->FindChan(redirect);
+                                               if ((!newchan) || (!(newchan->IsModeSet('L'))))
+                                               {
+                                                       user->WriteNumeric(926, "%s %s :Channel %s is forbidden, redirecting to %s: %s",user->nick,cname,cname,redirect.c_str(), reason.c_str());
+                                                       Channel::JoinUser(ServerInstance,user,redirect.c_str(),false,"",false,ServerInstance->Time());
+                                                       return 1;
+                                               }
+                                       }
+
+                                       user->WriteNumeric(926, "%s %s :Channel %s is forbidden: %s",user->nick,cname,cname,reason.c_str());
                                        return 1;
                                }
                        }