X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;ds=sidebyside;f=src%2Fsocket.cpp;h=ccae48a6d7cc654e2f7db2f063e5191f4dbd3595;hb=0ed3ef9dee2cb8f4b0c37da218a2fd1eee8c31f6;hp=e32bda90168358ac2d09c66d7b94c0c26cb2069f;hpb=06e0d5c7f0f78c489090fe5f0c18a91e8cb41ae8;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/socket.cpp b/src/socket.cpp
index e32bda901..ccae48a6d 100644
--- a/src/socket.cpp
+++ b/src/socket.cpp
@@ -27,37 +27,20 @@ bool InspIRCd::BindSocket(int sockfd, int port, const char* addr, bool dolisten)
 	sockaddrs servaddr;
 	int ret;
 
-	if (*addr == '*')
-		addr = "";
+	if (*addr == '*' || *addr == '\0')
+		addr = NULL;
 
-	if (*addr)
-	{
-		irc::sockets::aptosa(addr, port, &servaddr);
-	}
-	else
+	if (port == -1 && !addr)
 	{
+		/* Port -1: Means UDP IPV4 port binding - Special case
+		 * used by DNS engine.
+		 */
 		memset(&servaddr, 0, sizeof(servaddr));
-		if (port == -1)
-		{
-			/* Port -1: Means UDP IPV4 port binding - Special case
-			 * used by DNS engine.
-			 */
-			servaddr.in4.sin_family = AF_INET;
-		}
-		else
-		{
-			/* No address */
-#ifdef IPV6
-			/* Default to ipv6 bind to all */
-			servaddr.in6.sin6_family = AF_INET6;
-			servaddr.in6.sin6_port = htons(port);
-#else
-			/* Bind ipv4 to all */
-			servaddr.in4.sin_family = AF_INET;
-			servaddr.in4.sin_port = htons(port);
-#endif
-		}
+		servaddr.in4.sin_family = AF_INET;
 	}
+	else if (!irc::sockets::aptosa(addr, port, &servaddr))
+		return false;
+
 	ret = SE->Bind(sockfd, &servaddr.sa, sa_size(servaddr));
 
 	if (ret < 0)
@@ -128,12 +111,7 @@ int InspIRCd::BindPorts(FailedPortList &failed_ports)
 {
 	char configToken[MAXBUF], Addr[MAXBUF], Type[MAXBUF], Desc[MAXBUF];
 	int bound = 0;
-	bool started_with_nothing = (ports.size() == 0);
-	std::vector<std::pair<std::string, int> > old_ports;
-
-	/* XXX: Make a copy of the old ip/port pairs here */
-	for (std::vector<ListenSocketBase *>::iterator o = ports.begin(); o != ports.end(); ++o)
-		old_ports.push_back(make_pair((*o)->GetIP(), (*o)->GetPort()));
+	std::vector<ListenSocketBase*> old_ports(ports.begin(), ports.end());
 
 	for (int count = 0; count < Config->ConfValueEnum("bind"); count++)
 	{
@@ -154,21 +132,18 @@ int InspIRCd::BindPorts(FailedPortList &failed_ports)
 				if (*Addr == '*')
 					*Addr = 0;
 
+				irc::sockets::sockaddrs bindspec;
+				irc::sockets::aptosa(Addr, portno, &bindspec);
+				std::string bind_readable = irc::sockets::satouser(&bindspec);
+
 				bool skip = false;
-				for (std::vector<ListenSocketBase *>::iterator n = ports.begin(); n != ports.end(); ++n)
+				for (std::vector<ListenSocketBase *>::iterator n = old_ports.begin(); n != old_ports.end(); ++n)
 				{
-					if (((*n)->GetIP() == Addr) && ((*n)->GetPort() == portno))
+					if ((*n)->GetBindDesc() == bind_readable)
 					{
 						skip = true;
-						/* XXX: Here, erase from our copy of the list */
-						for (std::vector<std::pair<std::string, int> >::iterator k = old_ports.begin(); k != old_ports.end(); ++k)
-						{
-							if ((k->first == Addr) && (k->second == portno))
-							{
-								old_ports.erase(k);
-								break;
-							}
-						}
+						old_ports.erase(n);
+						break;
 					}
 				}
 				if (!skip)
@@ -182,29 +157,30 @@ int InspIRCd::BindPorts(FailedPortList &failed_ports)
 					}
 					else
 					{
-						failed_ports.push_back(std::make_pair((*Addr ? Addr : "*") + std::string(":") + ConvToStr(portno), strerror(errno)));
+						failed_ports.push_back(std::make_pair(bind_readable, strerror(errno)));
 					}
 				}
 			}
 		}
 	}
 
-	/* XXX: Here, anything left in our copy list, close as removed */
-	if (!started_with_nothing)
+	std::vector<ListenSocketBase *>::iterator n = ports.begin();
+	for (std::vector<ListenSocketBase *>::iterator o = old_ports.begin(); o != old_ports.end(); ++o)
 	{
-		for (size_t k = 0; k < old_ports.size(); ++k)
+		while (n != ports.end() && *n != *o)
+			n++;
+		if (n == ports.end())
 		{
-			for (std::vector<ListenSocketBase *>::iterator n = ports.begin(); n != ports.end(); ++n)
-			{
-				if (((*n)->GetIP() == old_ports[k].first) && ((*n)->GetPort() == old_ports[k].second))
-				{
-					this->Logs->Log("SOCKET",DEFAULT,"Port binding %s:%d was removed from the config file, closing.", old_ports[k].first.c_str(), old_ports[k].second);
-					delete *n;
-					ports.erase(n);
-					break;
-				}
-			}
+			this->Logs->Log("SOCKET",ERROR,"Port bindings slipped out of vector, aborting close!");
+			break;
 		}
+
+		this->Logs->Log("SOCKET",DEFAULT, "Port binding %s was removed from the config file, closing.",
+			(*n)->GetBindDesc().c_str());
+		delete *n;
+
+		// this keeps the iterator valid, pointing to the next element
+		n = ports.erase(n);
 	}
 
 	return bound;