X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fcull_list.cpp;h=a7f8d3316c6375d0ab396679fbb47632cbe54150;hb=5ca2a6ffdb4dbb7f7d8f120df5f99d85d484b3ab;hp=a8188701a0114ebf11c7602d0dee8fb9df7fcb24;hpb=7a3d08f06cc22688a70c5274fe0dd489f93e9794;p=user%2Fhenk%2Fcode%2Finspircd.git diff --git a/src/cull_list.cpp b/src/cull_list.cpp index a8188701a..a7f8d3316 100644 --- a/src/cull_list.cpp +++ b/src/cull_list.cpp @@ -2,146 +2,145 @@ * | Inspire Internet Relay Chat Daemon | * +------------------------------------+ * - * InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev. - * E-mail: - * - * + * InspIRCd: (C) 2002-2008 InspIRCd Development Team + * See: http://www.inspircd.org/wiki/index.php/Credits * - * Written by Craig Edwards, Craig McLure, and others. * This program is free but copyrighted software; see * the file COPYING for details. * * --------------------------------------------------- */ -using namespace std; +/* $Core: libIRCDcull_list */ -#include "inspircd_config.h" #include "inspircd.h" -#include "inspircd_io.h" -#include -#include -#include -#include -#include -#ifdef GCC3 -#include -#else -#include -#endif -#include -#include -#include -#include -#include "users.h" -#include "ctables.h" -#include "globals.h" -#include "modules.h" -#include "dynamic.h" -#include "wildcard.h" -#include "message.h" -#include "commands.h" -#include "xline.h" -#include "inspstring.h" -#include "inspircd.h" -#include "helperfuncs.h" -#include "hashcomp.h" -#include "typedefs.h" #include "cull_list.h" -extern InspIRCd* ServerInstance; -extern user_hash clientlist; - -bool CullList::IsValid(userrec* user) +CullList::CullList(InspIRCd* Instance) : ServerInstance(Instance) { - time_t esignon = 0; - std::map::iterator es = exempt.find(user); - if (es != exempt.end()) - esignon = es->second; + list.clear(); +} - for (user_hash::iterator u = clientlist.begin(); u != clientlist.end(); u++) +void CullList::AddItem(User* user) +{ + if (user->quitting) { - /* - * BUGFIX - * - * Because there is an undetermined period of time between a user existing, - * and this function being called, we have to check for the following condition: - * - * Between CullList::AddItem(u) being called, and CullList::IsValid(u) being called, - * the user with the pointer u has quit, but only to be REPLACED WITH A NEW USER WHO - * BECAUSE OF ALLOCATION RULES, HAS THE SAME MEMORY ADDRESS! To prevent this, we - * cross reference each pointer to the user's signon time, and if the signon times - * do not match, we return false here to indicate this user is NOT valid as it - * seems to differ from the pointer snapshot we got a few seconds earlier. Should - * prevent a few random crashes during netsplits. - */ - if (user == u->second) - return (u->second->signon == esignon); + ServerInstance->Logs->Log("CULLLIST",DEBUG, "*** Warning *** - You tried to quit a user (%s) twice. Did your module call QuitUser twice?", user->nick.c_str()); + return; } - return false; -} -CullItem::CullItem(userrec* u, std::string &r) -{ - this->user = u; - this->reason = r; + user->quitting = true; + list.push_back(user); } -userrec* CullItem::GetUser() +void CullList::MakeSilent(User* user) { - return this->user; + user->quietquit = true; + return; } -std::string CullItem::GetReason() +int CullList::Apply() { - return this->reason; -} + int n = list.size(); + int i = 0; -CullList::CullList() -{ - list.clear(); - exempt.clear(); -} - -void CullList::AddItem(userrec* user, std::string &reason) -{ - if (exempt.find(user) == exempt.end()) + while (list.size() && i++ != 100) { - CullItem item(user,reason); - list.push_back(item); - exempt[user] = user->signon; - } -} + std::vector::iterator a = list.begin(); -int CullList::Apply() -{ - int n = 0; - while (list.size()) - { - std::vector::iterator a = list.begin(); - userrec* u = a->GetUser(); - /* Because ServerInstance->DoOneIteration can - * take the user away from us in the middle of - * our operation, we should check to see if this - * pointer is still valid by iterating the hash. - * It's expensive, yes, but the DoOneIteration - * call stops it being horrendously bad. + User *u = (*a); + user_hash::iterator iter = ServerInstance->Users->clientlist->find(u->nick); + const std::string& preset_reason = u->GetOperQuit(); + std::string reason; + std::string oper_reason; + + reason.assign(u->quitmsg, 0, ServerInstance->Config->Limits.MaxQuit); + oper_reason.assign(preset_reason.empty() ? preset_reason : u->operquitmsg, 0, ServerInstance->Config->Limits.MaxQuit); + + if (u->registered != REG_ALL) + if (ServerInstance->Users->unregistered_count) + ServerInstance->Users->unregistered_count--; + + if (IS_LOCAL(u)) + { + if ((!u->sendq.empty()) && (!(*u->GetWriteError()))) + u->FlushWriteBuf(); + } + + if (u->registered == REG_ALL) + { + FOREACH_MOD_I(ServerInstance,I_OnUserQuit,OnUserQuit(u, reason, oper_reason)); + u->PurgeEmptyChannels(); + u->WriteCommonQuit(reason, oper_reason); + } + + FOREACH_MOD_I(ServerInstance,I_OnUserDisconnect,OnUserDisconnect(u)); + + if (IS_LOCAL(u)) + { + if (u->io) + { + try + { + u->io->OnRawSocketClose(u->GetFd()); + } + catch (CoreException& modexcept) + { + ServerInstance->Logs->Log("CULLLIST",DEBUG, "%s threw an exception: %s", modexcept.GetSource(), modexcept.GetReason()); + } + } + + ServerInstance->SE->DelFd(u); + u->CloseSocket(); + } + + /* + * this must come before the ServerInstance->SNO->WriteToSnoMaskso that it doesnt try to fill their buffer with anything + * if they were an oper with +sn +qQ. */ - if (IsValid(u)) + if (u->registered == REG_ALL) + { + if (IS_LOCAL(u)) + { + if (!u->quietquit) + { + ServerInstance->SNO->WriteToSnoMask('q',"Client exiting: %s!%s@%s [%s]", u->nick.c_str(), u->ident.c_str(), u->host.c_str(), oper_reason.c_str()); + } + } + else + { + if ((!ServerInstance->SilentULine(u->server)) && (!u->quietquit)) + { + ServerInstance->SNO->WriteToSnoMask('Q',"Client exiting on server %s: %s!%s@%s [%s]", u->server, u->nick.c_str(), u->ident.c_str(), u->host.c_str(), oper_reason.c_str()); + } + } + u->AddToWhoWas(); + } + + if (iter != ServerInstance->Users->clientlist->end()) { - kill_link(u,a->GetReason().c_str()); - list.erase(list.begin()); - /* So that huge numbers of quits dont block, - * we yield back to our mainloop every 15 - * iterations. - * The DoOneIteration call basically acts - * like a software threading mechanism. - */ - if (((n++) % 15) == 0) + ServerInstance->Users->clientlist->erase(iter); + } + else + { + ServerInstance->Logs->Log("CULLLIST", DEBUG, "iter == clientlist->end, can't remove them from hash... problematic.."); + } + + if (IS_LOCAL(u)) + { + std::vector::iterator x = find(ServerInstance->Users->local_users.begin(),ServerInstance->Users->local_users.end(),u); + if (x != ServerInstance->Users->local_users.end()) + ServerInstance->Users->local_users.erase(x); + else { - ServerInstance->DoOneIteration(false); + ServerInstance->Logs->Log("CULLLIST", DEBUG, "Failed to remove user from vector.."); } } - } - return n; + + delete u; + list.erase(list.begin()); + } + + return n; } +