X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fcmd_nick.cpp;h=518d600f19c956b2b0fe6f2c91c252f4559311cf;hb=59b1a8955142935b02af6446005ab47fc7c3fc8c;hp=83bb96859f39e15f054544ec2bf27830db321250;hpb=293df6a8b55e89c127e60e92711ef0ef1027bff8;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/cmd_nick.cpp b/src/cmd_nick.cpp
index 83bb96859..518d600f1 100644
--- a/src/cmd_nick.cpp
+++ b/src/cmd_nick.cpp
@@ -2,10 +2,10 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  Inspire is copyright (C) 2002-2005 ChatSpike-Dev.
+ *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
  *                       E-mail:
- *                <brain.net>
- *                <Craig.net>
+ *                <brain@chatspike.net>
+ *                <Craig@chatspike.net>
  *
  * Written by Craig Edwards, Craig McLure, and others.
  * This program is free but copyrighted software; see
@@ -42,6 +42,7 @@ using namespace std;
 #include "xline.h"
 #include "inspstring.h"
 #include "dnsqueue.h"
+#include "dns.h"
 #include "helperfuncs.h"
 #include "hashcomp.h"
 #include "socketengine.h"
@@ -60,10 +61,12 @@ extern chan_hash chanlist;
 extern whowas_hash whowas;
 extern std::vector<userrec*> all_opers;
 extern std::vector<userrec*> local_users;
-extern userrec* fd_ref_table[65536];
+extern userrec* fd_ref_table[MAX_DESCRIPTORS];
 
 void cmd_nick::Handle (char **parameters, int pcnt, userrec *user)
 {
+	char oldnick[NICKMAX];
+
 	if (pcnt < 1) 
 	{
 		log(DEBUG,"not enough params for handle_nick");
@@ -89,9 +92,25 @@ void cmd_nick::Handle (char **parameters, int pcnt, userrec *user)
 		log(DEBUG,"invalid old nick passed to handle_nick");
 		return;
 	}
-	if (!strcasecmp(user->nick,parameters[0]))
+	if (irc::string(user->nick) == irc::string(parameters[0]))
 	{
-		log(DEBUG,"old nick is new nick, skipping");
+		/* If its exactly the same, even case, dont do anything. */
+		if (!strcmp(user->nick,parameters[0]))
+			return;
+		/* Its a change of case. People insisted that they should be
+		 * able to do silly things like this even though the RFC says
+		 * the nick AAA is the same as the nick aaa.
+		 */
+		log(DEBUG,"old nick is new nick, not updating hash (case change only)");
+		strlcpy(oldnick, user->nick, NICKMAX - 1);
+		int MOD_RESULT = 0;
+		FOREACH_RESULT(I_OnUserPreNick,OnUserPreNick(user,parameters[0]));
+		if (MOD_RESULT)
+			return;
+		if (user->registered == 7)
+			WriteCommon(user,"NICK %s",parameters[0]);
+		strlcpy(user->nick, parameters[0], NICKMAX - 1);
+		FOREACH_MOD(I_OnUserPostNick,OnUserPostNick(user,oldnick));
 		return;
 	}
 	else
@@ -124,7 +143,7 @@ void cmd_nick::Handle (char **parameters, int pcnt, userrec *user)
 	if (user->registered == 7)
 	{
 		int MOD_RESULT = 0;
-		FOREACH_RESULT(OnUserPreNick(user,parameters[0]));
+		FOREACH_RESULT(I_OnUserPreNick,OnUserPreNick(user,parameters[0]));
 		if (MOD_RESULT) {
 			// if a module returns true, the nick change is silently forbidden.
 			return;
@@ -133,9 +152,8 @@ void cmd_nick::Handle (char **parameters, int pcnt, userrec *user)
 		WriteCommon(user,"NICK %s",parameters[0]);
 		
 	}
-	
-	char oldnick[NICKMAX];
-	strlcpy(oldnick,user->nick,NICKMAX);
+
+	strlcpy(oldnick, user->nick, NICKMAX - 1);
 
 	/* change the nick of the user in the users_hash */
 	user = ReHashNick(user->nick, parameters[0]);
@@ -143,7 +161,7 @@ void cmd_nick::Handle (char **parameters, int pcnt, userrec *user)
 	if (!user) return;
 	if (!user->nick) return;
 
-	strlcpy(user->nick, parameters[0],NICKMAX);
+	strlcpy(user->nick, parameters[0], NICKMAX - 1);
 
 	log(DEBUG,"new nick set: %s",user->nick);
 	
@@ -172,12 +190,12 @@ void cmd_nick::Handle (char **parameters, int pcnt, userrec *user)
 	if (user->registered == 3)
 	{
 		/* user is registered now, bit 0 = USER command, bit 1 = sent a NICK command */
-		FOREACH_MOD OnUserRegister(user);
-		ConnectUser(user);
+		FOREACH_MOD(I_OnUserRegister,OnUserRegister(user));
+		//ConnectUser(user,NULL);
 	}
 	if (user->registered == 7)
 	{
-		FOREACH_MOD OnUserPostNick(user,oldnick);
+		FOREACH_MOD(I_OnUserPostNick,OnUserPostNick(user,oldnick));
 	}
 }