X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_denychans.cpp;h=cab26122d94b286caa72160b114993aec6560cbe;hb=3039e0663f7e45a45e9574a6aa7463bbe77e849f;hp=0ce9c3d752525c2cc13d76185dafd8368dea4162;hpb=8f7870b8131824f4b4587a8e979c1c0ddf70d379;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_denychans.cpp b/src/modules/m_denychans.cpp
index 0ce9c3d75..cab26122d 100644
--- 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
@@ -12,10 +12,6 @@
  */
 
 #include "inspircd.h"
-#include "users.h"
-#include "channels.h"
-#include "modules.h"
-#include "hashcomp.h"
 #include "wildcard.h"
 
 /* $ModDesc: Implements config tags which allow blocking of joins to channels */
@@ -32,30 +28,66 @@ class ModuleDenyChannels : public Module
 	{
 		
 		Conf = new ConfigReader(ServerInstance);
+		Implementation eventlist[] = { I_OnUserPreJoin, I_OnRehash };
+		ServerInstance->Modules->Attach(eventlist, this, 2);
 	}
 	
-	virtual void OnRehash(userrec* user, const std::string &param)
+	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.c_str(), Conf->ReadValue("badchan","name",j).c_str()))
+					{
+						bool goodchan = false;
+						for (int k =0; k < Conf->Enumerate("goodchan"); k++)
+						{
+							if (match(redirect.c_str(), Conf->ReadValue("goodchan","name",k).c_str()))
+								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(userrec* user, chanrec* 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++)
 		{
@@ -68,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).c_str()))
+						{
+							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;
 				}
 			}
@@ -77,4 +131,4 @@ class ModuleDenyChannels : public Module
 	}
 };
 
-MODULE_INIT(ModuleDenyChannels);
+MODULE_INIT(ModuleDenyChannels)