From 2be86e1958ca8de8aaad54f75a8d6afec59d49af Mon Sep 17 00:00:00 2001 From: special Date: Wed, 26 Sep 2007 23:49:15 +0000 Subject: Fixed bug #404; this was caused by recieving multiple OPERTYPEs from a remote server, which eventually resulted in bad pointers inside the all_opers list git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@8084 e03df62e-2008-0410-955e-edbf42e46eb7 --- src/cmd_who.cpp | 2 +- src/helperfuncs.cpp | 2 +- src/snomasks.cpp | 2 +- src/users.cpp | 13 +++---------- 4 files changed, 6 insertions(+), 13 deletions(-) (limited to 'src') diff --git a/src/cmd_who.cpp b/src/cmd_who.cpp index 07370f31e..93855dce2 100644 --- a/src/cmd_who.cpp +++ b/src/cmd_who.cpp @@ -295,7 +295,7 @@ CmdResult cmd_who::Handle (const char** parameters, int pcnt, userrec *user) if (opt_viewopersonly) { /* Showing only opers */ - for (std::vector::iterator i = ServerInstance->all_opers.begin(); i != ServerInstance->all_opers.end(); i++) + for (std::list::iterator i = ServerInstance->all_opers.begin(); i != ServerInstance->all_opers.end(); i++) { userrec* oper = *i; diff --git a/src/helperfuncs.cpp b/src/helperfuncs.cpp index 1defef047..1cad143c5 100644 --- a/src/helperfuncs.cpp +++ b/src/helperfuncs.cpp @@ -114,7 +114,7 @@ void InspIRCd::WriteOpers(const char* text, ...) void InspIRCd::WriteOpers(const std::string &text) { - for (std::vector::iterator i = this->all_opers.begin(); i != this->all_opers.end(); i++) + for (std::list::iterator i = this->all_opers.begin(); i != this->all_opers.end(); i++) { userrec* a = *i; if (IS_LOCAL(a) && a->IsModeSet('s')) diff --git a/src/snomasks.cpp b/src/snomasks.cpp index e447cd637..df524b51f 100644 --- a/src/snomasks.cpp +++ b/src/snomasks.cpp @@ -53,7 +53,7 @@ void SnomaskManager::WriteToSnoMask(char letter, const std::string &text) if (n != SnoMasks.end()) { /* Only opers can receive snotices, so we iterate the oper list */ - for (std::vector::iterator i = ServerInstance->all_opers.begin(); i != ServerInstance->all_opers.end(); i++) + for (std::list::iterator i = ServerInstance->all_opers.begin(); i != ServerInstance->all_opers.end(); i++) { userrec* a = *i; if (IS_LOCAL(a) && a->IsModeSet('s') && a->IsModeSet('n') && a->IsNoticeMaskSet(n->first)) diff --git a/src/users.cpp b/src/users.cpp index b357fd149..4f5c567d2 100644 --- a/src/users.cpp +++ b/src/users.cpp @@ -830,16 +830,9 @@ void userrec::UnOper() // unset their oper type (what IS_OPER checks), and remove +o *this->oper = 0; this->modes[UM_OPERATOR] = 0; - - // remove them from the opers list. - for (std::vector::iterator a = ServerInstance->all_opers.begin(); a < ServerInstance->all_opers.end(); a++) - { - if (*a == this) - { - ServerInstance->all_opers.erase(a); - return; - } - } + + // remove the user from the oper list. Will remove multiple entries as a safeguard against bug #404 + ServerInstance->all_opers.remove(this); } } -- cgit v1.2.3