Fix for bug #513

[user/henk/code/inspircd.git] / src / modules / m_messageflood.cpp

diff --git a/src/modules/m_messageflood.cpp b/src/modules/m_messageflood.cpp
index 638eda6b0e9f9da2ac10d59d5ee68237e7af764c..c2a0b32ec67b61d7cb33c13374409f85db9efe1b 100644 (file)
--- a/src/modules/m_messageflood.cpp
+++ b/src/modules/m_messageflood.cpp
@@ -131,7 +131,7 @@ class MsgFlood : public ModeHandler
                                int nsecs = atoi(secs);
                                if ((nlines<1) || (nsecs<1))
                                {
-                                       source->WriteServ("608 %s %s :Invalid flood parameter",source->nick,channel->name);
+                                       source->WriteNumeric(608, "%s %s :Invalid flood parameter",source->nick,channel->name);
                                        parameter.clear();
                                        return MODEACTION_DENY;
                                }
@@ -177,7 +177,7 @@ class MsgFlood : public ModeHandler
                        }
                        else
                        {
-                               source->WriteServ("608 %s %s :Invalid flood parameter",source->nick,channel->name);
+                               source->WriteNumeric(608, "%s %s :Invalid flood parameter",source->nick,channel->name);
                                parameter.clear();
                                return MODEACTION_DENY;
                        }
@@ -236,27 +236,20 @@ class ModuleMsgFlood : public Module
                                        parameters[0] = dest->name;
                                        parameters[1] = "+b";
                                        parameters[2] = user->MakeWildHost();
-                                       ServerInstance->SendMode(parameters,3,user);
-                                       std::deque<std::string> n;
-                                       /* Propogate the ban to other servers.
-                                        * We dont know what protocol we may be using,
-                                        * so this event is picked up by our protocol
-                                        * module and formed into a ban command that
-                                        * suits the protocol in use.
-                                        */
-                                       n.push_back(dest->name);
-                                       n.push_back("+b");
-                                       n.push_back(user->MakeWildHost());
-                                       Event rmode((char *)&n, NULL, "send_mode");
-                                       rmode.Send(ServerInstance);
+                                       ServerInstance->SendMode(parameters, 3, ServerInstance->FakeClient);
+
+                                       ServerInstance->PI->SendModeStr(dest->name, std::string("+b ") + user->MakeWildHost());
                                }
+
                                char kickmessage[MAXBUF];
                                snprintf(kickmessage, MAXBUF, "Channel flood triggered (limit is %d lines in %d secs)", f->lines, f->secs);
+
                                if (!dest->ServerKickUser(user, kickmessage, true))
                                {
                                        delete dest;
-                                       return 1;
                                }
+
+                               return 1;
                        }
                }
                
@@ -298,7 +291,7 @@ class ModuleMsgFlood : public Module
        
        virtual Version GetVersion()
        {
-               return Version(1, 1, 0, 0, VF_COMMON | VF_VENDOR, API_VERSION);
+               return Version(1, 2, 0, 0, VF_COMMON | VF_VENDOR, API_VERSION);
        }
 };