Allow forward linking compatability by accepting higher protocol version numbers...

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

diff --git a/src/modules/m_spanningtree/capab.cpp b/src/modules/m_spanningtree/capab.cpp
index d363f31c1af4353f0ba0c59669dddac4206990b5..7981bc2a5ca7ad3a7541116463fd788878c50d86 100644 (file)
--- a/src/modules/m_spanningtree/capab.cpp
+++ b/src/modules/m_spanningtree/capab.cpp
@@ -178,12 +178,19 @@ bool TreeSocket::Capab(const std::deque<std::string> &params)
                        reason = "We don't both support linking to IPV6 servers";
                if (((this->CapKeys.find("IP6NATIVE") != this->CapKeys.end()) && (this->CapKeys.find("IP6NATIVE")->second == "1")) && (!ip6support))
                        reason = "The remote server is IPV6 native, and we don't support linking to IPV6 servers";
-               if (((this->CapKeys.find("PROTOCOL") == this->CapKeys.end()) || ((this->CapKeys.find("PROTOCOL") != this->CapKeys.end()) && (this->CapKeys.find("PROTOCOL")->second != ConvToStr(ProtocolVersion)))))
+               if (this->CapKeys.find("PROTOCOL") == this->CapKeys.end())
                {
-                       if (this->CapKeys.find("PROTOCOL") != this->CapKeys.end())
-                               reason = "Mismatched protocol versions "+this->CapKeys.find("PROTOCOL")->second+" and "+ConvToStr(ProtocolVersion);
-                       else
-                               reason = "Protocol version not specified";
+                       reason = "Protocol version not specified";
+               }
+               else
+               {
+                       int otherProto = atoi(CapKeys.find("PROTOCOL")->second.c_str());
+                       if (otherProto < MinCompatProtocol)
+                       {
+                               reason = "Server is using protocol version " + ConvToStr(otherProto) +
+                                       " which is too old to link with this server (version " + ConvToStr(ProtocolVersion)
+                                       + (ProtocolVersion != MinCompatProtocol ? ", links with " + ConvToStr(MinCompatProtocol) + " and above)" : ")");
+                       }
                }
 
                if(this->CapKeys.find("PREFIX") != this->CapKeys.end() && this->CapKeys.find("PREFIX")->second != this->ServerInstance->Modes->BuildPrefixes())
@@ -196,11 +203,6 @@ bool TreeSocket::Capab(const std::deque<std::string> &params)
                        reason = "One or more of the user modes on the remote server are invalid on this server.";
 
 
-               if (((this->CapKeys.find("HALFOP") == this->CapKeys.end()) && (ServerInstance->Config->AllowHalfop)) || ((this->CapKeys.find("HALFOP") != this->CapKeys.end()) && (this->CapKeys.find("HALFOP")->second != ConvToStr(ServerInstance->Config->AllowHalfop))))
-                       reason = "We don't both have halfop support enabled/disabled identically";
-
-
-
                /* Challenge response, store their challenge for our password */
                std::map<std::string,std::string>::iterator n = this->CapKeys.find("CHALLENGE");
                if (Utils->ChallengeResponse && (n != this->CapKeys.end()) && (ServerInstance->Modules->Find("m_sha256.so")))

diff --git a/src/modules/m_spanningtree/main.h b/src/modules/m_spanningtree/main.h
index 3935c3dd6e0e9b5db35700f537839f7ede81bfa6..4e7a8a426dd917ccc5e30a0bcb845540c23716d1 100644 (file)
--- a/src/modules/m_spanningtree/main.h
+++ b/src/modules/m_spanningtree/main.h
@@ -27,6 +27,7 @@
  * painful death by pain. You have been warned.
  */
 const long ProtocolVersion = 1201;
+const long MinCompatProtocol = 1201;
 
 /** Forward declarations
  */