X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_nicklock.cpp;h=d475d59e0eeeaa136fa4f1b5f1008ed368f44278;hb=00fa6d592ed2640fcdf74444786de555c1c3da25;hp=caa71b53f6f03a64551fe8bb8e1d03dc1a216243;hpb=d185decae97752368d5cf62311cbc0d1a52aa22c;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_nicklock.cpp b/src/modules/m_nicklock.cpp
index caa71b53f..d475d59e0 100644
--- a/src/modules/m_nicklock.cpp
+++ b/src/modules/m_nicklock.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
@@ -19,7 +19,7 @@
  */
 class CommandNicklock : public Command
 {
-	char* dummy;
+
  public:
 	CommandNicklock (InspIRCd* Instance) : Command(Instance,"NICKLOCK", "o", 2)
 	{
@@ -31,45 +31,59 @@ class CommandNicklock : public Command
 	CmdResult Handle(const std::vector<std::string>& parameters, User *user)
 	{
 		User* target = ServerInstance->FindNick(parameters[0]);
-		irc::string server;
-		irc::string me;
 
-		// check user exists
-		if (!target)
+		/* Do local sanity checks and bails */
+		if (IS_LOCAL(user))
 		{
-			return CMD_FAILURE;
-		}
+			if (target && ServerInstance->ULine(target->server))
+			{
+				user->WriteNumeric(ERR_NOPRIVILEGES, "%s :Cannot use an NICKLOCK command on a u-lined client",user->nick.c_str());
+				return CMD_FAILURE;
+			}
 
-		// check if user is locked
-		if (target->GetExt("nick_locked", dummy))
-		{
-			user->WriteNumeric(946, "%s %s :This user's nickname is already locked.",user->nick.c_str(),target->nick.c_str());
-			return CMD_FAILURE;
-		}
+			if (!target)
+			{
+				user->WriteServ("NOTICE %s :*** No such nickname: '%s'", user->nick.c_str(), parameters[0].c_str());
+				return CMD_FAILURE;
+			}
 
-		// check nick is valid
-		if (IS_LOCAL(user) && !ServerInstance->IsNick(parameters[1].c_str(), ServerInstance->Config->Limits.NickMax))
-		{
-			return CMD_FAILURE;
-		}
+			if (target->GetExt("nick_locked"))
+			{
+				user->WriteNumeric(946, "%s %s :This user's nickname is already locked.",user->nick.c_str(),target->nick.c_str());
+				return CMD_FAILURE;
+			}
 
-		// let others know
-		ServerInstance->SNO->WriteToSnoMask('A', std::string(user->nick)+" used NICKLOCK to change and hold "+parameters[0]+" to "+parameters[1]);
+			if (!ServerInstance->IsNick(parameters[1].c_str(), ServerInstance->Config->Limits.NickMax))
+			{
+				user->WriteServ("NOTICE %s :*** Invalid nickname '%s'", user->nick.c_str(), parameters[1].c_str());
+				return CMD_FAILURE;
+			}
+		}
 
-		if (!target->ForceNickChange(parameters[1].c_str()))
+		/* If we made it this far, extend the user */
+		if (target)
 		{
-			// ugh, nickchange failed for some reason -- possibly existing nick?
-			if (!target->ForceNickChange(target->uuid.c_str()))
+			target->Extend("nick_locked", "ON");
+			ServerInstance->SNO->WriteToSnoMask('A', user->nick+" used NICKLOCK to change and hold "+target->nick+" to "+parameters[1]);
+
+			/* Only send out nick from local server */
+			if (IS_LOCAL(target))
 			{
-				// Well shit, we cant even change them to their UID (this should not happen!)
-				ServerInstance->Users->QuitUser(target, "Nickname collision");
+				std::string oldnick = user->nick;
+				std::string newnick = target->nick;
+				if (!target->ForceNickChange(parameters[1].c_str()))
+				{
+					/* XXX: We failed, this *should* not happen but if it does
+					 * tell everybody. Note user is still nick locked on their old
+					 * nick instead.
+					 */
+					ServerInstance->SNO->WriteToSnoMask('A', oldnick+" failed nickchange on NICKLOCK (from "+newnick+" to "+parameters[1]+") Locked to "+newnick+" instead");
+					ServerInstance->PI->SendSNONotice("A", oldnick+" failed nickchange on NICKLOCK (from "+newnick+" to "+parameters[1]+") Locked to "+newnick+" instead");
+				}
 			}
 		}
 
-		// give them a lock flag
-		target->Extend("nick_locked", "ON");
-
-		/* route */
+		/* Route it */
 		return CMD_SUCCESS;
 	}
 };
