From: brain <brain@e03df62e-2008-0410-955e-edbf42e46eb7>
Date: Sat, 11 Nov 2006 14:44:46 +0000 (+0000)
Subject: Support hybrid-style port ranges in spanningtree, too
X-Git-Tag: v2.0.23~6689
X-Git-Url: https://git.netwichtig.de/gitweb/?a=commitdiff_plain;h=b6df59028d6a1351be5cc9c75a931c60d734b6c5;p=user%2Fhenk%2Fcode%2Finspircd.git

Support hybrid-style port ranges in spanningtree, too


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

diff --git a/src/modules/m_spanningtree.cpp b/src/modules/m_spanningtree.cpp
index 92c95c525..7f854809b 100644
--- a/src/modules/m_spanningtree.cpp
+++ b/src/modules/m_spanningtree.cpp
@@ -3871,27 +3871,71 @@ void SpanningTreeUtilities::ReadConfiguration(bool rebind)
 		{
 			std::string Type = Conf->ReadValue("bind","type",j);
 			std::string IP = Conf->ReadValue("bind","address",j);
-			int Port = Conf->ReadInteger("bind","port",j,true);
+			std::string Port = Conf->ReadValue("bind","port",j);
 			if (Type == "servers")
 			{
-				ServerInstance->Log(DEBUG,"m_spanningtree: Binding server port %s:%d", IP.c_str(), Port);
-				if (IP == "*")
+				irc::commasepstream portrange(Port);
+				std::string portno = "*";
+				while ((portno = portrange.GetToken()) != "")
 				{
-					IP = "";
-				}
-				TreeSocket* listener = new TreeSocket(this, ServerInstance, IP.c_str(),Port,true,10);
-				if (listener->GetState() == I_LISTENING)
-				{
-					ServerInstance->Log(DEFAULT,"m_spanningtree: Binding server port %s:%d successful!", IP.c_str(), Port);
-					Bindings.push_back(listener);
-				}
-				else
-				{
-					ServerInstance->Log(DEFAULT,"m_spanningtree: Warning: Failed to bind server port %d",Port);
-					listener->Close();
-					DELETE(listener);
+					std::string::size_type dash = portno.rfind('-');
+					if (dash != std::string::npos)
+					{
+						std::string sbegin = portno.substr(0, dash);
+						std::string send = portno.substr(dash+1, portno.length());
+						long begin = atoi(sbegin.c_str());
+						long end = atoi(send.c_str());
+
+						if ((begin < 0) || (end < 0) || (begin > 65535) || (end > 65535) || (begin >= end))
+						{
+							ServerInstance->Log(DEFAULT,"WARNING: Port range \"%d-%d\" discarded. begin >= end, or begin/end out of range.", begin, end);
+						}
+						else
+						{
+							for (long port = begin; port <= end; ++port)
+							{
+								ServerInstance->Log(DEBUG,"m_spanningtree: Binding server port %s:%d (part of range %s)", IP.c_str(), port, portno.c_str());
+								if (IP == "*")
+									IP = "";
+
+								TreeSocket* listener = new TreeSocket(this, ServerInstance, IP.c_str(), port, true, 10);
+								if (listener->GetState() == I_LISTENING)
+								{
+									ServerInstance->Log(DEFAULT,"m_spanningtree: Binding server port %s:%d successful!", IP.c_str(), port);
+									Bindings.push_back(listener);
+								}
+								else
+								{
+									ServerInstance->Log(DEFAULT,"m_spanningtree: Warning: Failed to bind server port %s:%d (forms part of range '%s')",IP.c_str(),
+											port, portno.c_str());
+									listener->Close();
+									DELETE(listener);
+								}
+								ServerInstance->Log(DEBUG,"Done with this binding");
+							}
+						}
+					}
+					else
+					{
+						ServerInstance->Log(DEBUG,"m_spanningtree: Binding server port %s:%s (single port %s)", IP.c_str(), portno.c_str(), portno.c_str());
+						if (IP == "*")
+							IP = "";
+
+						TreeSocket* listener = new TreeSocket(this, ServerInstance, IP.c_str(),atoi(portno.c_str()),true,10);
+						if (listener->GetState() == I_LISTENING)
+						{
+							ServerInstance->Log(DEFAULT,"m_spanningtree: Binding server port %s:%s successful!", IP.c_str(), portno.c_str());
+							Bindings.push_back(listener);
+						}
+						else
+						{
+							ServerInstance->Log(DEFAULT,"m_spanningtree: Warning: Failed to bind server port %s:%s",IP.c_str(), portno.c_str());
+							listener->Close();
+							DELETE(listener);
+						}
+						ServerInstance->Log(DEBUG,"Done with this binding");
+					}
 				}
-				ServerInstance->Log(DEBUG,"Done with this binding");
 			}
 		}
 	}