Tidyups, rename some vars

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

diff --git a/include/dns.h b/include/dns.h
index 8334d5dcbbd1cabc2494100c5207297f665ec46a..5ef3f60e52f49adf2415df2d103e93e997753de3 100644 (file)
--- a/include/dns.h
+++ b/include/dns.h
@@ -1,5 +1,22 @@
+/*       +------------------------------------+
+ *       | Inspire Internet Relay Chat Daemon |
+ *       +------------------------------------+
+ *
+ *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
+ *                     E-mail:
+ *              <brain@chatspike.net>
+ *              <Craig@chatspike.net>
+ *
+ * Written by Craig Edwards, Craig McLure, and others.
+ * This program is free but copyrighted software; see
+ *          the file COPYING for details.
+ *
+ * ---------------------------------------------------
+ */
+
 /*
-dns.h - dns library declarations based on firedns Copyright (C) 2002 Ian Gulliver
+dns.h - dns library very very loosely based on
+firedns, Copyright (C) 2002 Ian Gulliver
 
 This program is free software; you can redistribute it and/or modify
 it under the terms of version 2 of the GNU General Public License as
@@ -129,6 +146,7 @@ class DNS : public Extensible
  private:
        insp_inaddr myserver;
        static int MasterSocket;
+       Resolver* Classes[65536];
  public:
        static int GetMasterSocket();
        int GetIP(const char* name);
@@ -142,3 +160,4 @@ class DNS : public Extensible
 };
 
 #endif
+

diff --git a/src/dns.cpp b/src/dns.cpp
index 713946033456fb0df549d9e886aa70da02523f72..cf1f02a5963edb708dfe902b8b7e9834cf621340 100644 (file)
--- a/src/dns.cpp
+++ b/src/dns.cpp
@@ -77,13 +77,6 @@ enum QueryFlags
 };
 
 
-/* Lookup table of Resolver classes. Because the request ID can be between
- * 0 and 65535 of these, we have 65536 of them. This could be a map, saving
- * some ram, but that will also slow down DNS requests, and the DNS request
- * mechanism needs to be pretty fast.
- */
-Resolver* dns_classes[65536];
-
 /* Represents a dns resource record (rr) */
 class ResourceRecord
 {
@@ -261,7 +254,7 @@ int DNS::GetMasterSocket()
 DNS::DNS()
 {
        log(DEBUG,"----- Initialize dns class ----- ");
-       memset(dns_classes,0,sizeof(dns_classes));
+       memset(Classes,0,sizeof(Classes));
        insp_inaddr addr;
        srand((unsigned int)time(NULL));
        memset(&myserver,0,sizeof(insp_inaddr));
@@ -340,7 +333,7 @@ int DNSMakePayload(const char * const name, const unsigned short rr, const unsig
                payload[payloadpos++] = l;
                memcpy(&payload[payloadpos],tempchr2,l);
                payloadpos += l;
-               payload[payloadpos++] = '\0';
+               payload[payloadpos++] = 0;
        }
        if (payloadpos > 508)
                return -1;
@@ -588,7 +581,7 @@ DNSInfo DNSRequest::ResultIsReady(DNSHeader &header, int length)
                                        }
                                        else
                                        {
-                                               res[o] = '\0';
+                                               res[o] = 0;
                                                if (o != 0)
                                                        res[o++] = '.';
                                                memcpy(&res[o],&header.payload[i + 1],header.payload[i]);
@@ -597,15 +590,15 @@ DNSInfo DNSRequest::ResultIsReady(DNSHeader &header, int length)
                                        }
                                }
                        }
-                       res[o] = '\0';
+                       res[o] = 0;
                break;
                case DNS_QRY_A:
                        memcpy(res,&header.payload[i],rr.rdlength);
-                       res[rr.rdlength] = '\0';
+                       res[rr.rdlength] = 0;
                        break;
                default:
                        memcpy(res,&header.payload[i],rr.rdlength);
-                       res[rr.rdlength] = '\0';
+                       res[rr.rdlength] = 0;
                        break;
        }
        return std::make_pair(res,"No error");;
@@ -679,21 +672,21 @@ void DNS::MarshallReads(int fd)
                                res.first -= ERROR_MASK;
 
                                log(DEBUG,"Error available, id=%d",res.first);
-                               if (dns_classes[res.first])
+                               if (Classes[res.first])
                                {
-                                       dns_classes[res.first]->OnError(RESOLVER_NXDOMAIN, res.second);
-                                       delete dns_classes[res.first];
-                                       dns_classes[res.first] = NULL;
+                                       Classes[res.first]->OnError(RESOLVER_NXDOMAIN, res.second);
+                                       delete Classes[res.first];
+                                       Classes[res.first] = NULL;
                                }
                        }
                        else
                        {
                                log(DEBUG,"Result available, id=%d",res.first);
-                               if (dns_classes[res.first])
+                               if (Classes[res.first])
                                {
-                                       dns_classes[res.first]->OnLookupComplete(res.second);
-                                       delete dns_classes[res.first];
-                                       dns_classes[res.first] = NULL;
+                                       Classes[res.first]->OnLookupComplete(res.second);
+                                       delete Classes[res.first];
+                                       Classes[res.first] = NULL;
                                }
                        }
                }
@@ -704,9 +697,9 @@ bool DNS::AddResolverClass(Resolver* r)
 {
        if ((r) && (r->GetId() > -1))
        {
-               if (!dns_classes[r->GetId()])
+               if (!Classes[r->GetId()])
                {
-                       dns_classes[r->GetId()] = r;
+                       Classes[r->GetId()] = r;
                        return true;
                }
                else