]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/commands/cmd_kill.cpp
Replace hardcoded mode letters, part 3
[user/henk/code/inspircd.git] / src / commands / cmd_kill.cpp
index 3490199c8b800ffb60d1797c84aab334fcd196dc..dddfe429120b42c1374ea79f89290a1a058414bf 100644 (file)
@@ -1,16 +1,24 @@
-/*       +------------------------------------+
- *       | Inspire Internet Relay Chat Daemon |
- *       +------------------------------------+
+/*
+ * InspIRCd -- Internet Relay Chat Daemon
  *
- *  InspIRCd: (C) 2002-2010 InspIRCd Development Team
- * See: http://wiki.inspircd.org/Credits
+ *   Copyright (C) 2009 Daniel De Graaf <danieldg@inspircd.org>
+ *   Copyright (C) 2008 Craig Edwards <craigedwards@brainbox.cc>
+ *   Copyright (C) 2007 Robin Burchell <robin+git@viroteck.net>
  *
- * This program is free but copyrighted software; see
- *            the file COPYING for details.
+ * This file is part of InspIRCd.  InspIRCd is free software: you can
+ * redistribute it and/or modify it under the terms of the GNU General Public
+ * License as published by the Free Software Foundation, version 2.
  *
- * ---------------------------------------------------
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+
 #include "inspircd.h"
 
 /** Handle /KILL. These command handlers can be reloaded by the core,
  */
 class CommandKill : public Command
 {
+       std::string lastuuid;
+       std::string killreason;
+
  public:
        /** Constructor for kill.
         */
        CommandKill ( Module* parent) : Command(parent,"KILL",2,2) {
                flags_needed = 'o';
                syntax = "<nickname> <reason>";
-               TRANSLATE3(TR_NICK, TR_TEXT, TR_END);
+               TRANSLATE2(TR_CUSTOM, TR_CUSTOM);
        }
        /** Handle command.
         * @param parameters The parameters to the comamnd
@@ -37,11 +48,21 @@ class CommandKill : public Command
        CmdResult Handle(const std::vector<std::string>& parameters, User *user);
        RouteDescriptor GetRouting(User* user, const std::vector<std::string>& parameters)
        {
-               // local kills of remote users are routed via the OnRemoteKill hook
-               if (IS_LOCAL(user))
+               // FindNick() doesn't work here because we quit the target user in Handle() which
+               // removes it from the nicklist, so we check lastuuid: if it's empty then this KILL
+               // was for a local user, otherwise it contains the uuid of the user who was killed.
+               if (lastuuid.empty())
                        return ROUTE_LOCALONLY;
                return ROUTE_BROADCAST;
        }
+
+       void EncodeParameter(std::string& param, int index)
+       {
+               // Manually translate the nick -> uuid (see above), and also the reason (params[1])
+               // because we decorate it if the oper is local and want remote servers to see the
+               // decorated reason not the original.
+               param = ((index == 0) ? lastuuid : killreason);
+       }
 };
 
 /** Handle /KILL
@@ -49,14 +70,15 @@ class CommandKill : public Command
 CmdResult CommandKill::Handle (const std::vector<std::string>& parameters, User *user)
 {
        /* Allow comma seperated lists of users for /KILL (thanks w00t) */
-       if (ServerInstance->Parser->LoopCall(user, this, parameters, 0))
-               return CMD_SUCCESS;
+       if (CommandParser::LoopCall(user, this, parameters, 0))
+       {
+               // If we got a colon delimited list of nicks then the handler ran for each nick,
+               // and KILL commands were broadcast for remote targets.
+               return CMD_FAILURE;
+       }
 
        User *u = ServerInstance->FindNick(parameters[0]);
-       char killreason[MAXBUF];
-       ModResult MOD_RESULT;
-
-       if (u)
+       if ((u) && (!IS_SERVER(u)))
        {
                /*
                 * Here, we need to decide how to munge kill messages. Whether to hide killer, what to show opers, etc.
@@ -65,32 +87,37 @@ CmdResult CommandKill::Handle (const std::vector<std::string>& parameters, User
                 * This conditional is so that we only append the "Killed (" prefix ONCE. If killer is remote, then the kill
                 * just gets processed and passed on, otherwise, if they are local, it gets prefixed. Makes sense :-) -- w00t
                 */
+
                if (IS_LOCAL(user))
                {
                        /*
                         * Moved this event inside the IS_LOCAL check also, we don't want half the network killing a user
                         * and the other half not. This would be a bad thing. ;p -- w00t
                         */
+                       ModResult MOD_RESULT;
                        FIRST_MOD_RESULT(OnKill, MOD_RESULT, (user, u, parameters[1]));
 
                        if (MOD_RESULT == MOD_RES_DENY)
                                return CMD_FAILURE;
 
+                       killreason = "Killed (";
                        if (!ServerInstance->Config->HideKillsServer.empty())
                        {
                                // hidekills is on, use it
-                               snprintf(killreason, ServerInstance->Config->Limits.MaxQuit, "Killed (%s (%s))", ServerInstance->Config->HideKillsServer.c_str(), parameters[1].c_str());
+                               killreason += ServerInstance->Config->HideKillsServer;
                        }
                        else
                        {
                                // hidekills is off, do nothing
-                               snprintf(killreason, ServerInstance->Config->Limits.MaxQuit, "Killed (%s (%s))", user->nick.c_str(), parameters[1].c_str());
+                               killreason += user->nick;
                        }
+
+                       killreason += " (" + parameters[1] + "))";
                }
                else
                {
                        /* Leave it alone, remote server has already formatted it */
-                       strlcpy(killreason, parameters[1].c_str(), ServerInstance->Config->Limits.MaxQuit);
+                       killreason.assign(parameters[1], 0, ServerInstance->Config->Limits.MaxQuit);
                }
 
                /*
@@ -100,8 +127,8 @@ CmdResult CommandKill::Handle (const std::vector<std::string>& parameters, User
                if (!IS_LOCAL(u))
                {
                        // remote kill
-                       ServerInstance->SNO->WriteToSnoMask('K', "Remote kill by %s: %s!%s@%s (%s)", user->nick.c_str(), u->nick.c_str(), u->ident.c_str(), u->host.c_str(), parameters[1].c_str());
-                       FOREACH_MOD(I_OnRemoteKill, OnRemoteKill(user, u, killreason, killreason));
+                       ServerInstance->SNO->WriteToSnoMask('K', "Remote kill by %s: %s (%s)", user->nick.c_str(), u->GetFullRealHost().c_str(), parameters[1].c_str());
+                       this->lastuuid = u->uuid;
                }
                else
                {
@@ -110,8 +137,11 @@ CmdResult CommandKill::Handle (const std::vector<std::string>& parameters, User
                         * XXX - this isn't entirely correct, servers A - B - C, oper on A, client on C. Oper kills client, A and B will get remote kill
                         * snotices, C will get a local kill snotice. this isn't accurate, and needs fixing at some stage. -- w00t
                         */
-                       ServerInstance->SNO->WriteToSnoMask('k',"Local Kill by %s: %s!%s@%s (%s)", user->nick.c_str(), u->nick.c_str(), u->ident.c_str(), u->host.c_str(), parameters[1].c_str());
-                       ServerInstance->Logs->Log("KILL",DEFAULT,"LOCAL KILL: %s :%s!%s!%s (%s)", u->nick.c_str(), ServerInstance->Config->ServerName.c_str(), user->dhost.c_str(), user->nick.c_str(), parameters[1].c_str());
+                       if (IS_LOCAL(user))
+                               ServerInstance->SNO->WriteGlobalSno('k',"Local Kill by %s: %s (%s)", user->nick.c_str(), u->GetFullRealHost().c_str(), parameters[1].c_str());
+                       else
+                               ServerInstance->SNO->WriteToSnoMask('k',"Local Kill by %s: %s (%s)", user->nick.c_str(), u->GetFullRealHost().c_str(), parameters[1].c_str());
+                       ServerInstance->Logs->Log("KILL", LOG_DEFAULT, "LOCAL KILL: %s :%s!%s!%s (%s)", u->nick.c_str(), ServerInstance->Config->ServerName.c_str(), user->dhost.c_str(), user->nick.c_str(), parameters[1].c_str());
                        /* Bug #419, make sure this message can only occur once even in the case of multiple KILL messages crossing the network, and change to show
                         * hidekillsserver as source if possible
                         */
@@ -124,6 +154,8 @@ CmdResult CommandKill::Handle (const std::vector<std::string>& parameters, User
                                                ServerInstance->Config->HideKillsServer.empty() ? user->nick.c_str() : ServerInstance->Config->HideKillsServer.c_str(),
                                                parameters[1].c_str());
                        }
+
+                       this->lastuuid.clear();
                }
 
                // send the quit out