X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fcoremods%2Fcore_hostname_lookup.cpp;h=ec93732b124c732b7569f36454f6fdde55b3b762;hb=6adca3e0997781eae4adb02f19a2f8c312512ae1;hp=4320b1e574207eaf85078590929d7bcf99635172;hpb=124c17e14134a4999afc1a5e981ab7c75b3694b9;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/coremods/core_hostname_lookup.cpp b/src/coremods/core_hostname_lookup.cpp
index 4320b1e57..ec93732b1 100644
--- a/src/coremods/core_hostname_lookup.cpp
+++ b/src/coremods/core_hostname_lookup.cpp
@@ -80,7 +80,7 @@ class UserResolver : public DNS::Request
 			ph->set(bound_user, ans_record->rdata);
 
 			UserResolver* res_forward;
-			if (bound_user->client_sa.sa.sa_family == AF_INET6)
+			if (bound_user->client_sa.family() == AF_INET6)
 			{
 				/* IPV6 forward lookup */
 				res_forward = new UserResolver(this->manager, this->creator, bound_user, ans_record->rdata, DNS::QUERY_AAAA);
@@ -109,7 +109,7 @@ class UserResolver : public DNS::Request
 
 			irc::sockets::sockaddrs* user_ip = &bound_user->client_sa;
 			bool rev_match = false;
-			if (user_ip->sa.sa_family == AF_INET6)
+			if (user_ip->family() == AF_INET6)
 			{
 				struct in6_addr res_bin;
 				if (inet_pton(AF_INET6, ans_record->rdata.c_str(), &res_bin))
@@ -199,6 +199,13 @@ class ModuleHostnameLookup : public Module
 			return;
 		}
 
+		// Clients can't have a DNS hostname if they aren't connected via IPv4 or IPv6.
+		if (user->client_sa.family() != AF_INET && user->client_sa.family() != AF_INET6)
+		{
+			user->WriteNotice("*** Skipping host resolution (connected via a non-IP socket)");
+			return;
+		}
+
 		user->WriteNotice("*** Looking up your hostname...");
 
 		UserResolver* res_reverse = new UserResolver(*this->DNS, this, user, user->GetIPString(), DNS::QUERY_PTR);