Fix incorrect iterator use leading to prematurely exiting loops when quitting users while looping the local user list

1 files changed, 2 insertions, 2 deletions

diff --git a/src/inspircd.cpp b/src/inspircd.cpp
index cb2b5db9d..5dfcca6d9 100644
--- a/src/inspircd.cpp
+++ b/src/inspircd.cpp
@@ -110,8 +110,8 @@ void InspIRCd::Cleanup()
 
 	/* Close all client sockets, or the new process inherits them */
 	const UserManager::LocalList& list = Users.GetLocalUsers();
-	for (UserManager::LocalList::const_iterator i = list.begin(); i != list.end(); ++i)
-		Users->QuitUser(*i, "Server shutdown");
+	while (!list.empty())
+		Users->QuitUser(list.front(), "Server shutdown");
 
 	GlobalCulls.Apply();
 	Modules->UnloadAll();