@@ -88,15 +102,41 @@ class CommandNickunlock : public Command
 	CmdResult Handle (const std::vector<std::string>& parameters, User *user)
 	{
 		User* target = ServerInstance->FindNick(parameters[0]);
+
+		/* Do local sanity checks and bails */
+		if (IS_LOCAL(user))
+		{
+			if (target && ServerInstance->ULine(target->server))
+			{
+				user->WriteNumeric(ERR_NOPRIVILEGES, "%s :Cannot use an NICKUNLOCK command on a u-lined client",user->nick.c_str());
+				return CMD_FAILURE;
+			}
+
+			if (!target)
+			{
+				user->WriteServ("NOTICE %s :*** No such nickname: '%s'", user->nick.c_str(), parameters[0].c_str());
+				return CMD_FAILURE;
+			}
+
+			if (!target->GetExt("nick_locked"))
+			{
+				user->WriteNumeric(946, "%s %s :This user's nickname is not locked.",user->nick.c_str(),target->nick.c_str());
+				return CMD_FAILURE;
+			}
+		}
+
+		/* If we made it this far, the command is going out on the wire so send local snotice */
+		ServerInstance->SNO->WriteToSnoMask('A', std::string(user->nick)+" used NICKUNLOCK on "+parameters[0]);
+
 		if (target)
 		{
 			target->Shrink("nick_locked");
-			user->WriteNumeric(945, "%s %s :Nickname now unlocked.",user->nick.c_str(),target->nick.c_str());
-			ServerInstance->SNO->WriteToSnoMask('A', std::string(user->nick)+" used NICKUNLOCK on "+parameters[0]);
-			return CMD_SUCCESS;
+			if (IS_LOCAL(user))
+				user->WriteNumeric(945, "%s %s :Nickname now unlocked.",user->nick.c_str(),target->nick.c_str());
 		}
 
-		return CMD_FAILURE;
+		/* Route it */
+		return CMD_SUCCESS;
 	}
 };
 
@@ -125,13 +165,19 @@ class ModuleNickLock : public Module
 
 	virtual Version GetVersion()
 	{
-		return Version(1, 2, 0, 1, VF_COMMON | VF_VENDOR, API_VERSION);
+		return Version("$Id$", VF_COMMON | VF_VENDOR, API_VERSION);
 	}
 
 
 	virtual int OnUserPreNick(User* user, const std::string &newnick)
 	{
-		if (isdigit(newnick[0])) /* allow a switch to a UID */
+		if (!IS_LOCAL(user))
+			return 0;
+
+		if (isdigit(newnick[0])) /* Allow a switch to a UID */
+			return 0;
+
+		if (user->GetExt("NICKForced")) /* Allow forced nick changes */
 			return 0;
 
 		if (user->GetExt("nick_locked", n))
@@ -147,6 +193,12 @@ class ModuleNickLock : public Module
 		user->Shrink("nick_locked");
 	}
 
+	void Prioritize()
+	{
+		Module *nflood = ServerInstance->Modules->Find("m_nickflood.so");
+		ServerInstance->Modules->SetPriority(this, I_OnUserPreJoin, PRIORITY_BEFORE, &nflood);
+	}
+
 	virtual void OnCleanup(int target_type, void* item)
 	{
 		if(target_type == TYPE_USER)