From 36cb60f872712e8e5a59df74b5b36177addc4248 Mon Sep 17 00:00:00 2001 From: danieldg Date: Mon, 17 Aug 2009 15:09:46 +0000 Subject: Allow forward linking compatability by accepting higher protocol version numbers (other server should fail the link if it becomes link-incompatable) git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@11531 e03df62e-2008-0410-955e-edbf42e46eb7 --- src/modules/m_spanningtree/capab.cpp | 22 ++++++++++++---------- src/modules/m_spanningtree/main.h | 1 + 2 files changed, 13 insertions(+), 10 deletions(-) diff --git a/src/modules/m_spanningtree/capab.cpp b/src/modules/m_spanningtree/capab.cpp index d363f31c1..7981bc2a5 100644 --- a/src/modules/m_spanningtree/capab.cpp +++ b/src/modules/m_spanningtree/capab.cpp @@ -178,12 +178,19 @@ bool TreeSocket::Capab(const std::deque ¶ms) 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 ¶ms) 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::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 3935c3dd6..4e7a8a426 100644 --- 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 */ -- cgit v1.2.3