X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_cloaking.cpp;h=31f9eea5d9cc7fdc1d0cc38a8b22a098f6d5abfa;hb=92796ffe6430162a9cb70f1361aaa9ca64645c90;hp=f5b3d08da3a99f40f8643f486f99f8ced90ab4fb;hpb=517a26252a091eaee0149458a86fc43731bce618;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_cloaking.cpp b/src/modules/m_cloaking.cpp
index f5b3d08da..31f9eea5d 100644
--- a/src/modules/m_cloaking.cpp
+++ b/src/modules/m_cloaking.cpp
@@ -2,7 +2,7 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
+ *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
  * See: http://www.inspircd.org/wiki/index.php/Credits
  *
  * This program is free but copyrighted software; see
@@ -69,9 +69,6 @@ class CloakUser : public ModeHandler
 
 	ModeAction OnModeChange(User* source, User* dest, Channel* channel, std::string &parameter, bool adding, bool)
 	{
-		if (source != dest)
-			return MODEACTION_DENY;
-
 		/* For remote clients, we dont take any action, we just allow it.
 		 * The local server where they are will set their cloak instead.
 		 * This is fine, as we will recieve it later.
@@ -100,9 +97,13 @@ class CloakUser : public ModeHandler
 
 				std::string* cloak;
 
+				if (!dest->GetExt("cloaked_host", cloak))
+				{
+					/* Force creation of missing cloak */
+					Sender->OnUserConnect(dest);
+				}
 				if (dest->GetExt("cloaked_host", cloak))
 				{
-					/* Cloaked host has been set before on this user, don't bother to recalculate and waste cpu */
 					dest->ChangeDisplayedHost(cloak->c_str());
 					dest->SetMode('x',true);
 					return MODEACTION_ALLOW;
@@ -159,11 +160,6 @@ class CloakUser : public ModeHandler
 
 	std::string Cloak6(const char* ip)
 	{
-		/* Theyre using 4in6 (YUCK). Translate as ipv4 cloak */
-		if (!strncmp(ip, "0::ffff:", 8))
-			return Cloak4(ip + 8);
-
-		/* If we get here, yes it really is an ipv6 ip */
 		unsigned int iv[] = { key1, key2, key3, key4 };
 		std::vector<std::string> hashies;
 		std::string item;
@@ -288,8 +284,10 @@ class ModuleCloaking : public Module
 
 		ServerInstance->Modules->UseInterface("HashRequest");
 
-		Implementation eventlist[] = { I_OnRehash, I_OnUserDisconnect, I_OnCleanup, I_OnCheckBan, I_OnUserConnect, I_OnSyncUserMetaData };
+		Implementation eventlist[] = { I_OnRehash, I_OnUserDisconnect, I_OnCleanup, I_OnCheckBan, I_OnUserConnect, I_OnSyncUserMetaData, I_OnCleanup };
 		ServerInstance->Modules->Attach(eventlist, this, 6);
+
+		CloakExistingUsers();
 	}
 
 	void OnSyncUserMetaData(User* user, Module* proto,void* opaque, const std::string &extname, bool displayable)
@@ -302,6 +300,17 @@ class ModuleCloaking : public Module
 		}
 	}
 
+	void CloakExistingUsers()
+	{
+		std::string* cloak;
+		for (std::vector<User*>::iterator u = ServerInstance->Users->local_users.begin(); u != ServerInstance->Users->local_users.end(); u++)
+		{
+			if (!(*u)->GetExt("cloaked_host", cloak))
+			{
+				OnUserConnect(*u);
+			}
+		}
+	}
 
 	virtual int OnCheckBan(User* user, Channel* chan)
 	{
@@ -324,13 +333,20 @@ class ModuleCloaking : public Module
 	{
 		/* Needs to be after m_banexception etc. */
 		ServerInstance->Modules->SetPriority(this, I_OnCheckBan, PRIO_LAST);
+
+		/* but before m_conn_umodes, so host is generated ready to apply */
+		Module *um = ServerInstance->Modules->Find("m_conn_umodes.so");
+		ServerInstance->Modules->SetPriority(this, I_OnUserConnect, PRIO_AFTER, &um);
 	}
 
 	virtual void OnUserDisconnect(User* user)
 	{
 		std::string* tofree;
 		if (user->GetExt("cloaked_host", tofree))
+		{
 			delete tofree;
+			user->Shrink("cloaked_host");
+		}
 	}
 
 	virtual void OnCleanup(int target_type, void* item)
@@ -350,7 +366,7 @@ class ModuleCloaking : public Module
 	{
 		// returns the version number of the module to be
 		// listed in /MODULES
-		return Version(1,2,0,2,VF_COMMON|VF_VENDOR,API_VERSION);
+		return Version("$Id$", VF_COMMON|VF_VENDOR,API_VERSION);
 	}
 
 	virtual void OnRehash(User* user, const std::string &parameter)
@@ -360,6 +376,10 @@ class ModuleCloaking : public Module
 
 	virtual void OnUserConnect(User* dest)
 	{
+		std::string* tofree;
+		if (dest->GetExt("cloaked_host", tofree))
+			return;
+
 		if (dest->host.find('.') != std::string::npos || dest->host.find(':') != std::string::npos)
 		{
 			unsigned int iv[] = { cu->key1, cu->key2, cu->key3, cu->key4 };