summaryrefslogtreecommitdiff
path: root/src/commands/cmd_nick.cpp
diff options
context:
space:
mode:
authordanieldg <danieldg@e03df62e-2008-0410-955e-edbf42e46eb7>2010-01-30 18:40:51 +0000
committerdanieldg <danieldg@e03df62e-2008-0410-955e-edbf42e46eb7>2010-01-30 18:40:51 +0000
commitab8305ad99d7bb7d2cfe4154b2deb007546d5b70 (patch)
tree69459f1050bcbb39111ec4afa5f269656c21cc95 /src/commands/cmd_nick.cpp
parent845135ba55d2bad81588a80546c8abe2b76ed47c (diff)
Move nick change code out of cmd_nick; fixes double-invocation of hooks and may eliminate bug #911
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@12330 e03df62e-2008-0410-955e-edbf42e46eb7
Diffstat (limited to 'src/commands/cmd_nick.cpp')
-rw-r--r--src/commands/cmd_nick.cpp157
1 files changed, 16 insertions, 141 deletions
diff --git a/src/commands/cmd_nick.cpp b/src/commands/cmd_nick.cpp
index 397aea2b7..a69f1ddac 100644
--- a/src/commands/cmd_nick.cpp
+++ b/src/commands/cmd_nick.cpp
@@ -24,7 +24,7 @@ class CommandNick : public Command
public:
/** Constructor for nick.
*/
- CommandNick ( Module* parent) : Command(parent,"NICK", 1, 1) { works_before_reg = true; syntax = "<newnick>"; Penalty = 3; }
+ CommandNick ( Module* parent) : Command(parent,"NICK", 1, 1) { works_before_reg = true; syntax = "<newnick>"; Penalty = 0; }
/** Handle command.
* @param parameters The parameters to the comamnd
* @param pcnt The number of parameters passed to teh command
@@ -41,156 +41,39 @@ class CommandNick : public Command
*/
CmdResult CommandNick::Handle (const std::vector<std::string>& parameters, User *user)
{
- std::string oldnick;
+ std::string oldnick = user->nick;
+ std::string newnick = parameters[0];
- if (parameters[0].empty())
- {
- /* We cant put blanks in the parameters, so for this (extremely rare) issue we just put '*' here. */
- user->WriteNumeric(432, "%s * :Erroneous Nickname", user->nick.empty() ? user->nick.c_str() : "*");
- return CMD_FAILURE;
- }
+ // anything except the initial NICK gets a flood penalty
+ if (user->registered == REG_ALL && IS_LOCAL(user))
+ IS_LOCAL(user)->CommandFloodPenalty += 4000;
- if (((!ServerInstance->IsNick(parameters[0].c_str(), ServerInstance->Config->Limits.NickMax))) && (IS_LOCAL(user)))
+ if (newnick.empty())
{
- if (!ServerInstance->NICKForced.get(user))
- {
- if (parameters[0] == "0")
- {
- // Special case, Fake a /nick UIDHERE. Useful for evading "ERR: NICK IN USE" on connect etc.
- std::vector<std::string> p2;
- p2.push_back(user->uuid);
- ServerInstance->NICKForced.set(user, 1);
- this->Handle(p2, user);
- ServerInstance->NICKForced.set(user, 0);
- return CMD_SUCCESS;
- }
-
- user->WriteNumeric(432, "%s %s :Erroneous Nickname", user->nick.c_str(),parameters[0].c_str());
- return CMD_FAILURE;
- }
+ user->WriteNumeric(432, "%s * :Erroneous Nickname", oldnick.c_str());
+ return CMD_FAILURE;
}
- if (assign(user->nick) == parameters[0])
+ if (newnick == "0")
{
- /* If its exactly the same, even case, dont do anything. */
- if (parameters[0] == user->nick)
- {
- return CMD_SUCCESS;
- }
-
- /* 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.
- */
- oldnick.assign(user->nick, 0, IS_LOCAL(user) ? ServerInstance->Config->Limits.NickMax : MAXBUF);
- ModResult MOD_RESULT;
- FIRST_MOD_RESULT(OnUserPreNick, MOD_RESULT, (user,parameters[0]));
- if (MOD_RESULT == MOD_RES_DENY)
- return CMD_FAILURE;
- if (user->registered == REG_ALL)
- user->WriteCommon("NICK %s",parameters[0].c_str());
- user->nick.assign(parameters[0], 0, IS_LOCAL(user) ? ServerInstance->Config->Limits.NickMax : MAXBUF);
- user->InvalidateCache();
- FOREACH_MOD(I_OnUserPostNick,OnUserPostNick(user,oldnick));
- return CMD_SUCCESS;
+ newnick = user->uuid;
}
- else
+ else if (!ServerInstance->IsNick(newnick.c_str(), ServerInstance->Config->Limits.NickMax))
{
- /*
- * Don't check Q:Lines if it's a server-enforced change, just on the off-chance some fucking *moron*
- * tries to Q:Line SIDs, also, this means we just get our way period, as it really should be.
- * Thanks Kein for finding this. -- w00t
- *
- * Also don't check Q:Lines for remote nickchanges, they should have our Q:Lines anyway to enforce themselves.
- * -- w00t
- */
- if (!IS_LOCAL(user))
- {
- XLine* mq = ServerInstance->XLines->MatchesLine("Q",parameters[0]);
- if (mq)
- {
- if (user->registered == REG_ALL)
- {
- ServerInstance->SNO->WriteGlobalSno('a', "Q-Lined nickname %s from %s!%s@%s: %s",
- parameters[0].c_str(), user->nick.c_str(), user->ident.c_str(), user->host.c_str(), mq->reason.c_str());
- }
- user->WriteNumeric(432, "%s %s :Invalid nickname: %s",user->nick.c_str(), parameters[0].c_str(), mq->reason.c_str());
- return CMD_FAILURE;
- }
-
- if (ServerInstance->Config->RestrictBannedUsers)
- {
- for (UCListIter i = user->chans.begin(); i != user->chans.end(); i++)
- {
- Channel *chan = *i;
- if (chan->GetPrefixValue(user) < VOICE_VALUE && chan->IsBanned(user))
- {
- user->WriteNumeric(404, "%s %s :Cannot send to channel (you're banned)", user->nick.c_str(), chan->name.c_str());
- return CMD_FAILURE;
- }
- }
- }
- }
-
- /*
- * Uh oh.. if the nickname is in use, and it's not in use by the person using it (doh) --
- * then we have a potential collide. Check whether someone else is camping on the nick
- * (i.e. connect -> send NICK, don't send USER.) If they are camping, force-change the
- * camper to their UID, and allow the incoming nick change.
- *
- * If the guy using the nick is already using it, tell the incoming nick change to gtfo,
- * because the nick is already (rightfully) in use. -- w00t
- */
- User* InUse = ServerInstance->FindNickOnly(parameters[0]);
- if (InUse && (InUse != user))
- {
- if (InUse->registered != REG_ALL)
- {
- /* force the camper to their UUID, and ask them to re-send a NICK. */
- InUse->WriteTo(InUse, "NICK %s", InUse->uuid.c_str());
- InUse->WriteNumeric(433, "%s %s :Nickname overruled.", InUse->nick.c_str(), InUse->nick.c_str());
- InUse->UpdateNickHash(InUse->uuid.c_str());
- InUse->nick.assign(InUse->uuid, 0, IS_LOCAL(InUse) ? ServerInstance->Config->Limits.NickMax : MAXBUF);
- InUse->InvalidateCache();
- InUse->registered &= ~REG_NICK;
- }
- else
- {
- /* No camping, tell the incoming user to stop trying to change nick ;p */
- user->WriteNumeric(433, "%s %s :Nickname is already in use.", user->registered >= REG_NICK ? user->nick.c_str() : "*", parameters[0].c_str());
- return CMD_FAILURE;
- }
- }
- }
-
-
- ModResult MOD_RESULT;
- FIRST_MOD_RESULT(OnUserPreNick, MOD_RESULT, (user, parameters[0]));
- if (MOD_RESULT == MOD_RES_DENY)
- // if a module returns true, the nick change is silently forbidden.
+ user->WriteNumeric(432, "%s %s :Erroneous Nickname", user->nick.c_str(),newnick.c_str());
return CMD_FAILURE;
+ }
- if (user->registered == REG_ALL)
- user->WriteCommon("NICK %s", parameters[0].c_str());
-
- oldnick.assign(user->nick, 0, IS_LOCAL(user) ? ServerInstance->Config->Limits.NickMax : MAXBUF);
-
- /* change the nick of the user in the users_hash */
- user = user->UpdateNickHash(parameters[0].c_str());
-
- /* actually change the nick within the record */
- if (!user)
+ if (!user->ChangeNick(newnick, false))
return CMD_FAILURE;
- user->nick.assign(parameters[0], 0, IS_LOCAL(user) ? ServerInstance->Config->Limits.NickMax : MAXBUF);
- user->InvalidateCache();
-
if (user->registered < REG_NICKUSER)
{
user->registered = (user->registered | REG_NICK);
if (user->registered == REG_NICKUSER)
{
/* user is registered now, bit 0 = USER command, bit 1 = sent a NICK command */
+ ModResult MOD_RESULT;
FIRST_MOD_RESULT(OnUserRegister, MOD_RESULT, (IS_LOCAL(user)));
if (MOD_RESULT == MOD_RES_DENY)
return CMD_FAILURE;
@@ -200,15 +83,7 @@ CmdResult CommandNick::Handle (const std::vector<std::string>& parameters, User
}
}
- if (user->registered == REG_ALL)
- {
- if (IS_LOCAL(user))
- IS_LOCAL(user)->CommandFloodPenalty += 5000;
- FOREACH_MOD(I_OnUserPostNick,OnUserPostNick(user, oldnick));
- }
-
return CMD_SUCCESS;
-
}