]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/commands/cmd_kill.cpp
m_check Fix non-standard format specifier %s passed to strftime() causing problems...
[user/henk/code/inspircd.git] / src / commands / cmd_kill.cpp
index 6932c8ed6debb3eb65558c10ef2a14e467c1d4a8..17c8a76a053479968f229916dd216ce69c9ab138 100644 (file)
@@ -1,37 +1,67 @@
-/*       +------------------------------------+
- *       | Inspire Internet Relay Chat Daemon |
- *       +------------------------------------+
+/*
+ * InspIRCd -- Internet Relay Chat Daemon
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/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"
-#include "commands/cmd_kill.h"
 
-extern "C" DllExport Command* init_command(InspIRCd* Instance)
+/** Handle /KILL. These command handlers can be reloaded by the core,
+ * and handle basic RFC1459 commands. Commands within modules work
+ * the same way, however, they can be fully unloaded, where these
+ * may not.
+ */
+class CommandKill : public Command
 {
-       return new CommandKill(Instance);
-}
+ 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);
+       }
+       /** Handle command.
+        * @param parameters The parameters to the comamnd
+        * @param pcnt The number of parameters passed to teh command
+        * @param user The user issuing the command
+        * @return A value from CmdResult to indicate command success or failure.
+        */
+       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))
+                       return ROUTE_LOCALONLY;
+               return ROUTE_BROADCAST;
+       }
+};
 
 /** Handle /KILL
  */
-CmdResult CommandKill::Handle (const char* const* parameters, int pcnt, User *user)
+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, pcnt, 0))
+       if (ServerInstance->Parser->LoopCall(user, this, parameters, 0))
                return CMD_SUCCESS;
 
        User *u = ServerInstance->FindNick(parameters[0]);
-       char killreason[MAXBUF];
-       int MOD_RESULT = 0;
-
-       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.
@@ -40,32 +70,38 @@ CmdResult CommandKill::Handle (const char* const* parameters, int pcnt, User *us
                 * 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
                 */
+
+               std::string killreason;
                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
                         */
-                       FOREACH_RESULT(I_OnKill, OnKill(user, u, parameters[1]));
+                       ModResult MOD_RESULT;
+                       FIRST_MOD_RESULT(OnKill, MOD_RESULT, (user, u, parameters[1]));
 
-                       if (MOD_RESULT)
+                       if (MOD_RESULT == MOD_RES_DENY)
                                return CMD_FAILURE;
 
-                       if (*ServerInstance->Config->HideKillsServer)
+                       killreason = "Killed (";
+                       if (!ServerInstance->Config->HideKillsServer.empty())
                        {
                                // hidekills is on, use it
-                               snprintf(killreason, MAXQUIT, "Killed (%s (%s))", ServerInstance->Config->HideKillsServer, parameters[1]);
+                               killreason += ServerInstance->Config->HideKillsServer;
                        }
                        else
                        {
                                // hidekills is off, do nothing
-                               snprintf(killreason, MAXQUIT, "Killed (%s (%s))", user->nick, parameters[1]);
+                               killreason += user->nick;
                        }
+
+                       killreason += " (" + parameters[1] + "))";
                }
                else
                {
                        /* Leave it alone, remote server has already formatted it */
-                       snprintf(killreason, MAXQUIT, "%s", parameters[1]);
+                       killreason.assign(parameters[1], 0, ServerInstance->Config->Limits.MaxQuit);
                }
 
                /*
@@ -75,7 +111,7 @@ CmdResult CommandKill::Handle (const char* const* parameters, int pcnt, User *us
                if (!IS_LOCAL(u))
                {
                        // remote kill
-                       ServerInstance->SNO->WriteToSnoMask('K', "Remote kill by %s: %s!%s@%s (%s)", user->nick, u->nick, u->ident, u->host, parameters[1]);
+                       ServerInstance->SNO->WriteToSnoMask('K', "Remote kill by %s: %s (%s)", user->nick.c_str(), u->GetFullRealHost().c_str(), parameters[1].c_str());
                        FOREACH_MOD(I_OnRemoteKill, OnRemoteKill(user, u, killreason, killreason));
                }
                else
@@ -85,19 +121,22 @@ CmdResult CommandKill::Handle (const char* const* parameters, int pcnt, User *us
                         * 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, u->nick, u->ident, u->host, parameters[1]);
-                       ServerInstance->Logs->Log("KILL",DEFAULT,"LOCAL KILL: %s :%s!%s!%s (%s)", u->nick, ServerInstance->Config->ServerName, user->dhost, user->nick, parameters[1]);
+                       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",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
                         */
                        if (!u->quitting)
                        {
-                               u->Write(":%s KILL %s :%s!%s!%s (%s)", *ServerInstance->Config->HideKillsServer ? ServerInstance->Config->HideKillsServer : user->GetFullHost(),
-                                               u->nick,
-                                               ServerInstance->Config->ServerName,
-                                               user->dhost,
-                                               *ServerInstance->Config->HideKillsServer ? ServerInstance->Config->HideKillsServer : user->nick,
-                                               parameters[1]);
+                               u->Write(":%s KILL %s :%s!%s!%s (%s)", ServerInstance->Config->HideKillsServer.empty() ? user->GetFullHost().c_str() : ServerInstance->Config->HideKillsServer.c_str(),
+                                               u->nick.c_str(),
+                                               ServerInstance->Config->ServerName.c_str(),
+                                               user->dhost.c_str(),
+                                               ServerInstance->Config->HideKillsServer.empty() ? user->nick.c_str() : ServerInstance->Config->HideKillsServer.c_str(),
+                                               parameters[1].c_str());
                        }
                }
 
@@ -106,10 +145,12 @@ CmdResult CommandKill::Handle (const char* const* parameters, int pcnt, User *us
        }
        else
        {
-               user->WriteServ( "401 %s %s :No such nick/channel", user->nick, parameters[0]);
+               user->WriteServ( "401 %s %s :No such nick/channel", user->nick.c_str(), parameters[0].c_str());
                return CMD_FAILURE;
        }
 
        return CMD_SUCCESS;
 }
 
+
+COMMAND_INIT(CommandKill)