This is no longer the case, userrec::QuitUser is safe, so use it.

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

diff --git a/src/modules/m_filter.h b/src/modules/m_filter.h
index 9c9a1238e546169b34d02935620f74d08ee142b0..4f8690fe6cd09686696a03e8f42dd6e37dc9bee4 100644 (file)
--- a/src/modules/m_filter.h
+++ b/src/modules/m_filter.h
@@ -359,12 +359,8 @@ int FilterBase::OnPreCommand(const std::string &command, const char** parameters
                                /* Are they parting, if so, kill is applicable */
                                if ((parting) && (f->action == "kill"))
                                {
-                                       user->SetWriteError("Filtered: "+f->reason);
-                                       /* This WriteServ causes the write error to be applied.
-                                        * Its not safe to kill here with QuitUser in a PreCommand handler,
-                                        * so we do it this way, which is safe just about anywhere.
-                                        */
                                        user->WriteServ("NOTICE %s :*** Your PART message was filtered: %s", user->nick, f->reason.c_str());
+                                       userrec::QuitUser(ServerInstance, "Filtered: " + f->reason);
                                }
                                if (f->action == "gline")
                                {