Special check in cmd_quit and cmd_kill and special return value CMD_USER_DELETED, to prevent theoretical segfault (this isnt the bug youre trying to find, pip)

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@5230 e03df62e-2008-0410-955e-edbf42e46eb7

3 files changed, 12 insertions, 3 deletions

diff --git a/src/cmd_kill.cpp b/src/cmd_kill.cpp
index 6fe855a19..dfcf99590 100644
--- a/src/cmd_kill.cpp
+++ b/src/cmd_kill.cpp
@@ -66,6 +66,11 @@ CmdResult cmd_kill::Handle (const char** parameters, int pcnt, userrec *user)
 				u->PurgeEmptyChannels();
 			}
 
+			if (u == user)
+			{
+				FOREACH_MOD(I_OnPostCommand,OnPostCommand("KILL", parameters, pcnt, user, CMD_SUCCESS));
+				return CMD_USER_DELETED;
+			}
 			DELETE(u);
 		}
 		else
diff --git a/src/cmd_quit.cpp b/src/cmd_quit.cpp
index bf64aea3d..41cc48c8e 100644
--- a/src/cmd_quit.cpp
+++ b/src/cmd_quit.cpp
@@ -99,8 +99,9 @@ CmdResult cmd_quit::Handle (const char** parameters, int pcnt, userrec *user)
 	if (user->registered == REG_ALL) {
 		user->PurgeEmptyChannels();
 	}
-	DELETE(user);
 
-	return CMD_SUCCESS;
+	FOREACH_MOD(I_OnPostCommand,OnPostCommand("QUIT", parameters, pcnt, user, CMD_SUCCESS));
+	DELETE(user);
+	return CMD_USER_DELETED;
 }
 
diff --git a/src/command_parse.cpp b/src/command_parse.cpp
index 1545c8e1b..5cc85a2a3 100644
--- a/src/command_parse.cpp
+++ b/src/command_parse.cpp
@@ -386,7 +386,10 @@ void CommandParser::ProcessCommand(userrec *user, std::string &cmd)
 				 */
 				CmdResult result = cm->second->Handle(command_p,items,user);
 
-				FOREACH_MOD(I_OnPostCommand,OnPostCommand(command, command_p, items, user, result));
+				if (result != CMD_USER_DELETED)
+				{
+					FOREACH_MOD(I_OnPostCommand,OnPostCommand(command, command_p, items, user, result));
+				}
 				return;
 			}
 			else