]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_dnsbl.cpp
Mark +P mode as oper-only now that it no longer requires an explicit permission string
[user/henk/code/inspircd.git] / src / modules / m_dnsbl.cpp
index cb32ede78e6cd950266f6a1ed69db5b562a37f6f..8fa627ab85a073d421723883d651c84d7222932c 100644 (file)
@@ -2,7 +2,7 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
+ *  InspIRCd: (C) 2002-2010 InspIRCd Development Team
  * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
@@ -24,7 +24,7 @@
 /* $ModDesc: Provides handling of DNS blacklists */
 
 /* Class holding data for a single entry */
-class DNSBLConfEntry : public classbase
+class DNSBLConfEntry
 {
        public:
                enum EnumBanaction { I_UNKNOWN, I_KILL, I_ZLINE, I_KLINE, I_GLINE, I_MARK };
@@ -46,12 +46,12 @@ class DNSBLConfEntry : public classbase
 class DNSBLResolver : public Resolver
 {
        int theirfd;
-       User* them;
+       LocalUser* them;
        DNSBLConfEntry *ConfEntry;
 
  public:
 
-       DNSBLResolver(Module *me, const std::string &hostname, User* u, int userfd, DNSBLConfEntry *conf, bool &cached)
+       DNSBLResolver(Module *me, const std::string &hostname, LocalUser* u, int userfd, DNSBLConfEntry *conf, bool &cached)
                : Resolver(hostname, DNS_QUERY_A, cached, me)
        {
                theirfd = userfd;
@@ -63,7 +63,7 @@ class DNSBLResolver : public Resolver
        virtual void OnLookupComplete(const std::string &result, unsigned int ttl, bool cached)
        {
                /* Check the user still exists */
-               if ((them) && (them == ServerInstance->SE->GetRef(theirfd)))
+               if ((them) && (&them->eh == ServerInstance->SE->GetRef(theirfd)))
                {
                        // Now we calculate the bitmask: 256*(256*(256*a+b)+c)+d
                        if(result.length())
@@ -125,7 +125,7 @@ class DNSBLResolver : public Resolver
                                                }
                                                case DNSBLConfEntry::I_KLINE:
                                                {
-                                                       KLine* kl = new KLine(ServerInstance->Time(), ConfEntry->duration, ServerInstance->Config->ServerName, reason.c_str(),
+                                                       KLine* kl = new KLine(ServerInstance->Time(), ConfEntry->duration, ServerInstance->Config->ServerName.c_str(), reason.c_str(),
                                                                        "*", them->GetIPString());
                                                        if (ServerInstance->XLines->AddLine(kl,NULL))
                                                        {
@@ -139,7 +139,7 @@ class DNSBLResolver : public Resolver
                                                }
                                                case DNSBLConfEntry::I_GLINE:
                                                {
-                                                       GLine* gl = new GLine(ServerInstance->Time(), ConfEntry->duration, ServerInstance->Config->ServerName, reason.c_str(),
+                                                       GLine* gl = new GLine(ServerInstance->Time(), ConfEntry->duration, ServerInstance->Config->ServerName.c_str(), reason.c_str(),
                                                                        "*", them->GetIPString());
                                                        if (ServerInstance->XLines->AddLine(gl,NULL))
                                                        {
@@ -153,7 +153,7 @@ class DNSBLResolver : public Resolver
                                                }
                                                case DNSBLConfEntry::I_ZLINE:
                                                {
-                                                       ZLine* zl = new ZLine(ServerInstance->Time(), ConfEntry->duration, ServerInstance->Config->ServerName, reason.c_str(),
+                                                       ZLine* zl = new ZLine(ServerInstance->Time(), ConfEntry->duration, ServerInstance->Config->ServerName.c_str(), reason.c_str(),
                                                                        them->GetIPString());
                                                        if (ServerInstance->XLines->AddLine(zl,NULL))
                                                        {
@@ -228,7 +228,7 @@ class ModuleDNSBL : public Module
 
        virtual Version GetVersion()
        {
-               return Version("Provides handling of DNS blacklists", VF_VENDOR, API_VERSION);
+               return Version("Provides handling of DNS blacklists", VF_VENDOR);
        }
 
 
@@ -245,36 +245,36 @@ class ModuleDNSBL : public Module
         */
        virtual void ReadConf()
        {
-               ConfigReader *MyConf = new ConfigReader;
+               ConfigReader MyConf;
                ClearEntries();
 
-               for (int i=0; i< MyConf->Enumerate("dnsbl"); i++)
+               for (int i=0; i< MyConf.Enumerate("dnsbl"); i++)
                {
                        DNSBLConfEntry *e = new DNSBLConfEntry();
 
-                       e->name = MyConf->ReadValue("dnsbl", "name", i);
-                       e->ident = MyConf->ReadValue("dnsbl", "ident", i);
-                       e->host = MyConf->ReadValue("dnsbl", "host", i);
-                       e->reason = MyConf->ReadValue("dnsbl", "reason", i);
-                       e->domain = MyConf->ReadValue("dnsbl", "domain", i);
+                       e->name = MyConf.ReadValue("dnsbl", "name", i);
+                       e->ident = MyConf.ReadValue("dnsbl", "ident", i);
+                       e->host = MyConf.ReadValue("dnsbl", "host", i);
+                       e->reason = MyConf.ReadValue("dnsbl", "reason", i);
+                       e->domain = MyConf.ReadValue("dnsbl", "domain", i);
 
-                       if (MyConf->ReadValue("dnsbl", "type", i) == "bitmask")
+                       if (MyConf.ReadValue("dnsbl", "type", i) == "bitmask")
                        {
                                e->type = DNSBLConfEntry::A_BITMASK;
-                               e->bitmask = MyConf->ReadInteger("dnsbl", "bitmask", i, false);
+                               e->bitmask = MyConf.ReadInteger("dnsbl", "bitmask", i, false);
                        }
                        else
                        {
                                memset(e->records, 0, sizeof(e->records));
                                e->type = DNSBLConfEntry::A_RECORD;
-                               irc::portparser portrange(MyConf->ReadValue("dnsbl", "records", i), false);
+                               irc::portparser portrange(MyConf.ReadValue("dnsbl", "records", i), false);
                                long item = -1;
                                while ((item = portrange.GetToken()))
                                        e->records[item] = 1;
                        }
 
-                       e->banaction = str2banaction(MyConf->ReadValue("dnsbl", "action", i));
-                       e->duration = ServerInstance->Duration(MyConf->ReadValue("dnsbl", "duration", "60", i));
+                       e->banaction = str2banaction(MyConf.ReadValue("dnsbl", "action", i));
+                       e->duration = ServerInstance->Duration(MyConf.ReadValue("dnsbl", "duration", "60", i));
 
                        /* Use portparser for record replies */
 
@@ -315,8 +315,6 @@ class ModuleDNSBL : public Module
                        /* delete and drop it, error somewhere */
                        delete e;
                }
-
-               delete MyConf;
        }
 
        virtual void OnRehash(User* user)
@@ -324,42 +322,38 @@ class ModuleDNSBL : public Module
                ReadConf();
        }
 
-       virtual ModResult OnUserRegister(User* user)
+       virtual ModResult OnUserRegister(LocalUser* user)
        {
-               /* only do lookups on local users */
-               if (IS_LOCAL(user))
-               {
-                       /* following code taken from bopm, reverses an IP address. */
-                       struct in_addr in;
-                       unsigned char a, b, c, d;
-                       char reversedipbuf[128];
-                       std::string reversedip;
-                       bool success;
+               /* following code taken from bopm, reverses an IP address. */
+               struct in_addr in;
+               unsigned char a, b, c, d;
+               char reversedipbuf[128];
+               std::string reversedip;
+               bool success;
 
-                       success = inet_aton(user->GetIPString(), &in);
+               success = inet_aton(user->GetIPString(), &in);
 
-                       if (!success)
-                               return MOD_RES_PASSTHRU;
+               if (!success)
+                       return MOD_RES_PASSTHRU;
 
-                       d = (unsigned char) (in.s_addr >> 24) & 0xFF;
-                       c = (unsigned char) (in.s_addr >> 16) & 0xFF;
-                       b = (unsigned char) (in.s_addr >> 8) & 0xFF;
-                       a = (unsigned char) in.s_addr & 0xFF;
+               d = (unsigned char) (in.s_addr >> 24) & 0xFF;
+               c = (unsigned char) (in.s_addr >> 16) & 0xFF;
+               b = (unsigned char) (in.s_addr >> 8) & 0xFF;
+               a = (unsigned char) in.s_addr & 0xFF;
 
-                       snprintf(reversedipbuf, 128, "%d.%d.%d.%d", d, c, b, a);
-                       reversedip = std::string(reversedipbuf);
+               snprintf(reversedipbuf, 128, "%d.%d.%d.%d", d, c, b, a);
+               reversedip = std::string(reversedipbuf);
 
-                       // For each DNSBL, we will run through this lookup
-                       for (std::vector<DNSBLConfEntry *>::iterator i = DNSBLConfEntries.begin(); i != DNSBLConfEntries.end(); i++)
-                       {
-                               // Fill hostname with a dnsbl style host (d.c.b.a.domain.tld)
-                               std::string hostname = reversedip + "." + (*i)->domain;
+               // For each DNSBL, we will run through this lookup
+               for (std::vector<DNSBLConfEntry *>::iterator i = DNSBLConfEntries.begin(); i != DNSBLConfEntries.end(); i++)
+               {
+                       // Fill hostname with a dnsbl style host (d.c.b.a.domain.tld)
+                       std::string hostname = reversedip + "." + (*i)->domain;
 
-                               /* now we'd need to fire off lookups for `hostname'. */
-                               bool cached;
-                               DNSBLResolver *r = new DNSBLResolver(this, hostname, user, user->GetFd(), *i, cached);
-                               ServerInstance->AddResolver(r, cached);
-                       }
+                       /* now we'd need to fire off lookups for `hostname'. */
+                       bool cached;
+                       DNSBLResolver *r = new DNSBLResolver(this, hostname, user, user->GetFd(), *i, cached);
+                       ServerInstance->AddResolver(r, cached);
                }
 
                /* don't do anything with this hot potato */
@@ -378,12 +372,12 @@ class ModuleDNSBL : public Module
                        total_hits += (*i)->stats_hits;
                        total_misses += (*i)->stats_misses;
 
-                       results.push_back(std::string(ServerInstance->Config->ServerName) + " 304 " + user->nick + " :DNSBLSTATS DNSbl \"" + (*i)->name + "\" had " +
+                       results.push_back(std::string(ServerInstance->Config->ServerName.c_str()) + " 304 " + user->nick + " :DNSBLSTATS DNSbl \"" + (*i)->name + "\" had " +
                                        ConvToStr((*i)->stats_hits) + " hits and " + ConvToStr((*i)->stats_misses) + " misses");
                }
 
-               results.push_back(std::string(ServerInstance->Config->ServerName) + " 304 " + user->nick + " :DNSBLSTATS Total hits: " + ConvToStr(total_hits));
-               results.push_back(std::string(ServerInstance->Config->ServerName) + " 304 " + user->nick + " :DNSBLSTATS Total misses: " + ConvToStr(total_misses));
+               results.push_back(std::string(ServerInstance->Config->ServerName.c_str()) + " 304 " + user->nick + " :DNSBLSTATS Total hits: " + ConvToStr(total_hits));
+               results.push_back(std::string(ServerInstance->Config->ServerName.c_str()) + " 304 " + user->nick + " :DNSBLSTATS Total misses: " + ConvToStr(total_misses));
 
                return MOD_RES_PASSTHRU;
        }