Fix for bug #792 reported by recyclebin, do not attempt to ping local servers which...

[user/henk/code/inspircd.git] / src / modules / m_spanningtree / treeserver.cpp

diff --git a/src/modules/m_spanningtree/treeserver.cpp b/src/modules/m_spanningtree/treeserver.cpp
index c3167b9b31a8bd9499901ccda8db398dc752c9a1..455b49c846030a174746ffbc40b29e7365aaaaf7 100644 (file)
--- a/src/modules/m_spanningtree/treeserver.cpp
+++ b/src/modules/m_spanningtree/treeserver.cpp
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
+ * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
@@ -32,7 +32,7 @@ TreeServer::TreeServer(SpanningTreeUtilities* Util, InspIRCd* Instance, const st
        VersionString.clear();
        ServerUserCount = ServerOperCount = 0;
        StartBurst = rtt = 0;
-       Warned = Hidden = DupError = false;
+       Warned = Hidden = false;
        VersionString = ServerInstance->GetVersionString();
        SetID(id);
 }
@@ -52,7 +52,7 @@ TreeServer::TreeServer(SpanningTreeUtilities* Util, InspIRCd* Instance, std::str
        Route = NULL;
        Socket = NULL; /* Fix by brain */
        StartBurst = rtt = 0;
-       Warned = Hidden = DupError = false;
+       Warned = Hidden = false;
        AddHashEntry();
        SetID(id);
 }
@@ -67,9 +67,9 @@ TreeServer::TreeServer(SpanningTreeUtilities* Util, InspIRCd* Instance, std::str
        bursting = false;
        VersionString.clear();
        ServerUserCount = ServerOperCount = 0;
-       this->SetNextPingTime(time(NULL) + Utils->PingFreq);
+       this->SetNextPingTime(ServerInstance->Time() + Utils->PingFreq);
        this->SetPingFlag();
-       Warned = DupError = false;
+       Warned = false;
        StartBurst = rtt = 0;
 
        timeval t;
@@ -139,9 +139,19 @@ std::string& TreeServer::GetID()
        return sid;
 }
 
-void TreeServer::FinishBurst()
+void TreeServer::FinishBurstInternal()
 {
        this->bursting = false;
+       for(unsigned int q=0; q < ChildCount(); q++)
+       {
+               TreeServer* child = GetChild(q);
+               child->FinishBurstInternal();
+       }
+}
+
+void TreeServer::FinishBurst()
+{
+       FinishBurstInternal();
        ServerInstance->XLines->ApplyLines();
        timeval t;
        gettimeofday(&t, NULL);
@@ -156,16 +166,7 @@ void TreeServer::SetID(const std::string &id)
 {
        ServerInstance->Logs->Log("m_spanningtree",DEBUG, "Setting SID to " + id);
        sid = id;
-       server_hash::iterator iter = Utils->sidlist.find(sid);
-       if (iter == Utils->sidlist.end())
-               Utils->sidlist[sid] = this;
-       else
-               DupError = true;
-}
-
-bool TreeServer::DuplicateID()
-{
-       return DupError;
+       Utils->sidlist[sid] = this;
 }
 
 int TreeServer::QuitUsers(const std::string &reason)
@@ -369,5 +370,3 @@ TreeServer::~TreeServer()
        if (iter != Utils->sidlist.end())
                Utils->sidlist.erase(iter);
 }
-
-