Encapsulate CHostLookup member variables

The host lookup job is currently only used for the DNSBL lookup, so only one change was necessary.
Robyt3 · Oct 22, 2023 · 67480f1 · 67480f1
1 parent bab9356
commit 67480f1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/src/engine/server/server.cpp b/src/engine/server/server.cpp
@@ -2821,7 +2821,7 @@ int CServer::Run()
 					else if(m_aClients[ClientID].m_DnsblState == CClient::DNSBL_STATE_PENDING &&
 						m_aClients[ClientID].m_pDnsblLookup->Status() == IJob::STATE_DONE)
 					{
-						if(m_aClients[ClientID].m_pDnsblLookup->m_Result != 0)
+						if(m_aClients[ClientID].m_pDnsblLookup->Result() != 0)
 						{
 							// entry not found -> whitelisted
 							m_aClients[ClientID].m_DnsblState = CClient::DNSBL_STATE_WHITELISTED;

diff --git a/src/engine/shared/host_lookup.h b/src/engine/shared/host_lookup.h
@@ -10,16 +10,21 @@
 class CHostLookup : public IJob
 {
 private:
+	int m_Result;
+	char m_aHostname[128];
+	int m_Nettype;
+	NETADDR m_Addr;
+
 	void Run() override;
 
 public:
 	CHostLookup();
 	CHostLookup(const char *pHostname, int Nettype);
 
-	int m_Result;
-	char m_aHostname[128];
-	int m_Nettype;
-	NETADDR m_Addr;
+	int Result() const { return m_Result; }
+	const char *Hostname() const { return m_aHostname; }
+	int Nettype() const { return m_Nettype; }
+	NETADDR Addr() const { return m_Addr; }
 };
 
 #endif