summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
6f15bf7)
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@9784
e03df62e-2008-0410-955e-
edbf42e46eb7
if (iter != ServerInstance->Users->clientlist->end())
{
ServerInstance->Users->clientlist->erase(iter);
}
else
{
if (iter != ServerInstance->Users->clientlist->end())
{
ServerInstance->Users->clientlist->erase(iter);
}
else
{
- /*
- * Trying to track down Jason's issue.. this should never happen obviously.
- */
- ServerInstance->Logs->Log("CULLLIST", DEBUG, "iter == clientlist->end, can't remove them from hash... problematic?");
-// deleteu = false;
-// actually, delete them anyway.. the local vector is the real problem here
+ ServerInstance->Logs->Log("CULLLIST", DEBUG, "iter == clientlist->end, can't remove them from hash... problematic..");
ServerInstance->Users->local_users.erase(x);
else
{
ServerInstance->Users->local_users.erase(x);
else
{
- /*
- * This code is in here to monitor an issue of Jason's, where it seems to be trying to quit already quit users.
- * The only way that can happen is if this find fails, so log it just in case.
- * Also, (perhaps incorrectly, but oh well), return here so we don't delete the user and then start trampling
- * on deleted memory, which leads to big problems..
- */
- ServerInstance->Logs->Log("CULLLIST", DEBUG, "Failed to remove user from vector, we're all gonna die!!! Not deleting the user to save our sanity");
- deleteu = false;
+ ServerInstance->Logs->Log("CULLLIST", DEBUG, "Failed to remove user from vector..");
- if (deleteu)
- {
- delete u;
- }
-
list.erase(list.begin());
}
list.erase(list.begin());
}