diff options
-rw-r--r-- | include/dns.h | 19 | ||||
-rw-r--r-- | src/dns.cpp | 46 |
2 files changed, 53 insertions, 12 deletions
diff --git a/include/dns.h b/include/dns.h index 7b96f792d..db058be9b 100644 --- a/include/dns.h +++ b/include/dns.h @@ -147,14 +147,21 @@ class Resolver : public Extensible * the object to go 'out of scope' and cause a segfault in the core if the result * arrives at a later time. * @param source The IP or hostname to resolve - * @param forward Set to true to perform a forward lookup (hostname to ip) or false - * to perform a reverse lookup (ip to hostname). Lookups on A records and PTR - * records are supported. CNAME and MX are not supported by this resolver. - * If InspIRCd is compiled with ipv6 support, lookups on AAAA records are preferred - * and supported over A records. + * @param qt The query type to perform. If you just want to perform a forward + * or reverse lookup, and you don't care wether you get ipv4 or ipv6, then use + * the constants DNS_QUERY_FORWARD and DNS_QUERY_REVERSE, which automatically + * select from 'A' record or 'AAAA' record lookups. However, if you want to resolve + * a specific record type, resolution of 'A', 'AAAA', 'PTR' and 'CNAME' records + * is supported. Use one of the QueryType enum values to initiate this type of + * lookup. Resolution of 'AAAA' ipv6 records is always supported, regardless of + * wether InspIRCd is built with ipv6 support. + * If you attempt to resolve a 'PTR' record and InspIRCd is built with ipv6 support, + * the 'PTR' record will be formatted to ipv6 specs, e.g. x.x.x.x.x....ip6.int. + * otherwise it will be formatted to ipv4 specs, e.g. x.x.x.x.in-addr.arpa. * @throw ModuleException This class may throw an instance of ModuleException, in the * event a lookup could not be allocated, or a similar hard error occurs such as - * the network being down. + * the network being down. This will also be thrown if an invalid IP address is + * passed when resolving a 'PTR' record. */ Resolver(const std::string &source, QueryType qt); /** diff --git a/src/dns.cpp b/src/dns.cpp index 3db0c2486..ae0e203cd 100644 --- a/src/dns.cpp +++ b/src/dns.cpp @@ -399,17 +399,52 @@ int DNS::GetCName(const char *alias) /* Start lookup of an IP address to a hostname */ int DNS::GetName(const insp_inaddr *ip) { -#ifdef IPV6 - return -1; -#else char query[29]; DNSHeader h; int id; int length; +#ifdef IPV6 + /* XXX: This SUCKS. and i mean REALLY, REALLY sucks. Anyone who rewrites it pretty gets a cookie. */ + sprintf(query,"%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.%0x.ip6.int", + ip->s6_addr[15] & 0x0f, + (ip->s6_addr[15] & 0xf0) >> 4, + ip->s6_addr[14] & 0x0f, + (ip->s6_addr[14] & 0xf0) >> 4, + ip->s6_addr[13] & 0x0f, + (ip->s6_addr[13] & 0xf0) >> 4, + ip->s6_addr[12] & 0x0f, + (ip->s6_addr[12] & 0xf0) >> 4, + ip->s6_addr[11] & 0x0f, + (ip->s6_addr[11] & 0xf0) >> 4, + ip->s6_addr[10] & 0x0f, + (ip->s6_addr[10] & 0xf0) >> 4, + ip->s6_addr[9] & 0x0f, + (ip->s6_addr[9] & 0xf0) >> 4, + ip->s6_addr[8] & 0x0f, + (ip->s6_addr[8] & 0xf0) >> 4, + ip->s6_addr[7] & 0x0f, + (ip->s6_addr[7] & 0xf0) >> 4, + ip->s6_addr[6] & 0x0f, + (ip->s6_addr[6] & 0xf0) >> 4, + ip->s6_addr[5] & 0x0f, + (ip->s6_addr[5] & 0xf0) >> 4, + ip->s6_addr[4] & 0x0f, + (ip->s6_addr[4] & 0xf0) >> 4, + ip->s6_addr[3] & 0x0f, + (ip->s6_addr[3] & 0xf0) >> 4, + ip->s6_addr[2] & 0x0f, + (ip->s6_addr[2] & 0xf0) >> 4, + ip->s6_addr[1] & 0x0f, + (ip->s6_addr[1] & 0xf0) >> 4, + ip->s6_addr[0] & 0x0f, + (ip->s6_addr[0] & 0xf0) >> 4 + ); +#else unsigned char* c = (unsigned char*)&ip->s_addr; sprintf(query,"%d.%d.%d.%d.in-addr.arpa",c[3],c[2],c[1],c[0]); +#endif if ((length = this->MakePayload(query, DNS_QUERY_PTR, 1, (unsigned char*)&h.payload)) == -1) return -1; @@ -420,7 +455,6 @@ int DNS::GetName(const insp_inaddr *ip) return -1; return id; -#endif } /* Return the next id which is ready, and the result attached to it */ @@ -718,10 +752,10 @@ Resolver::Resolver(const std::string &source, QueryType qt) : input(source), que break; case DNS_QUERY_PTR: - if (insp_aton(source.c_str(), &binip) > 0) + if (insp_aton(source.c_str(), &binip) > 0) { /* Valid ip address */ - this->myid = ServerInstance->Res->GetName(&binip); + this->myid = ServerInstance->Res->GetName(&binip); } else { |