InspSocket no longer resolves hosts.

InspSocket::DoResolve() and the stuff that calls it is gone, if you pass InspSocket an invalid ip, it will bail during its connect.
You must now use Resolver classes to resolve hostnames into IP addresses, if you wish to do this. Currently, only one non-extra module
does this, see class ServernameResolver within m_spanningtree


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

3 files changed, 75 insertions, 32 deletions

diff --git a/src/dns.cpp b/src/dns.cpp
index e0cb3ccaa..753df7cf8 100644
--- a/src/dns.cpp
+++ b/src/dns.cpp
@@ -578,10 +578,12 @@ Resolver::Resolver(const std::string &source, bool forward) : input(source), fwd
 {
 	if (forward)
 	{
+		log(DEBUG,"Resolver: Forward lookup on %s",source.c_str());
 		this->myid = Res->dns_getip4(source.c_str());
 	}
 	else
 	{
+		log(DEBUG,"Resolver: Reverse lookup on %s",source.c_str());
 		insp_inaddr binip;
 	        if (insp_aton(source.c_str(), &binip) > 0)
 		{
diff --git a/src/inspsocket.cpp b/src/inspsocket.cpp
index 99ef731c6..f443873e5 100644
--- a/src/inspsocket.cpp
+++ b/src/inspsocket.cpp
@@ -32,9 +32,11 @@
 extern InspIRCd* ServerInstance;
 extern ServerConfig* Config;
 extern time_t TIME;
+extern Server* MyServer;
 
 InspSocket* socket_ref[MAX_DESCRIPTORS];
 
+
 InspSocket::InspSocket()
 {
 	this->state = I_DISCONNECTED;
@@ -101,15 +103,13 @@ InspSocket::InspSocket(const std::string &ahost, int aport, bool listening, unsi
 
 		if (insp_aton(host,&addy) < 1)
 		{
-			log(DEBUG,"Attempting to resolve %s",this->host);
-			/* Its not an ip, spawn the resolver */
-
-			/* TODO: Implement resolver with new Resolver class */
-
-			timeout_end = time(NULL) + maxtime;
-			timeout = false;
-			this->state = I_RESOLVING;
-			socket_ref[this->fd] = this;
+			log(DEBUG,"You cannot pass hostnames to InspSocket, resolve them first with Resolver!");
+			this->Close();
+			this->fd = -1;
+			this->state = I_ERROR;
+			this->OnError(I_ERR_RESOLVE);
+			this->ClosePending = true;
+			return;
 		}
 		else
 		{
@@ -147,14 +147,6 @@ void InspSocket::SetQueues(int nfd)
 	setsockopt(nfd,SOL_SOCKET,SO_RCVBUF,(const void *)&recvbuf,sizeof(sendbuf));
 }
 
-bool InspSocket::DoResolve()
-{
-	log(DEBUG,"In DoResolve(), trying to resolve IP");
-
-	log(DEBUG,"No result for socket yet!");
-	return true;
-}
-
 /* Most irc servers require you to specify the ip you want to bind to.
  * If you dont specify an IP, they rather dumbly bind to the first IP
  * of the box (e.g. INADDR_ANY). In InspIRCd, we scan thought the IP
@@ -399,7 +391,7 @@ bool InspSocket::Timeout(time_t current)
 		return true;
 	}
 
-	if (((this->state == I_RESOLVING) || (this->state == I_CONNECTING)) && (current > timeout_end))
+	if ((this->state == I_CONNECTING) && (current > timeout_end))
 	{
 		log(DEBUG,"Timed out, current=%lu timeout_end=%lu");
 		// for non-listening sockets, the timeout can occur
@@ -429,10 +421,6 @@ bool InspSocket::Poll()
 
 	switch (this->state)
 	{
-		case I_RESOLVING:
-			log(DEBUG,"State = I_RESOLVING, calling DoResolve()");
-			return this->DoResolve();
-		break;
 		case I_CONNECTING:
 			log(DEBUG,"State = I_CONNECTING");
 			this->SetState(I_CONNECTED);
diff --git a/src/modules/m_spanningtree.cpp b/src/modules/m_spanningtree.cpp
index 0725d7b51..c91926d98 100644
--- a/src/modules/m_spanningtree.cpp
+++ b/src/modules/m_spanningtree.cpp
@@ -186,6 +186,7 @@ class UserManager : public classbase
 	}
 };
 
+
 /* Each server in the tree is represented by one class of
  * type TreeServer. A locally connected TreeServer can
  * have a class of type TreeSocket associated with it, for
@@ -3064,6 +3065,36 @@ class TreeSocket : public InspSocket
 	}
 };
 
+class ServernameResolver : public Resolver
+{       
+ private:
+        Link MyLink;
+ public:        
+        ServernameResolver(const std::string &hostname, Link x) : Resolver(hostname, true), MyLink(x)
+        {
+        }
+                
+        void OnLookupComplete(const std::string &result)
+        {
+                TreeSocket* newsocket = new TreeSocket(result,MyLink.Port,false,10,MyLink.Name.c_str());
+                if (newsocket->GetFd() > -1)
+                {
+                        Srv->AddSocket(newsocket);
+                }
+                else
+                {
+                        WriteOpers("*** CONNECT: Error connecting \002%s\002: %s.",MyLink.Name.c_str(),strerror(errno));
+                        delete newsocket;
+                }
+        }
+                
+        void OnError(ResolverError e)
+        {
+                WriteOpers("*** CONNECT: Error connecting \002%s\002: Unable to resolve hostname.",MyLink.Name.c_str());
+        }
+};
+
+
 void AddThisServer(TreeServer* server, std::deque<TreeServer*> &list)
 {
 	for (unsigned int c = 0; c < list.size(); c++)
@@ -3666,16 +3697,28 @@ class ModuleSpanningTree : public Module
 				{
 					// an autoconnected server is not connected. Check if its time to connect it
 					WriteOpers("*** AUTOCONNECT: Auto-connecting server \002%s\002 (%lu seconds until next attempt)",x->Name.c_str(),x->AutoConnect);
-					TreeSocket* newsocket = new TreeSocket(x->IPAddr,x->Port,false,10,x->Name.c_str());
-					if (newsocket->GetFd() > -1)
+
+					insp_inaddr binip;
+
+					if (insp_aton(x->IPAddr.c_str(), &binip) > 0)
 					{
-						Srv->AddSocket(newsocket);
+						TreeSocket* newsocket = new TreeSocket(x->IPAddr,x->Port,false,10,x->Name.c_str());
+						if (newsocket->GetFd() > -1)
+						{
+							Srv->AddSocket(newsocket);
+						}
+						else
+						{
+							WriteOpers("*** AUTOCONNECT: Error autoconnecting \002%s\002: %s.",x->Name.c_str(),strerror(errno));
+							delete newsocket;
+						}
 					}
 					else
 					{
-						WriteOpers("*** AUTOCONNECT: Error autoconnecting \002%s\002: %s.",x->Name.c_str(),strerror(errno));
-						DELETE(newsocket);
+						ServernameResolver* snr = new ServernameResolver(x->IPAddr, *x);
+						Srv->AddResolver(snr);
 					}
+
 				}
 			}
 		}
@@ -3728,15 +3771,25 @@ class ModuleSpanningTree : public Module
 				if (!CheckDupe)
 				{
 					WriteServ(user->fd,"NOTICE %s :*** CONNECT: Connecting to server: \002%s\002 (%s:%d)",user->nick,x->Name.c_str(),(x->HiddenFromStats ? "<hidden>" : x->IPAddr.c_str()),x->Port);
-					TreeSocket* newsocket = new TreeSocket(x->IPAddr,x->Port,false,10,x->Name.c_str());
-					if (newsocket->GetFd() > -1)
+					insp_inaddr binip;
+
+					if (insp_aton(x->IPAddr.c_str(), &binip) > 0)
 					{
-						Srv->AddSocket(newsocket);
+						TreeSocket* newsocket = new TreeSocket(x->IPAddr,x->Port,false,10,x->Name.c_str());
+						if (newsocket->GetFd() > -1)
+						{
+							Srv->AddSocket(newsocket);
+						}
+						else
+						{
+							WriteOpers("*** CONNECT: Error connecting \002%s\002: %s.",x->Name.c_str(),strerror(errno));
+							delete newsocket;
+						}
 					}
 					else
 					{
-						WriteServ(user->fd,"NOTICE %s :*** CONNECT: Error connecting \002%s\002: %s.",user->nick,x->Name.c_str(),strerror(errno));
-						DELETE(newsocket);
+						ServernameResolver* snr = new ServernameResolver(x->IPAddr, *x);
+						Srv->AddResolver(snr);
 					}
 					return 1;
 				}