Gah, im forgetting to SetMode!

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

diff --git a/src/modules/m_timedbans.cpp b/src/modules/m_timedbans.cpp
index c7f99b8b2ba6751d8302ec302d330fbdfc2f3eeb..b1b0de25f7ebaac3292f818e01d209ad4f9bbefa 100644 (file)
--- a/src/modules/m_timedbans.cpp
+++ b/src/modules/m_timedbans.cpp
@@ -26,7 +26,7 @@ using namespace std;
 #include "helperfuncs.h"
 #include "hashcomp.h"
 
-Server *Srv;
+static Server *Srv;
         
 class TimedBan
 {
@@ -44,7 +44,7 @@ class cmd_tban : public command_t
  public:
        cmd_tban () : command_t("TBAN", 0, 3)
        {
-               this->source = "m_timedbans.cpp";
+               this->source = "m_timedbans.so";
        }
 
        void Handle (char **parameters, int pcnt, userrec *user)
@@ -62,8 +62,8 @@ class cmd_tban : public command_t
                                }
                                for (timedbans::iterator i = TimedBanList.begin(); i < TimedBanList.end(); i++)
                                {
-                                       irc::string listitem = i->mask.c_str();
-                                       irc::string target = parameters[2];
+                                       irc::string listitem = i->mask.c_str();
+                                       irc::string target = parameters[2];
                                        irc::string listchan = i->channel.c_str();
                                        irc::string targetchan = parameters[0];
                                        if ((listitem == target) && (listchan == targetchan))
@@ -83,7 +83,7 @@ class cmd_tban : public command_t
                                char duration[MAXBUF];
                                snprintf(duration,MAXBUF,"%lu",Srv->CalcDuration(parameters[1]));
                                std::string mask = parameters[2];
-                               char *setban[3];
+                               char *setban[32];
                                setban[0] = parameters[0];
                                setban[1] = "+b";
                                setban[2] = parameters[2];
@@ -127,18 +127,18 @@ class ModuleTimedBans : public Module
                List[I_OnDelBan] = List[I_OnBackgroundTimer] = 1;
        }
 
-       virtual int OnDelBan(userrec* source, chanrec* chan, std::string banmask)
+       virtual int OnDelBan(userrec* source, chanrec* chan, const std::string &banmask)
        {
-                for (timedbans::iterator i = TimedBanList.begin(); i < TimedBanList.end(); i++)
-                {
-                        irc::string listitem = banmask.c_str();
-                        irc::string target = i->mask.c_str();
-                        if (listitem == target)
-                        {
-                                TimedBanList.erase(i);
-                                break;
-                        }
-                }
+               for (timedbans::iterator i = TimedBanList.begin(); i < TimedBanList.end(); i++)
+               {
+                       irc::string listitem = banmask.c_str();
+                       irc::string target = i->mask.c_str();
+                       if (listitem == target)
+                       {
+                               TimedBanList.erase(i);
+                               break;
+                       }
+               }
                return 0;
        }
 
@@ -170,7 +170,7 @@ class ModuleTimedBans : public Module
                                                temp->fd = FD_MAGIC_NUMBER;
                                                temp->server = "";
                                                Srv->SendMode(setban,3,temp);
-                                               delete temp;
+                                               DELETE(temp);
                                        }
                                        // we used to delete the item here, but we dont need to as the servermode above does it for us,
                                        break;