X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fcoremods%2Fcore_oper%2Fcmd_kill.cpp;h=13d6be55803da4bfb91ee7d52e5711c8e3585d00;hb=c71361e8e4f22cb4f72881399bce2832eb080b0e;hp=c331d8ccd35df4f906ea7b30992cbb4d4b981b91;hpb=0b0ef772755c265ef1fb5f9d9de3fdf9d50473e4;p=user%2Fhenk%2Fcode%2Finspircd.git diff --git a/src/coremods/core_oper/cmd_kill.cpp b/src/coremods/core_oper/cmd_kill.cpp index c331d8ccd..13d6be558 100644 --- a/src/coremods/core_oper/cmd_kill.cpp +++ b/src/coremods/core_oper/cmd_kill.cpp @@ -1,9 +1,15 @@ /* * InspIRCd -- Internet Relay Chat Daemon * - * Copyright (C) 2009 Daniel De Graaf - * Copyright (C) 2008 Craig Edwards + * Copyright (C) 2017-2018 Sadie Powell + * Copyright (C) 2016 Adam + * Copyright (C) 2012-2014, 2018 Attila Molnar + * Copyright (C) 2012, 2019 Robby + * Copyright (C) 2009-2010 Daniel De Graaf + * Copyright (C) 2009 Uli Schlachter * Copyright (C) 2007 Robin Burchell + * Copyright (C) 2007 Dennis Friis + * Copyright (C) 2007 Craig Edwards * * 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 @@ -24,18 +30,34 @@ CommandKill::CommandKill(Module* parent) : Command(parent, "KILL", 2, 2) + , protoev(parent, name) { flags_needed = 'o'; - syntax = " "; + syntax = "[,]+ :"; TRANSLATE2(TR_CUSTOM, TR_CUSTOM); } +class KillMessage : public ClientProtocol::Message +{ + public: + KillMessage(ClientProtocol::EventProvider& protoev, User* user, LocalUser* target, const std::string& text, const std::string& hidenick) + : ClientProtocol::Message("KILL", NULL) + { + if (hidenick.empty()) + SetSourceUser(user); + else + SetSource(hidenick); + + PushParamRef(target->nick); + PushParamRef(text); + } +}; /** Handle /KILL */ -CmdResult CommandKill::Handle (const std::vector& parameters, User *user) +CmdResult CommandKill::Handle(User* user, const Params& parameters) { - /* Allow comma seperated lists of users for /KILL (thanks w00t) */ + /* Allow comma separated lists of users for /KILL (thanks w00t) */ if (CommandParser::LoopCall(user, this, parameters, 0)) { // If we got a colon delimited list of nicks then the handler ran for each nick, @@ -71,10 +93,10 @@ CmdResult CommandKill::Handle (const std::vector& parameters, User return CMD_FAILURE; killreason = "Killed ("; - if (!ServerInstance->Config->HideKillsServer.empty()) + if (!hidenick.empty()) { // hidekills is on, use it - killreason += ServerInstance->Config->HideKillsServer; + killreason += hidenick; } else { @@ -90,41 +112,27 @@ CmdResult CommandKill::Handle (const std::vector& parameters, User killreason.assign(parameters[1], 0, ServerInstance->Config->Limits.MaxQuit); } - /* - * Now we need to decide whether or not to send a local or remote snotice. Currently this checking is a little flawed. - * No time to fix it right now, so left a note. -- w00t - */ - if (!IS_LOCAL(target)) + if ((!hideuline) || (!user->server->IsULine())) { - // remote kill - if ((!ServerInstance->Config->HideULineKills) || (!user->server->IsULine())) + if (IS_LOCAL(user) && IS_LOCAL(target)) + ServerInstance->SNO->WriteGlobalSno('k', "Local kill by %s: %s (%s)", user->nick.c_str(), target->GetFullRealHost().c_str(), parameters[1].c_str()); + else ServerInstance->SNO->WriteToSnoMask('K', "Remote kill by %s: %s (%s)", user->nick.c_str(), target->GetFullRealHost().c_str(), parameters[1].c_str()); - this->lastuuid = target->uuid; } - else - { - // local kill - /* - * 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 - */ - if ((!ServerInstance->Config->HideULineKills) || (!user->server->IsULine())) - { - if (IS_LOCAL(user)) - ServerInstance->SNO->WriteGlobalSno('k',"Local Kill by %s: %s (%s)", user->nick.c_str(), target->GetFullRealHost().c_str(), parameters[1].c_str()); - else - ServerInstance->SNO->WriteToSnoMask('k',"Local Kill by %s: %s (%s)", user->nick.c_str(), target->GetFullRealHost().c_str(), parameters[1].c_str()); - } - - ServerInstance->Logs->Log("KILL", LOG_DEFAULT, "LOCAL KILL: %s :%s!%s!%s (%s)", target->nick.c_str(), ServerInstance->Config->ServerName.c_str(), user->dhost.c_str(), user->nick.c_str(), parameters[1].c_str()); - target->Write(":%s KILL %s :%s", - ServerInstance->Config->HideKillsServer.empty() ? user->GetFullHost().c_str() : ServerInstance->Config->HideKillsServer.c_str(), - target->nick.c_str(), - parameters[1].c_str()); + if (IS_LOCAL(target)) + { + LocalUser* localu = IS_LOCAL(target); + KillMessage msg(protoev, user, localu, killreason, hidenick); + ClientProtocol::Event killevent(protoev, msg); + localu->Send(killevent); this->lastuuid.clear(); } + else + { + this->lastuuid = target->uuid; + } // send the quit out ServerInstance->Users->QuitUser(target, killreason); @@ -132,7 +140,7 @@ CmdResult CommandKill::Handle (const std::vector& parameters, User return CMD_SUCCESS; } -RouteDescriptor CommandKill::GetRouting(User* user, const std::vector& parameters) +RouteDescriptor CommandKill::GetRouting(User* user, const Params& parameters) { // 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 @@ -143,7 +151,7 @@ RouteDescriptor CommandKill::GetRouting(User* user, const std::vector 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