]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_spanningtree.cpp
Fixed a crash when spanningtree is given a blank line; really simple fix, and I had...
[user/henk/code/inspircd.git] / src / modules / m_spanningtree.cpp
index 2592d2a8431d10dcc315b34db40515032381d101..60eb650dd17748aa078a723db55f2b78a62a1e66 100644 (file)
@@ -16,8 +16,6 @@
 
 /* $ModDesc: Povides a spanning tree server link protocol */
 
-using namespace std;
-
 #include "configreader.h"
 #include "users.h"
 #include "channels.h"
@@ -28,13 +26,18 @@ using namespace std;
 #include "inspircd.h"
 #include "wildcard.h"
 #include "xline.h"
-#include "cull_list.h"
 #include "aes.h"
 
 /** If you make a change which breaks the protocol, increment this.
  * If you  completely change the protocol, completely change the number.
+ *
+ * IMPORTANT: If you make changes, document your changes here, without fail:
+ * http://www.inspircd.org/wiki/List_of_protocol_changes_between_versions
+ *
+ * Failure to document your protocol changes will result in a painfully
+ * painful death by pain. You have been warned.
  */
-const long ProtocolVersion = 1101;
+const long ProtocolVersion = 1102;
 
 /*
  * The server list in InspIRCd is maintained as two structures
@@ -55,14 +58,7 @@ const long ProtocolVersion = 1101;
  * any O(n) lookups. If however, during a split or sync, we want
  * to apply an operation to a server, and any of its child objects
  * we can resort to recursion to walk the tree structure.
- */
-
-using irc::sockets::MatchCIDR;
-
-class ModuleSpanningTree;
-static ModuleSpanningTree* TreeProtocolModule;
-
-/** Any socket can have one of five states at any one time.
+ * Any socket can have one of five states at any one time.
  * The LISTENER state indicates a socket which is listening
  * for connections. It cannot receive data itself, only incoming
  * sockets.
@@ -82,15 +78,37 @@ enum ServerState { LISTENER, CONNECTING, WAIT_AUTH_1, WAIT_AUTH_2, CONNECTED };
 /* Foward declarations */
 class TreeServer;
 class TreeSocket;
+class Link;
+class ModuleSpanningTree;
 
 /* This hash_map holds the hash equivalent of the server
  * tree, used for rapid linear lookups.
  */
 typedef nspace::hash_map<std::string, TreeServer*, nspace::hash<string>, irc::StrHashComp> server_hash;
-server_hash serverlist;
 
-typedef nspace::hash_map<std::string, userrec*> uid_hash;
-typedef nspace::hash_map<std::string, char*> sid_hash;
+typedef std::map<TreeServer*,TreeServer*> TreeServerList;
+
+/** The Link class might as well be a struct,
+ * but this is C++ and we don't believe in structs (!).
+ * It holds the entire information of one <link>
+ * tag from the main config file. We maintain a list
+ * of them, and populate the list on rehash/load.
+ */
+class Link : public classbase
+{
+ public:
+       irc::string Name;
+       std::string IPAddr;
+       int Port;
+       std::string SendPass;
+       std::string RecvPass;
+       unsigned long AutoConnect;
+       time_t NextConnectTime;
+       std::string EncryptionKey;
+       bool HiddenFromStats;
+       std::string FailOver;
+       int Timeout;
+};
 
 /** Contains helper functions and variables for this module,
  * and keeps them out of the global namespace
@@ -98,84 +116,92 @@ typedef nspace::hash_map<std::string, char*> sid_hash;
 class SpanningTreeUtilities
 {
  private:
+       /** Creator server
+        */
        InspIRCd* ServerInstance;
  public:
-       bool FlatLinks; /* Flatten links and /MAP for non-opers */
-       bool HideULines; /* Hide U-Lined servers in /MAP and /LINKS */
-       bool AnnounceTSChange; /* Announce TS changes to channels on merge */
-       std::vector<TreeSocket*> Bindings; /* Socket bindings */
-       /* This variable represents the root of the server tree
-        * (for all intents and purposes, it's us)
+       /** Creator module
+        */
+       ModuleSpanningTree* Creator;
+       /** Flatten links and /MAP for non-opers
+        */
+       bool FlatLinks;
+       /** Hide U-Lined servers in /MAP and /LINKS
+        */
+       bool HideULines;
+       /** Announce TS changes to channels on merge
+        */
+       bool AnnounceTSChange;
+       /** Synchronize timestamps between servers
+        */
+       bool EnableTimeSync;
+       /** Socket bindings for listening sockets
+        */
+       std::vector<TreeSocket*> Bindings;
+       /** This variable represents the root of the server tree
         */
        TreeServer *TreeRoot;
+       /** IPs allowed to link to us
+        */
+       std::vector<std::string> ValidIPs;
+       /** Hash of currently connected servers by name
+        */
+       server_hash serverlist;
+       /** Holds the data from the <link> tags in the conf
+        */
+       std::vector<Link> LinkBlocks;
 
-       SpanningTreeUtilities(InspIRCd* Instance);
+       /** Initialise utility class
+        */
+       SpanningTreeUtilities(InspIRCd* Instance, ModuleSpanningTree* Creator);
+       /** Destroy class and free listeners etc
+        */
        ~SpanningTreeUtilities();
-       bool DoOneToOne(std::string prefix, std::string command, std::deque<std::string> &params, std::string target);
-       bool DoOneToAllButSender(std::string prefix, std::string command, std::deque<std::string> &params, std::string omit);
+       /** Send a message from this server to one other local or remote
+        */
+       bool DoOneToOne(const std::string &prefix, const std::string &command, std::deque<std::string> &params, std::string target);
+       /** Send a message from this server to all but one other, local or remote
+        */
+       bool DoOneToAllButSender(const std::string &prefix, const std::string &command, std::deque<std::string> &params, std::string omit);
+       /** Send a message from this server to all but one other, local or remote
+        */
        bool DoOneToAllButSender(const char* prefix, const char* command, std::deque<std::string> &params, std::string omit);
-       bool DoOneToMany(std::string prefix, std::string command, std::deque<std::string> &params);
+       /** Send a message from this server to all others
+        */
+       bool DoOneToMany(const std::string &prefix, const std::string &command, std::deque<std::string> &params);
+       /** Send a message from this server to all others
+        */
        bool DoOneToMany(const char* prefix, const char* command, std::deque<std::string> &params);
-       bool DoOneToAllButSenderRaw(std::string data, std::string omit, std::string prefix, irc::string command, std::deque<std::string> &params);
+       /** Send a message from this server to all others, without doing any processing on the command (e.g. send it as-is with colons and all)
+        */
+       bool DoOneToAllButSenderRaw(const std::string &data, const std::string &omit, const std::string &prefix, const irc::string &command, std::deque<std::string> &params);
+       /** Read the spanningtree module's tags from the config file
+        */
        void ReadConfiguration(bool rebind);
-       void AddThisServer(TreeServer* server, std::deque<TreeServer*> &list);
-       void GetListOfServersForChannel(chanrec* c, std::deque<TreeServer*> &list);
-       TreeServer* FindServer(std::string ServerName);
-       TreeServer* BestRouteTo(std::string ServerName);
-       TreeServer* FindServerMask(std::string ServerName);
-       bool IsServer(std::string ServerName);
-};
-
-
-std::vector<std::string> ValidIPs;
-
-/** This will be used in a future version of InspIRCd for UID support
- */
-class UserManager : public classbase
-{
-       uid_hash uids;
-       sid_hash sids;
- public:
-       UserManager()
-       {
-               uids.clear();
-               sids.clear();
-       }
-
-       std::string UserToUID(userrec* user)
-       {
-               return "";
-       }
-
-       std::string UIDToUser(const std::string &UID)
-       {
-               return "";
-       }
-
-       std::string CreateAndAdd(userrec* user)
-       {
-               return "";
-       }
-
-       std::string CreateAndAdd(const std::string &servername)
-       {
-               return "";
-       }
-
-       std::string ServerToSID(const std::string &servername)
-       {
-               return "";
-       }
-
-       std::string SIDToServer(const std::string &SID)
-       {
-               return "";
-       }
-
-       userrec* FindByID(const std::string &UID)
-       {
-               return NULL;
-       }
+       /** Add a server to the server list for GetListOfServersForChannel
+        */
+       void AddThisServer(TreeServer* server, TreeServerList &list);
+       /** Compile a list of servers which contain members of channel c
+        */
+       void GetListOfServersForChannel(chanrec* c, TreeServerList &list, char status, const CUList &exempt_list);
+       /** Find a server by name
+        */
+       TreeServer* FindServer(const std::string &ServerName);
+       /** Find a route to a server by name
+        */
+       TreeServer* BestRouteTo(const std::string &ServerName);
+       /** Find a server by glob mask
+        */
+       TreeServer* FindServerMask(const std::string &ServerName);
+       /** Returns true if this is a server name we recognise
+        */
+       bool IsServer(const std::string &ServerName);
+       /** Attempt to connect to the failover link of link x
+        */
+       void DoFailOver(Link* x);
+       /** Find a link tag from a server name
+        */
+       Link* FindLink(const std::string& name);
 };
 
 
@@ -196,7 +222,6 @@ class UserManager : public classbase
 class TreeServer : public classbase
 {
        InspIRCd* ServerInstance;               /* Creator */
-       SpanningTreeUtilities* Utils;           /* Utility class */
        TreeServer* Parent;                     /* Parent entry */
        TreeServer* Route;                      /* Route entry */
        std::vector<TreeServer*> Children;      /* List of child objects */
@@ -208,13 +233,14 @@ class TreeServer : public classbase
        TreeSocket* Socket;                     /* For directly connected servers this points at the socket object */
        time_t NextPing;                        /* After this time, the server should be PINGed*/
        bool LastPingWasGood;                   /* True if the server responded to the last PING with a PONG */
+       SpanningTreeUtilities* Utils;           /* Utility class */
        
  public:
 
        /** We don't use this constructor. Its a dummy, and won't cause any insertion
         * of the TreeServer into the hash_map. See below for the two we DO use.
         */
-       TreeServer(SpanningTreeUtilities* Utils, InspIRCd* Instance) : ServerInstance(Instance)
+       TreeServer(SpanningTreeUtilities* Util, InspIRCd* Instance) : ServerInstance(Instance), Utils(Util)
        {
                Parent = NULL;
                ServerName = "";
@@ -228,11 +254,12 @@ class TreeServer : public classbase
         * represents our own server. Therefore, it has no route, no parent, and
         * no socket associated with it. Its version string is our own local version.
         */
-       TreeServer(SpanningTreeUtilities* Utils, InspIRCd* Instance, std::string Name, std::string Desc) : ServerInstance(Instance), ServerName(Name.c_str()), ServerDesc(Desc)
+       TreeServer(SpanningTreeUtilities* Util, InspIRCd* Instance, std::string Name, std::string Desc) : ServerInstance(Instance), ServerName(Name.c_str()), ServerDesc(Desc), Utils(Util)
        {
                Parent = NULL;
                VersionString = "";
-               UserCount = OperCount = 0;
+               UserCount = ServerInstance->UserCount();
+               OperCount = ServerInstance->OperCount();
                VersionString = ServerInstance->GetVersionString();
                Route = NULL;
                Socket = NULL; /* Fix by brain */
@@ -243,8 +270,8 @@ class TreeServer : public classbase
         * This constructor initializes the server's Route and Parent, and sets up
         * its ping counters so that it will be pinged one minute from now.
         */
-       TreeServer(SpanningTreeUtilities* Utils, InspIRCd* Instance, std::string Name, std::string Desc, TreeServer* Above, TreeSocket* Sock)
-               : ServerInstance(Instance), Parent(Above), ServerName(Name.c_str()), ServerDesc(Desc), Socket(Sock)
+       TreeServer(SpanningTreeUtilities* Util, InspIRCd* Instance, std::string Name, std::string Desc, TreeServer* Above, TreeSocket* Sock)
+               : ServerInstance(Instance), Parent(Above), ServerName(Name.c_str()), ServerDesc(Desc), Socket(Sock), Utils(Util)
        {
                VersionString = "";
                UserCount = OperCount = 0;
@@ -333,10 +360,9 @@ class TreeServer : public classbase
         */
        void AddHashEntry()
        {
-               server_hash::iterator iter;
-               iter = serverlist.find(this->ServerName.c_str());
-               if (iter == serverlist.end())
-                       serverlist[this->ServerName.c_str()] = this;
+               server_hash::iterator iter = Utils->serverlist.find(this->ServerName.c_str());
+               if (iter == Utils->serverlist.end())
+                       Utils->serverlist[this->ServerName.c_str()] = this;
        }
 
        /** This method removes the reference to this object
@@ -345,10 +371,9 @@ class TreeServer : public classbase
         */
        void DelHashEntry()
        {
-               server_hash::iterator iter;
-               iter = serverlist.find(this->ServerName.c_str());
-               if (iter != serverlist.end())
-                       serverlist.erase(iter);
+               server_hash::iterator iter = Utils->serverlist.find(this->ServerName.c_str());
+               if (iter != Utils->serverlist.end())
+                       Utils->serverlist.erase(iter);
        }
 
        /** These accessors etc should be pretty self-
@@ -425,7 +450,7 @@ class TreeServer : public classbase
                return Parent;
        }
 
-       void SetVersion(std::string Version)
+       void SetVersion(const std::string &Version)
        {
                VersionString = Version;
        }
@@ -500,45 +525,13 @@ class TreeServer : public classbase
        }
 };
 
-/** The Link class might as well be a struct,
- * but this is C++ and we don't believe in structs (!).
- * It holds the entire information of one <link>
- * tag from the main config file. We maintain a list
- * of them, and populate the list on rehash/load.
- */
-class Link : public classbase
-{
- public:
-       irc::string Name;
-       std::string IPAddr;
-       int Port;
-       std::string SendPass;
-       std::string RecvPass;
-       unsigned long AutoConnect;
-       time_t NextConnectTime;
-       std::string EncryptionKey;
-       bool HiddenFromStats;
-       irc::string FailOver;
-};
-
-void DoFailOver(Link* x);
-Link* FindLink(const std::string& name);
-
-/* The usual stuff for inspircd modules,
- * plus the vector of Link classes which we
- * use to store the <link> tags from the config
- * file.
- */
-ConfigReader *Conf;
-std::vector<Link> LinkBlocks;
-
 /** Yay for fast searches!
  * This is hundreds of times faster than recursion
  * or even scanning a linked list, especially when
  * there are more than a few servers to deal with.
  * (read as: lots).
  */
-TreeServer* SpanningTreeUtilities::FindServer(std::string ServerName)
+TreeServer* SpanningTreeUtilities::FindServer(const std::string &ServerName)
 {
        server_hash::iterator iter;
        iter = serverlist.find(ServerName.c_str());
@@ -558,7 +551,7 @@ TreeServer* SpanningTreeUtilities::FindServer(std::string ServerName)
  * See the comments for the constructor of TreeServer
  * for more details.
  */
-TreeServer* SpanningTreeUtilities::BestRouteTo(std::string ServerName)
+TreeServer* SpanningTreeUtilities::BestRouteTo(const std::string &ServerName)
 {
        if (ServerName.c_str() == TreeRoot->GetName())
                return NULL;
@@ -579,7 +572,7 @@ TreeServer* SpanningTreeUtilities::BestRouteTo(std::string ServerName)
  * and match each one until we get a hit. Yes its slow,
  * deal with it.
  */
-TreeServer* SpanningTreeUtilities::FindServerMask(std::string ServerName)
+TreeServer* SpanningTreeUtilities::FindServerMask(const std::string &ServerName)
 {
        for (server_hash::iterator i = serverlist.begin(); i != serverlist.end(); i++)
        {
@@ -590,7 +583,7 @@ TreeServer* SpanningTreeUtilities::FindServerMask(std::string ServerName)
 }
 
 /* A convenient wrapper that returns true if a server exists */
-bool SpanningTreeUtilities::IsServer(std::string ServerName)
+bool SpanningTreeUtilities::IsServer(const std::string &ServerName)
 {
        return (FindServer(ServerName) != NULL);
 }
@@ -746,7 +739,7 @@ class TreeSocket : public InspSocket
                if (this->LinkState == CONNECTING)
                {
                        /* we do not need to change state here. */
-                       for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
+                       for (std::vector<Link>::iterator x = Utils->LinkBlocks.begin(); x < Utils->LinkBlocks.end(); x++)
                        {
                                if (x->Name == this->myhost)
                                {
@@ -788,9 +781,9 @@ class TreeSocket : public InspSocket
                if (e == I_ERR_CONNECT)
                {
                        this->Instance->SNO->WriteToSnoMask('l',"Connection failed: Connection to \002"+myhost+"\002 refused");
-                       Link* MyLink = FindLink(myhost);
+                       Link* MyLink = Utils->FindLink(myhost);
                        if (MyLink)
-                               DoFailOver(MyLink);
+                               Utils->DoFailOver(MyLink);
                }
        }
 
@@ -918,7 +911,7 @@ class TreeSocket : public InspSocket
                return result;
        }
 
-       bool Capab(std::deque<std::string> params)
+       bool Capab(const std::deque<std::string> &params)
        {
                if (params.size() < 1)
                {
@@ -1071,10 +1064,13 @@ class TreeSocket : public InspSocket
         * does some validation first and passes on the SQUIT to all
         * other remaining servers.
         */
-       void Squit(TreeServer* Current,std::string reason)
+       void Squit(TreeServer* Current, const std::string &reason)
        {
                if ((Current) && (Current != Utils->TreeRoot))
                {
+                       Event rmode((char*)Current->GetName().c_str(), (Module*)Utils->Creator, "lost_server");
+                       rmode.Send(Instance);
+
                        std::deque<std::string> params;
                        params.push_back(Current->GetName());
                        params.push_back(":"+reason);
@@ -1103,7 +1099,7 @@ class TreeSocket : public InspSocket
        }
 
        /** FMODE command - server mode with timestamp checks */
-       bool ForceMode(std::string source, std::deque<std::string> &params)
+       bool ForceMode(const std::string &source, std::deque<std::string> &params)
        {
                /* Chances are this is a 1.0 FMODE without TS */
                if (params.size() < 3)
@@ -1491,7 +1487,7 @@ class TreeSocket : public InspSocket
        }
 
        /** FTOPIC command */
-       bool ForceTopic(std::string source, std::deque<std::string> &params)
+       bool ForceTopic(const std::string &source, std::deque<std::string> &params)
        {
                if (params.size() != 4)
                        return true;
@@ -1535,7 +1531,7 @@ class TreeSocket : public InspSocket
        }
 
        /** FJOIN, similar to TS6 SJOIN, but not quite. */
-       bool ForceJoin(std::string source, std::deque<std::string> &params)
+       bool ForceJoin(const std::string &source, std::deque<std::string> &params)
        {
                /* 1.1 FJOIN works as follows:
                 *
@@ -1596,6 +1592,7 @@ class TreeSocket : public InspSocket
                userrec* who = NULL;                    /* User we are currently checking */
                std::string channel = params[0];        /* Channel name, as a string */
                time_t TS = atoi(params[1].c_str());    /* Timestamp given to us for remote side */
+               bool created = false;
                
                /* Try and find the channel */
                chanrec* chan = this->Instance->FindChan(channel);
@@ -1606,11 +1603,13 @@ class TreeSocket : public InspSocket
                /* default TS is a high value, which if we dont have this
                 * channel will let the other side apply their modes.
                 */
-               time_t ourTS = time(NULL)+600;
+               time_t ourTS = Instance->Time(true)+600;
 
                /* Does this channel exist? if it does, get its REAL timestamp */
                if (chan)
                        ourTS = chan->age;
+               else
+                       created = true; /* don't perform deops, and set TS to correct time after processing. */
 
                /* In 1.1, if they have the newer channel, we immediately clear
                 * all status modes from our users. We then accept their modes.
@@ -1622,18 +1621,19 @@ class TreeSocket : public InspSocket
                {
                        std::deque<std::string> param_list;
 
-                       if (chan)
-                               chan->age = TS;
-
                        /* Lower the TS here */
                        if (Utils->AnnounceTSChange && chan)
                                chan->WriteChannelWithServ(Instance->Config->ServerName,
-                               "TS for %s changed from %lu to %lu", chan->name, ourTS, TS);
+                               "NOTICE %s :TS for %s changed from %lu to %lu", chan->name, chan->name, ourTS, TS);
                        ourTS = TS;
-
                        param_list.push_back(channel);
-                       /* Zap all the privilage modes on our side */
-                       this->RemoveStatus(Instance->Config->ServerName, param_list);
+
+                       /* Zap all the privilage modes on our side, if the channel exists here */
+                       if (!created)
+                       {
+                               this->RemoveStatus(Instance->Config->ServerName, param_list);
+                               chan->age = TS;
+                       }
                }
 
                /* Put the final parameter of the FJOIN into a tokenstream ready to split it */
@@ -1806,6 +1806,20 @@ class TreeSocket : public InspSocket
                                free(mode_users[f]);
                }
 
+               /* if we newly created the channel, set it's TS properly. */
+               if (created)
+               {
+                       /* find created channel .. */
+                       chan = this->Instance->FindChan(channel);
+                       if (chan)
+                               /* w00t said this shouldnt be needed but it is.
+                                * This isnt strictly true, as chan can be NULL
+                                * if a nick collision has occured and therefore
+                                * the channel was never created.
+                                */
+                               chan->age = TS;
+               }
+
                /* All done. That wasnt so bad was it, you can wipe
                 * the sweat from your forehead now. :-)
                 */
@@ -1813,7 +1827,7 @@ class TreeSocket : public InspSocket
        }
 
        /** NICK command */
-       bool IntroduceClient(std::string source, std::deque<std::string> &params)
+       bool IntroduceClient(const std::string &source, std::deque<std::string> &params)
        {
                if (params.size() < 8)
                        return true;
@@ -1893,6 +1907,8 @@ class TreeSocket : public InspSocket
         */
        void SendFJoins(TreeServer* Current, chanrec* c)
        {
+               std::string buffer;
+
                Instance->Log(DEBUG,"Sending FJOINs to other server for %s",c->name);
                char list[MAXBUF];
                std::string individual_halfops = std::string(":")+this->Instance->Config->ServerName+" FMODE "+c->name+" "+ConvToStr(c->age);
@@ -1903,8 +1919,6 @@ class TreeSocket : public InspSocket
                char* ptr = list + dlen;
 
                CUList *ulist = c->GetUsers();
-               std::vector<userrec*> specific_halfop;
-               std::vector<userrec*> specific_voice;
                std::string modes = "";
                std::string params = "";
 
@@ -1920,7 +1934,8 @@ class TreeSocket : public InspSocket
 
                        if (curlen > (480-NICKMAX))
                        {
-                               this->WriteLine(list);
+                               buffer.append(list).append("\r\n");
+
                                dlen = curlen = snprintf(list,MAXBUF,":%s FJOIN %s %lu",this->Instance->Config->ServerName,c->name,(unsigned long)c->age);
                                ptr = list + dlen;
                                ptrlen = 0;
@@ -1929,64 +1944,102 @@ class TreeSocket : public InspSocket
                }
 
                if (numusers)
-                       this->WriteLine(list);
+                       buffer.append(list).append("\r\n");
 
-                for (BanList::iterator b = c->bans.begin(); b != c->bans.end(); b++)
-                {
+               /* Sorry for the hax. Because newly created channels assume +nt,
+                * if this channel doesnt have +nt, explicitly send -n and -t for the missing modes.
+                */
+               bool inverted = false;
+               if (!c->IsModeSet('n'))
+               {
+                       modes.append("-n");
+                       inverted = true;
+               }
+               if (!c->IsModeSet('t'))
+               {
+                       modes.append("-t");
+                       inverted = true;
+               }
+               if (inverted)
+               {
+                       modes.append("+");
+               }
+
+               for (BanList::iterator b = c->bans.begin(); b != c->bans.end(); b++)
+               {
                        modes.append("b");
-                       params.append(b->data).append(" ");
-                }
-               this->WriteLine(std::string(":")+this->Instance->Config->ServerName+" FMODE "+c->name+" "+ConvToStr(c->age)+" +"+c->ChanModes(true)+modes+" "+params);
+                       params.append(" ").append(b->data);
+
+                       if (params.length() >= MAXMODES)
+                       {
+                               /* Wrap at MAXMODES */
+                               buffer.append(":").append(this->Instance->Config->ServerName).append(" FMODE ").append(c->name).append(" ").append(ConvToStr(c->age)).append(" +").append(modes).append(params).append("\r\n");
+                               modes = "";
+                               params = "";
+                       }
+               }
+
+               buffer.append(":").append(this->Instance->Config->ServerName).append(" FMODE ").append(c->name).append(" ").append(ConvToStr(c->age)).append(" +").append(c->ChanModes(true));
+
+               /* Only send these if there are any */
+               if (!modes.empty())
+                       buffer.append("\r\n").append(":").append(this->Instance->Config->ServerName).append(" FMODE ").append(c->name).append(" ").append(ConvToStr(c->age)).append(" +").append(modes).append(params);
+
+               this->WriteLine(buffer);
        }
 
        /** Send G, Q, Z and E lines */
        void SendXLines(TreeServer* Current)
        {
                char data[MAXBUF];
+               std::string buffer;
                std::string n = this->Instance->Config->ServerName;
                const char* sn = n.c_str();
                int iterations = 0;
                /* Yes, these arent too nice looking, but they get the job done */
                for (std::vector<ZLine*>::iterator i = Instance->XLines->zlines.begin(); i != Instance->XLines->zlines.end(); i++, iterations++)
                {
-                       snprintf(data,MAXBUF,":%s ADDLINE Z %s %s %lu %lu :%s",sn,(*i)->ipaddr,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
-                       this->WriteLine(data);
+                       snprintf(data,MAXBUF,":%s ADDLINE Z %s %s %lu %lu :%s\r\n",sn,(*i)->ipaddr,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
+                       buffer.append(data);
                }
                for (std::vector<QLine*>::iterator i = Instance->XLines->qlines.begin(); i != Instance->XLines->qlines.end(); i++, iterations++)
                {
-                       snprintf(data,MAXBUF,":%s ADDLINE Q %s %s %lu %lu :%s",sn,(*i)->nick,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
-                       this->WriteLine(data);
+                       snprintf(data,MAXBUF,":%s ADDLINE Q %s %s %lu %lu :%s\r\n",sn,(*i)->nick,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
+                       buffer.append(data);
                }
                for (std::vector<GLine*>::iterator i = Instance->XLines->glines.begin(); i != Instance->XLines->glines.end(); i++, iterations++)
                {
-                       snprintf(data,MAXBUF,":%s ADDLINE G %s %s %lu %lu :%s",sn,(*i)->hostmask,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
-                       this->WriteLine(data);
+                       snprintf(data,MAXBUF,":%s ADDLINE G %s@%s %s %lu %lu :%s\r\n",sn,(*i)->identmask,(*i)->hostmask,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
+                       buffer.append(data);
                }
                for (std::vector<ELine*>::iterator i = Instance->XLines->elines.begin(); i != Instance->XLines->elines.end(); i++, iterations++)
                {
-                       snprintf(data,MAXBUF,":%s ADDLINE E %s %s %lu %lu :%s",sn,(*i)->hostmask,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
-                       this->WriteLine(data);
+                       snprintf(data,MAXBUF,":%s ADDLINE E %s@%s %s %lu %lu :%s\r\n",sn,(*i)->identmask,(*i)->hostmask,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
+                       buffer.append(data);
                }
                for (std::vector<ZLine*>::iterator i = Instance->XLines->pzlines.begin(); i != Instance->XLines->pzlines.end(); i++, iterations++)
                {
-                       snprintf(data,MAXBUF,":%s ADDLINE Z %s %s %lu %lu :%s",sn,(*i)->ipaddr,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
-                       this->WriteLine(data);
+                       snprintf(data,MAXBUF,":%s ADDLINE Z %s %s %lu %lu :%s\r\n",sn,(*i)->ipaddr,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
+                       buffer.append(data);
                }
                for (std::vector<QLine*>::iterator i = Instance->XLines->pqlines.begin(); i != Instance->XLines->pqlines.end(); i++, iterations++)
                {
-                       snprintf(data,MAXBUF,":%s ADDLINE Q %s %s %lu %lu :%s",sn,(*i)->nick,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
-                       this->WriteLine(data);
+                       snprintf(data,MAXBUF,":%s ADDLINE Q %s %s %lu %lu :%s\r\n",sn,(*i)->nick,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
+                       buffer.append(data);
                }
                for (std::vector<GLine*>::iterator i = Instance->XLines->pglines.begin(); i != Instance->XLines->pglines.end(); i++, iterations++)
                {
-                       snprintf(data,MAXBUF,":%s ADDLINE G %s %s %lu %lu :%s",sn,(*i)->hostmask,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
-                       this->WriteLine(data);
+                       snprintf(data,MAXBUF,":%s ADDLINE G %s@%s %s %lu %lu :%s\r\n",sn,(*i)->identmask,(*i)->hostmask,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
+                       buffer.append(data);
                }
                for (std::vector<ELine*>::iterator i = Instance->XLines->pelines.begin(); i != Instance->XLines->pelines.end(); i++, iterations++)
                {
-                       snprintf(data,MAXBUF,":%s ADDLINE E %s %s %lu %lu :%s",sn,(*i)->hostmask,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
-                       this->WriteLine(data);
+                       snprintf(data,MAXBUF,":%s ADDLINE E %s@%s %s %lu %lu :%s\r\n",sn,(*i)->identmask,(*i)->hostmask,(*i)->source,(unsigned long)(*i)->set_time,(unsigned long)(*i)->duration,(*i)->reason);
+                       buffer.append(data);
                }
+
+               if (!buffer.empty())
+                       this->WriteLine(buffer);
        }
 
        /** Send channel modes and topics */
@@ -2005,12 +2058,12 @@ class TreeSocket : public InspSocket
                                snprintf(data,MAXBUF,":%s FTOPIC %s %lu %s :%s",sn,c->second->name,(unsigned long)c->second->topicset,c->second->setby,c->second->topic);
                                this->WriteLine(data);
                        }
-                       FOREACH_MOD_I(this->Instance,I_OnSyncChannel,OnSyncChannel(c->second,(Module*)TreeProtocolModule,(void*)this));
+                       FOREACH_MOD_I(this->Instance,I_OnSyncChannel,OnSyncChannel(c->second,(Module*)Utils->Creator,(void*)this));
                        list.clear();
                        c->second->GetExtList(list);
                        for (unsigned int j = 0; j < list.size(); j++)
                        {
-                               FOREACH_MOD_I(this->Instance,I_OnSyncChannelMetaData,OnSyncChannelMetaData(c->second,(Module*)TreeProtocolModule,(void*)this,list[j]));
+                               FOREACH_MOD_I(this->Instance,I_OnSyncChannelMetaData,OnSyncChannelMetaData(c->second,(Module*)Utils->Creator,(void*)this,list[j]));
                        }
                }
        }
@@ -2020,6 +2073,7 @@ class TreeSocket : public InspSocket
        {
                char data[MAXBUF];
                std::deque<std::string> list;
+               std::string dataline;
                int iterations = 0;
                for (user_hash::iterator u = this->Instance->clientlist.begin(); u != this->Instance->clientlist.end(); u++, iterations++)
                {
@@ -2029,18 +2083,20 @@ class TreeSocket : public InspSocket
                                this->WriteLine(data);
                                if (*u->second->oper)
                                {
-                                       this->WriteLine(":"+std::string(u->second->nick)+" OPERTYPE "+std::string(u->second->oper));
+                                       snprintf(data,MAXBUF,":%s OPERTYPE %s", u->second->nick, u->second->oper);
+                                       this->WriteLine(data);
                                }
                                if (*u->second->awaymsg)
                                {
-                                       this->WriteLine(":"+std::string(u->second->nick)+" AWAY :"+std::string(u->second->awaymsg));
+                                       snprintf(data,MAXBUF,":%s AWAY :%s", u->second->nick, u->second->awaymsg);
+                                       this->WriteLine(data);
                                }
-                               FOREACH_MOD_I(this->Instance,I_OnSyncUser,OnSyncUser(u->second,(Module*)TreeProtocolModule,(void*)this));
+                               FOREACH_MOD_I(this->Instance,I_OnSyncUser,OnSyncUser(u->second,(Module*)Utils->Creator,(void*)this));
                                list.clear();
                                u->second->GetExtList(list);
                                for (unsigned int j = 0; j < list.size(); j++)
                                {
-                                       FOREACH_MOD_I(this->Instance,I_OnSyncUserMetaData,OnSyncUserMetaData(u->second,(Module*)TreeProtocolModule,(void*)this,list[j]));
+                                       FOREACH_MOD_I(this->Instance,I_OnSyncUserMetaData,OnSyncUserMetaData(u->second,(Module*)Utils->Creator,(void*)this,list[j]));
                                }
                        }
                }
@@ -2053,7 +2109,7 @@ class TreeSocket : public InspSocket
         */
        void DoBurst(TreeServer* s)
        {
-               std::string burst = "BURST "+ConvToStr(time(NULL));
+               std::string burst = "BURST "+ConvToStr(Instance->Time(true));
                std::string endburst = "ENDBURST";
                // Because by the end of the netburst, it  could be gone!
                std::string name = s->GetName();
@@ -2068,7 +2124,7 @@ class TreeSocket : public InspSocket
                /* Send everything else (channel modes, xlines etc) */
                this->SendChannelModes(s);
                this->SendXLines(s);            
-               FOREACH_MOD_I(this->Instance,I_OnSyncOtherMetaData,OnSyncOtherMetaData((Module*)TreeProtocolModule,(void*)this));
+               FOREACH_MOD_I(this->Instance,I_OnSyncOtherMetaData,OnSyncOtherMetaData((Module*)Utils->Creator,(void*)this));
                this->WriteLine(endburst);
                this->Instance->SNO->WriteToSnoMask('l',"Finished bursting to \2"+name+"\2.");
        }
@@ -2095,6 +2151,10 @@ class TreeSocket : public InspSocket
                        {
                                std::string ret = in_buffer.substr(0,in_buffer.find("\n")-1);
                                in_buffer = in_buffer.substr(in_buffer.find("\n")+1,in_buffer.length()-in_buffer.find("\n"));
+                               /* Use rfind here not find, as theres more
+                                * chance of the \r being near the end of the
+                                * string, not the start.
+                                */
                                if (ret.find("\r") != std::string::npos)
                                        ret = in_buffer.substr(0,in_buffer.find("\r")-1);
                                /* Process this one, abort if it
@@ -2114,14 +2174,22 @@ class TreeSocket : public InspSocket
                                                {
                                                        ctx_in->Decrypt(out, result, nbytes, 0);
                                                        for (int t = 0; t < nbytes; t++)
-                                                               if (result[t] == '\7') result[t] = 0;
+                                                       {
+                                                               if (result[t] == '\7')
+                                                               {
+                                                                       /* We only need to stick a \0 on the
+                                                                        * first \7, the rest will be lost
+                                                                        */
+                                                                       result[t] = 0;
+                                                                       break;
+                                                               }
+                                                       }
                                                        ret = result;
                                                }
                                        }
                                }
                                if (!this->ProcessLine(ret))
                                {
-                                       Instance->Log(DEBUG,"ProcessLine says no!");
                                        return false;
                                }
                        }
@@ -2152,7 +2220,8 @@ class TreeSocket : public InspSocket
                        to64frombits((unsigned char*)result64,(unsigned char*)result,ll);
                        line = result64;
                }
-               return this->Write(line + "\r\n");
+               line.append("\r\n");
+               return this->Write(line);
        }
 
        /* Handle ERROR command */
@@ -2166,7 +2235,7 @@ class TreeSocket : public InspSocket
        }
 
        /** remote MOTD. leet, huh? */
-       bool Motd(std::string prefix, std::deque<std::string> &params)
+       bool Motd(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() > 0)
                {
@@ -2214,7 +2283,7 @@ class TreeSocket : public InspSocket
        }
 
        /** remote ADMIN. leet, huh? */
-       bool Admin(std::string prefix, std::deque<std::string> &params)
+       bool Admin(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() > 0)
                {
@@ -2254,7 +2323,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool Stats(std::string prefix, std::deque<std::string> &params)
+       bool Stats(const std::string &prefix, std::deque<std::string> &params)
        {
                /* Get the reply to a STATS query if it matches this servername,
                 * and send it back as a load of PUSH queries
@@ -2294,7 +2363,7 @@ class TreeSocket : public InspSocket
        /** Because the core won't let users or even SERVERS set +o,
         * we use the OPERTYPE command to do this.
         */
-       bool OperType(std::string prefix, std::deque<std::string> &params)
+       bool OperType(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() != 1)
                {
@@ -2315,7 +2384,7 @@ class TreeSocket : public InspSocket
        /** Because Andy insists that services-compatible servers must
         * implement SVSNICK and SVSJOIN, that's exactly what we do :p
         */
-       bool ForceNick(std::string prefix, std::deque<std::string> &params)
+       bool ForceNick(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 3)
                        return true;
@@ -2343,7 +2412,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool ServiceJoin(std::string prefix, std::deque<std::string> &params)
+       bool ServiceJoin(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 2)
                        return true;
@@ -2358,7 +2427,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool RemoteRehash(std::string prefix, std::deque<std::string> &params)
+       bool RemoteRehash(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 1)
                        return false;
@@ -2376,7 +2445,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool RemoteKill(std::string prefix, std::deque<std::string> &params)
+       bool RemoteKill(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() != 2)
                        return true;
@@ -2406,7 +2475,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool LocalPong(std::string prefix, std::deque<std::string> &params)
+       bool LocalPong(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 1)
                        return true;
@@ -2447,7 +2516,7 @@ class TreeSocket : public InspSocket
                return true;
        }
        
-       bool MetaData(std::string prefix, std::deque<std::string> &params)
+       bool MetaData(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 3)
                        return true;
@@ -2483,7 +2552,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool ServerVersion(std::string prefix, std::deque<std::string> &params)
+       bool ServerVersion(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 1)
                        return true;
@@ -2499,7 +2568,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool ChangeHost(std::string prefix, std::deque<std::string> &params)
+       bool ChangeHost(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 1)
                        return true;
@@ -2514,7 +2583,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool AddLine(std::string prefix, std::deque<std::string> &params)
+       bool AddLine(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 6)
                        return true;
@@ -2571,7 +2640,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool ChangeName(std::string prefix, std::deque<std::string> &params)
+       bool ChangeName(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 1)
                        return true;
@@ -2587,7 +2656,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool Whois(std::string prefix, std::deque<std::string> &params)
+       bool Whois(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 1)
                        return true;
@@ -2609,7 +2678,7 @@ class TreeSocket : public InspSocket
                                        char idle[MAXBUF];
 
                                        snprintf(signon,MAXBUF,"%lu",(unsigned long)x->signon);
-                                       snprintf(idle,MAXBUF,"%lu",(unsigned long)abs((x->idle_lastmsg)-time(NULL)));
+                                       snprintf(idle,MAXBUF,"%lu",(unsigned long)abs((x->idle_lastmsg)-Instance->Time(true)));
                                        std::deque<std::string> par;
                                        par.push_back(prefix);
                                        par.push_back(signon);
@@ -2646,7 +2715,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool Push(std::string prefix, std::deque<std::string> &params)
+       bool Push(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 2)
                        return true;
@@ -2669,7 +2738,46 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool Time(std::string prefix, std::deque<std::string> &params)
+       bool HandleSetTime(const std::string &prefix, std::deque<std::string> &params)
+       {
+               if (!params.size() || !Utils->EnableTimeSync)
+                       return true;
+               
+               bool force = false;
+               
+               if ((params.size() == 2) && (params[1] == "FORCE"))
+                       force = true;
+               
+               time_t rts = atoi(params[0].c_str());
+               time_t us = Instance->Time(true);
+               
+               if (rts == us)
+               {
+                       Instance->Log(DEBUG, "Timestamp from %s is equal", prefix.c_str());
+                       
+                       Utils->DoOneToAllButSender(prefix, "TIMESET", params, prefix);
+               }
+               else if (force || (rts < us))
+               {
+                       int old = Instance->SetTimeDelta(rts - us);
+                       Instance->Log(DEBUG, "%s TS (diff %d) from %s applied (old delta was %d)", (force) ? "Forced" : "Lower", rts - us, prefix.c_str(), old);
+                       
+                       Utils->DoOneToAllButSender(prefix, "TIMESET", params, prefix);
+               }
+               else
+               {
+                       Instance->Log(DEBUG, "Higher TS (diff %d) from %s overridden", us - rts, prefix.c_str());
+                       
+                       std::deque<std::string> oparams;
+                       oparams.push_back(ConvToStr(us));
+                       
+                       Utils->DoOneToMany(prefix, "TIMESET", oparams);
+               }
+               
+               return true;
+       }
+
+       bool Time(const std::string &prefix, std::deque<std::string> &params)
        {
                // :source.server TIME remote.server sendernick
                // :remote.server TIME source.server sendernick TS
@@ -2681,9 +2789,7 @@ class TreeSocket : public InspSocket
                                userrec* u = this->Instance->FindNick(params[1]);
                                if (u)
                                {
-                                       char curtime[256];
-                                       snprintf(curtime,256,"%lu",(unsigned long)time(NULL));
-                                       params.push_back(curtime);
+                                       params.push_back(ConvToStr(Instance->Time(false)));
                                        params[0] = prefix;
                                        Utils->DoOneToOne(this->Instance->Config->ServerName,"TIME",params,params[0]);
                                }
@@ -2719,7 +2825,7 @@ class TreeSocket : public InspSocket
                return true;
        }
        
-       bool LocalPing(std::string prefix, std::deque<std::string> &params)
+       bool LocalPing(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 1)
                        return true;
@@ -2749,7 +2855,7 @@ class TreeSocket : public InspSocket
                }
        }
 
-       bool RemoveStatus(std::string prefix, std::deque<std::string> &params)
+       bool RemoveStatus(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 1)
                        return true;
@@ -2807,7 +2913,7 @@ class TreeSocket : public InspSocket
                return true;
        }
 
-       bool RemoteServer(std::string prefix, std::deque<std::string> &params)
+       bool RemoteServer(const std::string &prefix, std::deque<std::string> &params)
        {
                if (params.size() < 4)
                        return false;
@@ -2855,7 +2961,7 @@ class TreeSocket : public InspSocket
                        return false;
                }
                std::string description = params[3];
-               for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
+               for (std::vector<Link>::iterator x = Utils->LinkBlocks.begin(); x < Utils->LinkBlocks.end(); x++)
                {
                        if ((x->Name == servername) && (x->RecvPass == password))
                        {
@@ -2905,7 +3011,7 @@ class TreeSocket : public InspSocket
                        return false;
                }
                std::string description = params[3];
-               for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
+               for (std::vector<Link>::iterator x = Utils->LinkBlocks.begin(); x < Utils->LinkBlocks.end(); x++)
                {
                        if ((x->Name == servername) && (x->RecvPass == password))
                        {
@@ -2942,7 +3048,7 @@ class TreeSocket : public InspSocket
                return false;
        }
 
-       void Split(std::string line, std::deque<std::string> &n)
+       void Split(const std::string &line, std::deque<std::string> &n)
        {
                n.clear();
                irc::tokenstream tokens(line);
@@ -2952,17 +3058,17 @@ class TreeSocket : public InspSocket
                return;
        }
 
-       bool ProcessLine(std::string line)
+       bool ProcessLine(std::string &line)
        {
                std::deque<std::string> params;
                irc::string command;
                std::string prefix;
                
+               line = line.substr(0, line.find_first_of("\r\n"));
+               
                if (line.empty())
                        return true;
                
-               line = line.substr(0, line.find_first_of("\r\n"));
-               
                Instance->Log(DEBUG,"IN: %s", line.c_str());
                
                this->Split(line.c_str(),params);
@@ -2979,7 +3085,7 @@ class TreeSocket : public InspSocket
                if ((!this->ctx_in) && (command == "AES"))
                {
                        std::string sserv = params[0];
-                       for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
+                       for (std::vector<Link>::iterator x = Utils->LinkBlocks.begin(); x < Utils->LinkBlocks.end(); x++)
                        {
                                if ((x->EncryptionKey != "") && (x->Name == sserv))
                                {
@@ -3054,20 +3160,40 @@ class TreeSocket : public InspSocket
                                }
                                else if (command == "BURST")
                                {
-                                       if (params.size())
+                                       if (params.size() && Utils->EnableTimeSync)
                                        {
-                                               /* If a time stamp is provided, try and check syncronization */
-                                               time_t THEM = atoi(params[0].c_str());
-                                               long delta = THEM-time(NULL);
+                                               /* If a time stamp is provided, apply synchronization */
+                                               bool force = false;
+                                               time_t them = atoi(params[0].c_str());
+                                               time_t us = Instance->Time(true);
+                                               int delta = them - us;
+
+                                               if ((params.size() == 2) && (params[1] == "FORCE"))
+                                                       force = true;
+
                                                if ((delta < -600) || (delta > 600))
                                                {
                                                        this->Instance->SNO->WriteToSnoMask('l',"\2ERROR\2: Your clocks are out by %d seconds (this is more than ten minutes). Link aborted, \2PLEASE SYNC YOUR CLOCKS!\2",abs(delta));
                                                        this->WriteLine("ERROR :Your clocks are out by "+ConvToStr(abs(delta))+" seconds (this is more than ten minutes). Link aborted, PLEASE SYNC YOUR CLOCKS!");
                                                        return false;
                                                }
-                                               else if ((delta < -60) || (delta > 60))
+                                               
+                                               if (us == them)
+                                               {
+                                                       this->Instance->Log(DEBUG, "Timestamps are equal; pat yourself on the back");
+                                               }
+                                               else if (force || (us > them))
+                                               {
+                                                       this->Instance->Log(DEBUG, "Remote server has lower TS (%d seconds)", them - us);
+                                                       this->Instance->SetTimeDelta(them - us);
+                                                       // Send this new timestamp to any other servers
+                                                       Utils->DoOneToMany(Utils->TreeRoot->GetName(), "TIMESET", params);
+                                               }
+                                               else
                                                {
-                                                       this->Instance->SNO->WriteToSnoMask('l',"\2WARNING\2: Your clocks are out by %d seconds, please consider synching your clocks.",abs(delta));
+                                                       // Override the timestamp
+                                                       this->Instance->Log(DEBUG, "We have a higher timestamp (by %d seconds), not updating delta", us - them);
+                                                       this->WriteLine(":" + Utils->TreeRoot->GetName() + " TIMESET " + ConvToStr(us));
                                                }
                                        }
                                        this->LinkState = CONNECTED;
@@ -3276,6 +3402,10 @@ class TreeSocket : public InspSocket
                                {
                                        return this->Push(prefix,params);
                                }
+                               else if (command == "TIMESET")
+                               {
+                                       return this->HandleSetTime(prefix, params);
+                               }
                                else if (command == "TIME")
                                {
                                        return this->Time(prefix,params);
@@ -3316,6 +3446,42 @@ class TreeSocket : public InspSocket
                                        }
                                        return true;
                                }
+                               else if (command == "OPERNOTICE")
+                               {
+                                       std::string sourceserv = this->myhost;
+
+                                       if (this->InboundServerName != "")
+                                               sourceserv = this->InboundServerName;
+
+                                       if (params.size() >= 1)
+                                               Instance->WriteOpers("*** From " + sourceserv + ": " + params[0]);
+
+                                       return Utils->DoOneToAllButSenderRaw(line, sourceserv, prefix, command, params);
+                               }
+                               else if (command == "MODENOTICE")
+                               {
+                                       std::string sourceserv = this->myhost;
+                                       if (this->InboundServerName != "")
+                                               sourceserv = this->InboundServerName;
+                                       if (params.size() >= 2)
+                                       {
+                                               Instance->WriteMode(params[0].c_str(), WM_AND, "*** From %s: %s", sourceserv.c_str(), params[1].c_str());
+                                       }
+
+                                       return Utils->DoOneToAllButSenderRaw(line, sourceserv, prefix, command, params);
+                               }
+                               else if (command == "SNONOTICE")
+                               {
+                                       std::string sourceserv = this->myhost;
+                                       if (this->InboundServerName != "")
+                                               sourceserv = this->InboundServerName;
+                                       if (params.size() >= 2)
+                                       {
+                                               Instance->SNO->WriteToSnoMask(*(params[0].c_str()), "From " + sourceserv + ": "+ params[1]);
+                                       }
+
+                                       return Utils->DoOneToAllButSenderRaw(line, sourceserv, prefix, command, params);
+                               }
                                else if (command == "ENDBURST")
                                {
                                        this->bursting = false;
@@ -3326,6 +3492,10 @@ class TreeSocket : public InspSocket
                                                sourceserv = this->InboundServerName;
                                        }
                                        this->Instance->SNO->WriteToSnoMask('l',"Received end of netburst from \2%s\2",sourceserv.c_str());
+
+                                       Event rmode((char*)sourceserv.c_str(), (Module*)Utils->Creator, "new_server");
+                                       rmode.Send(Instance);
+
                                        return true;
                                }
                                else
@@ -3339,6 +3509,25 @@ class TreeSocket : public InspSocket
                                        {
                                                sourceserv = this->InboundServerName;
                                        }
+                                       if ((!who) && (command == "MODE"))
+                                       {
+                                               if (Utils->IsServer(prefix))
+                                               {
+                                                       const char* modelist[127];
+                                                       for (size_t i = 0; i < params.size(); i++)
+                                                               modelist[i] = params[i].c_str();
+
+                                                       userrec* fake = new userrec(Instance);
+                                                       fake->SetFd(FD_MAGIC_NUMBER);
+
+                                                       this->Instance->SendMode(modelist, params.size(), fake);
+       
+                                                       delete fake;
+
+                                                       /* Hot potato! pass it on! */
+                                                       return Utils->DoOneToAllButSenderRaw(line,sourceserv,prefix,command,params);
+                                               }
+                                       }
                                        if (who)
                                        {
                                                if ((command == "NICK") && (params.size() > 0))
@@ -3425,9 +3614,9 @@ class TreeSocket : public InspSocket
                if (this->LinkState == CONNECTING)
                {
                        this->Instance->SNO->WriteToSnoMask('l',"CONNECT: Connection to \002"+myhost+"\002 timed out.");
-                       Link* MyLink = FindLink(myhost);
+                       Link* MyLink = Utils->FindLink(myhost);
                        if (MyLink)
-                               DoFailOver(MyLink);
+                               Utils->DoFailOver(MyLink);
                }
        }
 
@@ -3446,7 +3635,9 @@ class TreeSocket : public InspSocket
                {
                        Squit(s,"Remote host closed the connection");
                }
-               this->Instance->SNO->WriteToSnoMask('l',"Connection to '\2%s\2' failed.",quitserver.c_str());
+
+               if (quitserver != "")
+                       this->Instance->SNO->WriteToSnoMask('l',"Connection to '\2%s\2' failed.",quitserver.c_str());
        }
 
        virtual int OnIncomingConnection(int newsock, char* ip)
@@ -3457,11 +3648,11 @@ class TreeSocket : public InspSocket
                 */
                bool found = false;
 
-               found = (std::find(ValidIPs.begin(), ValidIPs.end(), ip) != ValidIPs.end());
+               found = (std::find(Utils->ValidIPs.begin(), Utils->ValidIPs.end(), ip) != Utils->ValidIPs.end());
                if (!found)
                {
-                       for (vector<std::string>::iterator i = ValidIPs.begin(); i != ValidIPs.end(); i++)
-                               if (MatchCIDR(ip, (*i).c_str()))
+                       for (vector<std::string>::iterator i = Utils->ValidIPs.begin(); i != Utils->ValidIPs.end(); i++)
+                               if (irc::sockets::MatchCIDR(ip, (*i).c_str()))
                                        found = true;
 
                        if (!found)
@@ -3493,7 +3684,7 @@ class ServernameResolver : public Resolver
        Link MyLink;
        SpanningTreeUtilities* Utils;
  public: 
-       ServernameResolver(SpanningTreeUtilities* Util, InspIRCd* Instance, const std::string &hostname, Link x) : Resolver(Instance, hostname, DNS_QUERY_FORWARD), MyLink(x), Utils(Util)
+       ServernameResolver(Module* me, SpanningTreeUtilities* Util, InspIRCd* Instance, const std::string &hostname, Link x) : Resolver(Instance, hostname, DNS_QUERY_FORWARD, me), MyLink(x), Utils(Util)
        {
                /* Nothing in here, folks */
        }
@@ -3507,7 +3698,7 @@ class ServernameResolver : public Resolver
                TreeServer* CheckDupe = Utils->FindServer(MyLink.Name.c_str());
                if (!CheckDupe) /* Check that nobody tried to connect it successfully while we were resolving */
                {
-                       TreeSocket* newsocket = new TreeSocket(this->Utils, ServerInstance, result,MyLink.Port,false,10,MyLink.Name.c_str());
+                       TreeSocket* newsocket = new TreeSocket(this->Utils, ServerInstance, result,MyLink.Port,false,MyLink.Timeout ? MyLink.Timeout : 10,MyLink.Name.c_str());
                        if (newsocket->GetFd() > -1)
                        {
                                /* We're all OK */
@@ -3517,7 +3708,7 @@ class ServernameResolver : public Resolver
                                /* Something barfed, show the opers */
                                ServerInstance->SNO->WriteToSnoMask('l',"CONNECT: Error connecting \002%s\002: %s.",MyLink.Name.c_str(),strerror(errno));
                                delete newsocket;
-                               DoFailOver(&MyLink);
+                               Utils->DoFailOver(&MyLink);
                        }
                }
        }
@@ -3526,7 +3717,7 @@ class ServernameResolver : public Resolver
        {
                /* Ooops! */
                ServerInstance->SNO->WriteToSnoMask('l',"CONNECT: Error connecting \002%s\002: Unable to resolve hostname - %s",MyLink.Name.c_str(),errormessage.c_str());
-               DoFailOver(&MyLink);
+               Utils->DoFailOver(&MyLink);
        }
 };
 
@@ -3536,15 +3727,16 @@ class SecurityIPResolver : public Resolver
 {
  private:
        Link MyLink;
+       SpanningTreeUtilities* Utils;
  public:
-       SecurityIPResolver(InspIRCd* Instance, const std::string &hostname, Link x) : Resolver(Instance, hostname, DNS_QUERY_FORWARD), MyLink(x)
+       SecurityIPResolver(Module* me, SpanningTreeUtilities* U, InspIRCd* Instance, const std::string &hostname, Link x) : Resolver(Instance, hostname, DNS_QUERY_FORWARD, me), MyLink(x), Utils(U)
        {
        }
 
        void OnLookupComplete(const std::string &result)
        {
                ServerInstance->Log(DEBUG,"Security IP cache: Adding IP address '%s' for Link '%s'",result.c_str(),MyLink.Name.c_str());
-               ValidIPs.push_back(result);
+               Utils->ValidIPs.push_back(result);
        }
 
        void OnError(ResolverError e, const std::string &errormessage)
@@ -3553,11 +3745,11 @@ class SecurityIPResolver : public Resolver
        }
 };
 
-SpanningTreeUtilities::SpanningTreeUtilities(InspIRCd* Instance) : ServerInstance(Instance)
+SpanningTreeUtilities::SpanningTreeUtilities(InspIRCd* Instance, ModuleSpanningTree* C) : ServerInstance(Instance), Creator(C)
 {
        Bindings.clear();
-       this->TreeRoot = new TreeServer(this, ServerInstance, ServerInstance->Config->ServerName, ServerInstance->Config->ServerDesc);
        this->ReadConfiguration(true);
+       this->TreeRoot = new TreeServer(this, ServerInstance, ServerInstance->Config->ServerName, ServerInstance->Config->ServerDesc);
 }
 
 SpanningTreeUtilities::~SpanningTreeUtilities()
@@ -3582,27 +3774,37 @@ SpanningTreeUtilities::~SpanningTreeUtilities()
                        DELETE(sock);
                }
        }
+       delete TreeRoot;
 }
 
-void SpanningTreeUtilities::AddThisServer(TreeServer* server, std::deque<TreeServer*> &list)
+void SpanningTreeUtilities::AddThisServer(TreeServer* server, TreeServerList &list)
 {
-       for (unsigned int c = 0; c < list.size(); c++)
-       {
-               if (list[c] == server)
-               {
-                       return;
-               }
-       }
-       list.push_back(server);
+       if (list.find(server) == list.end())
+               list[server] = server;
 }
 
 /** returns a list of DIRECT servernames for a specific channel */
-void SpanningTreeUtilities::GetListOfServersForChannel(chanrec* c, std::deque<TreeServer*> &list)
+void SpanningTreeUtilities::GetListOfServersForChannel(chanrec* c, TreeServerList &list, char status, const CUList &exempt_list)
 {
-       CUList *ulist = c->GetUsers();
+       CUList *ulist;
+       switch (status)
+       {
+               case '@':
+                       ulist = c->GetOppedUsers();
+               break;
+               case '%':
+                       ulist = c->GetHalfoppedUsers();
+               break;
+               case '+':
+                       ulist = c->GetVoicedUsers();
+               break;
+               default:
+                       ulist = c->GetUsers();
+               break;
+       }
        for (CUList::iterator i = ulist->begin(); i != ulist->end(); i++)
        {
-               if (i->second->GetFd() < 0)
+               if ((i->second->GetFd() < 0) && (exempt_list.find(i->second) == exempt_list.end()))
                {
                        TreeServer* best = this->BestRouteTo(i->second->server);
                        if (best)
@@ -3612,8 +3814,9 @@ void SpanningTreeUtilities::GetListOfServersForChannel(chanrec* c, std::deque<Tr
        return;
 }
 
-bool SpanningTreeUtilities::DoOneToAllButSenderRaw(std::string data, std::string omit, std::string prefix, irc::string command, std::deque<std::string> &params)
+bool SpanningTreeUtilities::DoOneToAllButSenderRaw(const std::string &data, const std::string &omit, const std::string &prefix, const irc::string &command, std::deque<std::string> &params)
 {
+       char pfx = 0;
        TreeServer* omitroute = this->BestRouteTo(omit);
        if ((command == "NOTICE") || (command == "PRIVMSG"))
        {
@@ -3622,6 +3825,7 @@ bool SpanningTreeUtilities::DoOneToAllButSenderRaw(std::string data, std::string
                        /* Prefixes */
                        if ((*(params[0].c_str()) == '@') || (*(params[0].c_str()) == '%') || (*(params[0].c_str()) == '+'))
                        {
+                               pfx = params[0][0];
                                params[0] = params[0].substr(1, params[0].length()-1);
                        }
                        if ((*(params[0].c_str()) != '#') && (*(params[0].c_str()) != '$'))
@@ -3648,15 +3852,18 @@ bool SpanningTreeUtilities::DoOneToAllButSenderRaw(std::string data, std::string
                        else
                        {
                                chanrec* c = ServerInstance->FindChan(params[0]);
-                               if (c)
+                               userrec* u = ServerInstance->FindNick(prefix);
+                               if (c && u)
                                {
-                                       std::deque<TreeServer*> list;
-                                       GetListOfServersForChannel(c,list);
-                                       unsigned int lsize = list.size();
-                                       for (unsigned int i = 0; i < lsize; i++)
+                                       CUList elist;
+                                       TreeServerList list;
+                                       FOREACH_MOD(I_OnBuildExemptList, OnBuildExemptList((command == "PRIVMSG" ? MSG_PRIVMSG : MSG_NOTICE), c, u, pfx, elist));
+                                       GetListOfServersForChannel(c,list,pfx,elist);
+
+                                       for (TreeServerList::iterator i = list.begin(); i != list.end(); i++)
                                        {
-                                               TreeSocket* Sock = list[i]->GetSocket();
-                                               if ((Sock) && (list[i]->GetName() != omit) && (omitroute != list[i]))
+                                               TreeSocket* Sock = i->second->GetSocket();
+                                               if ((Sock) && (i->second->GetName() != omit) && (omitroute != i->second))
                                                {
                                                        Sock->WriteLine(data);
                                                }
@@ -3680,7 +3887,7 @@ bool SpanningTreeUtilities::DoOneToAllButSenderRaw(std::string data, std::string
        return true;
 }
 
-bool SpanningTreeUtilities::DoOneToAllButSender(std::string prefix, std::string command, std::deque<std::string> &params, std::string omit)
+bool SpanningTreeUtilities::DoOneToAllButSender(const std::string &prefix, const std::string &command, std::deque<std::string> &params, std::string omit)
 {
        TreeServer* omitroute = this->BestRouteTo(omit);
        std::string FullLine = ":" + prefix + " " + command;
@@ -3707,7 +3914,7 @@ bool SpanningTreeUtilities::DoOneToAllButSender(std::string prefix, std::string
        return true;
 }
 
-bool SpanningTreeUtilities::DoOneToMany(std::string prefix, std::string command, std::deque<std::string> &params)
+bool SpanningTreeUtilities::DoOneToMany(const std::string &prefix, const std::string &command, std::deque<std::string> &params)
 {
        std::string FullLine = ":" + prefix + " " + command;
        unsigned int words = params.size();
@@ -3743,7 +3950,7 @@ bool SpanningTreeUtilities::DoOneToAllButSender(const char* prefix, const char*
        return this->DoOneToAllButSender(spfx, scmd, params, omit);
 }
        
-bool SpanningTreeUtilities::DoOneToOne(std::string prefix, std::string command, std::deque<std::string> &params, std::string target)
+bool SpanningTreeUtilities::DoOneToOne(const std::string &prefix, const std::string &command, std::deque<std::string> &params, std::string target)
 {
        TreeServer* Route = this->BestRouteTo(target);
        if (Route)
@@ -3770,40 +3977,45 @@ bool SpanningTreeUtilities::DoOneToOne(std::string prefix, std::string command,
 
 void SpanningTreeUtilities::ReadConfiguration(bool rebind)
 {
-       Conf = new ConfigReader(ServerInstance);
+       ConfigReader* Conf = new ConfigReader(ServerInstance);
        if (rebind)
        {
                for (int j =0; j < Conf->Enumerate("bind"); j++)
                {
                        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 == "*")
-                               {
-                                       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
+                               irc::portparser portrange(Port, false);
+                               int portno = -1;
+                               while ((portno = portrange.GetToken()))
                                {
-                                       ServerInstance->Log(DEFAULT,"m_spanningtree: Warning: Failed to bind server port %d",Port);
-                                       listener->Close();
-                                       DELETE(listener);
+                                       ServerInstance->Log(DEBUG,"m_spanningtree: Binding server port %s:%d", IP.c_str(), portno);
+                                       if (IP == "*")
+                                               IP = "";
+
+                                       TreeSocket* listener = new TreeSocket(this, ServerInstance, IP.c_str(), portno, true, 10);
+                                       if (listener->GetState() == I_LISTENING)
+                                       {
+                                               ServerInstance->Log(DEFAULT,"m_spanningtree: Binding server port %s:%d successful!", IP.c_str(), portno);
+                                               Bindings.push_back(listener);
+                                       }
+                                       else
+                                       {
+                                               ServerInstance->Log(DEFAULT,"m_spanningtree: Warning: Failed to bind server port %s:%d",IP.c_str(), portno);
+                                               listener->Close();
+                                               DELETE(listener);
+                                       }
+                                       ServerInstance->Log(DEBUG,"Done with this binding");
                                }
-                               ServerInstance->Log(DEBUG,"Done with this binding");
                        }
                }
        }
        FlatLinks = Conf->ReadFlag("options","flatlinks",0);
        HideULines = Conf->ReadFlag("options","hideulines",0);
        AnnounceTSChange = Conf->ReadFlag("options","announcets",0);
+       EnableTimeSync = !(Conf->ReadFlag("options","notimesync",0));
        LinkBlocks.clear();
        ValidIPs.clear();
        for (int j =0; j < Conf->Enumerate("link"); j++)
@@ -3819,6 +4031,7 @@ void SpanningTreeUtilities::ReadConfiguration(bool rebind)
                L.AutoConnect = Conf->ReadInteger("link","autoconnect",j,true);
                L.EncryptionKey =  Conf->ReadValue("link","encryptionkey",j);
                L.HiddenFromStats = Conf->ReadFlag("link","hidden",j);
+               L.Timeout = Conf->ReadInteger("link","timeout",j,true);
                L.NextConnectTime = time(NULL) + L.AutoConnect;
                /* Bugfix by brain, do not allow people to enter bad configurations */
                if (L.Name != ServerInstance->Config->ServerName)
@@ -3836,7 +4049,7 @@ void SpanningTreeUtilities::ReadConfiguration(bool rebind)
                                {
                                        try
                                        {
-                                               SecurityIPResolver* sr = new SecurityIPResolver(ServerInstance, L.IPAddr, L);
+                                               SecurityIPResolver* sr = new SecurityIPResolver((Module*)this->Creator, this, ServerInstance, L.IPAddr, L);
                                                ServerInstance->AddResolver(sr);
                                        }
                                        catch (ModuleException& e)
@@ -3880,7 +4093,19 @@ void SpanningTreeUtilities::ReadConfiguration(bool rebind)
        DELETE(Conf);
 }
 
-
+/** To create a timer which recurs every second, we inherit from InspTimer.
+ * InspTimer is only one-shot however, so at the end of each Tick() we simply
+ * insert another of ourselves into the pending queue :)
+ */
+class TimeSyncTimer : public InspTimer
+{
+ private:
+       InspIRCd *Instance;
+       ModuleSpanningTree *Module;
+ public:
+       TimeSyncTimer(InspIRCd *Instance, ModuleSpanningTree *Mod);
+       virtual void Tick(time_t TIME);
+};
 
 class ModuleSpanningTree : public Module
 {
@@ -3892,14 +4117,23 @@ class ModuleSpanningTree : public Module
        SpanningTreeUtilities* Utils;
 
  public:
+       TimeSyncTimer *SyncTimer;
 
        ModuleSpanningTree(InspIRCd* Me)
                : Module::Module(Me), max_local(0), max_global(0)
        {
-               Utils = new SpanningTreeUtilities(Me);
+               Utils = new SpanningTreeUtilities(Me, this);
 
                command_rconnect = new cmd_rconnect(ServerInstance, this, Utils);
                ServerInstance->AddCommand(command_rconnect);
+
+               if (Utils->EnableTimeSync)
+               {
+                       SyncTimer = new TimeSyncTimer(ServerInstance, this);
+                       ServerInstance->Timers->AddTimer(SyncTimer);
+               }
+               else
+                       SyncTimer = NULL;
        }
 
        void ShowLinks(TreeServer* Current, userrec* user, int hops)
@@ -3936,7 +4170,7 @@ class ModuleSpanningTree : public Module
 
        int CountServs()
        {
-               return serverlist.size();
+               return Utils->serverlist.size();
        }
 
        void HandleLinks(const char** parameters, int pcnt, userrec* user)
@@ -3956,14 +4190,33 @@ class ModuleSpanningTree : public Module
                if (n_users > max_global)
                        max_global = n_users;
 
-               user->WriteServ("251 %s :There are %d users and %d invisible on %d servers",user->nick,n_users-ServerInstance->InvisibleUserCount(),ServerInstance->InvisibleUserCount(),this->CountServs());
+               unsigned int ulined_count = 0;
+               unsigned int ulined_local_count = 0;
+
+               /* If ulined are hidden and we're not an oper, count the number of ulined servers hidden,
+                * locally and globally (locally means directly connected to us)
+                */
+               if ((Utils->HideULines) && (!*user->oper))
+               {
+                       for (server_hash::iterator q = Utils->serverlist.begin(); q != Utils->serverlist.end(); q++)
+                       {
+                               if (ServerInstance->ULine(q->second->GetName().c_str()))
+                               {
+                                       ulined_count++;
+                                       if (q->second->GetParent() == Utils->TreeRoot)
+                                               ulined_local_count++;
+                               }
+                       }
+               }
+
+               user->WriteServ("251 %s :There are %d users and %d invisible on %d servers",user->nick,n_users-ServerInstance->InvisibleUserCount(),ServerInstance->InvisibleUserCount(),ulined_count ? this->CountServs() - ulined_count : this->CountServs());
                if (ServerInstance->OperCount())
                        user->WriteServ("252 %s %d :operator(s) online",user->nick,ServerInstance->OperCount());
                if (ServerInstance->UnregisteredUserCount())
                        user->WriteServ("253 %s %d :unknown connections",user->nick,ServerInstance->UnregisteredUserCount());
                if (ServerInstance->ChannelCount())
                        user->WriteServ("254 %s %d :channels formed",user->nick,ServerInstance->ChannelCount());
-               user->WriteServ("254 %s :I have %d clients and %d servers",user->nick,ServerInstance->LocalUserCount(),this->CountLocalServs());
+               user->WriteServ("254 %s :I have %d clients and %d servers",user->nick,ServerInstance->LocalUserCount(),ulined_local_count ? this->CountLocalServs() - ulined_local_count : this->CountLocalServs());
                user->WriteServ("265 %s :Current Local Users: %d  Max: %d",user->nick,ServerInstance->LocalUserCount(),max_local);
                user->WriteServ("266 %s :Current Global Users: %d  Max: %d",user->nick,n_users,max_global);
                return;
@@ -4275,7 +4528,7 @@ class ModuleSpanningTree : public Module
                /* Do we already have an IP? If so, no need to resolve it. */
                if (insp_aton(x->IPAddr.c_str(), &binip) > 0)
                {
-                       TreeSocket* newsocket = new TreeSocket(Utils, ServerInstance, x->IPAddr,x->Port,false,10,x->Name.c_str());
+                       TreeSocket* newsocket = new TreeSocket(Utils, ServerInstance, x->IPAddr,x->Port,false,x->Timeout ? x->Timeout : 10,x->Name.c_str());
                        if (newsocket->GetFd() > -1)
                        {
                                /* Handled automatically on success */
@@ -4284,61 +4537,27 @@ class ModuleSpanningTree : public Module
                        {
                                ServerInstance->SNO->WriteToSnoMask('l',"CONNECT: Error connecting \002%s\002: %s.",x->Name.c_str(),strerror(errno));
                                delete newsocket;
-                               this->DoFailOver(x);
+                               Utils->DoFailOver(x);
                        }
                }
                else
                {
                        try
                        {
-                               ServernameResolver* snr = new ServernameResolver(Utils, ServerInstance,x->IPAddr, *x);
+                               ServernameResolver* snr = new ServernameResolver((Module*)this, Utils, ServerInstance,x->IPAddr, *x);
                                ServerInstance->AddResolver(snr);
                        }
                        catch (ModuleException& e)
                        {
                                ServerInstance->Log(DEBUG,"Error in resolver: %s",e.GetReason());
-                               this->DoFailOver(x);
-                       }
-               }
-       }
-
-       void DoFailOver(Link* x)
-       {
-               if (x->FailOver.length())
-               {
-                       if (x->FailOver == x->Name)
-                       {
-                               ServerInstance->SNO->WriteToSnoMask('l',"FAILOVER: Some muppet configured the failover for server \002%s\002 to point at itself. Not following it!", x->Name.c_str());
-                               return;
-                       }
-                       Link* TryThisOne = this->FindLink(x->FailOver.c_str());
-                       if (TryThisOne)
-                       {
-                               ServerInstance->SNO->WriteToSnoMask('l',"FAILOVER: Trying failover link for \002%s\002: \002%s\002...", x->Name.c_str(), TryThisOne->Name.c_str());
-                               ConnectServer(TryThisOne);
-                       }
-                       else
-                       {
-                               ServerInstance->SNO->WriteToSnoMask('l',"FAILOVER: Invalid failover server specified for server \002%s\002, will not follow!", x->Name.c_str());
+                               Utils->DoFailOver(x);
                        }
                }
        }
 
-       Link* FindLink(const std::string& name)
-       {
-               for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
-               {
-                       if (ServerInstance->MatchText(x->Name.c_str(), name.c_str()))
-                       {
-                               return &(*x);
-                       }
-               }
-               return NULL;
-       }
-
        void AutoConnectServers(time_t curtime)
        {
-               for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
+               for (std::vector<Link>::iterator x = Utils->LinkBlocks.begin(); x < Utils->LinkBlocks.end(); x++)
                {
                        if ((x->AutoConnect) && (curtime >= x->NextConnectTime))
                        {
@@ -4406,7 +4625,7 @@ class ModuleSpanningTree : public Module
        
        int HandleConnect(const char** parameters, int pcnt, userrec* user)
        {
-               for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
+               for (std::vector<Link>::iterator x = Utils->LinkBlocks.begin(); x < Utils->LinkBlocks.end(); x++)
                {
                        if (ServerInstance->MatchText(x->Name.c_str(),parameters[0]))
                        {
@@ -4428,14 +4647,22 @@ class ModuleSpanningTree : public Module
                return 1;
        }
 
+       void BroadcastTimeSync()
+       {
+               std::deque<std::string> params;
+               params.push_back(ConvToStr(ServerInstance->Time(true)));
+               Utils->DoOneToMany(Utils->TreeRoot->GetName(), "TIMESET", params);
+       }
+
        virtual int OnStats(char statschar, userrec* user, string_list &results)
        {
-               if (statschar == 'c')
+               if ((statschar == 'c') || (statschar == 'n'))
                {
-                       for (unsigned int i = 0; i < LinkBlocks.size(); i++)
+                       for (unsigned int i = 0; i < Utils->LinkBlocks.size(); i++)
                        {
-                               results.push_back(std::string(ServerInstance->Config->ServerName)+" 213 "+user->nick+" C *@"+(LinkBlocks[i].HiddenFromStats ? "<hidden>" : LinkBlocks[i].IPAddr)+" * "+LinkBlocks[i].Name.c_str()+" "+ConvToStr(LinkBlocks[i].Port)+" "+(LinkBlocks[i].EncryptionKey != "" ? 'e' : '-')+(LinkBlocks[i].AutoConnect ? 'a' : '-')+'s');
-                               results.push_back(std::string(ServerInstance->Config->ServerName)+" 244 "+user->nick+" H * * "+LinkBlocks[i].Name.c_str());
+                               results.push_back(std::string(ServerInstance->Config->ServerName)+" 213 "+user->nick+" "+statschar+" *@"+(Utils->LinkBlocks[i].HiddenFromStats ? "<hidden>" : Utils->LinkBlocks[i].IPAddr)+" * "+Utils->LinkBlocks[i].Name.c_str()+" "+ConvToStr(Utils->LinkBlocks[i].Port)+" "+(Utils->LinkBlocks[i].EncryptionKey != "" ? 'e' : '-')+(Utils->LinkBlocks[i].AutoConnect ? 'a' : '-')+'s');
+                               if (statschar == 'c')
+                                       results.push_back(std::string(ServerInstance->Config->ServerName)+" 244 "+user->nick+" H * * "+Utils->LinkBlocks[i].Name.c_str());
                        }
                        results.push_back(std::string(ServerInstance->Config->ServerName)+" 219 "+user->nick+" "+statschar+" :End of /STATS report");
                        ServerInstance->SNO->WriteToSnoMask('t',"Notice: %s '%c' requested by %s (%s@%s)",(!strcmp(user->server,ServerInstance->Config->ServerName) ? "Stats" : "Remote stats"),statschar,user->nick,user->ident,user->host);
@@ -4571,7 +4798,7 @@ class ModuleSpanningTree : public Module
                }
        }
 
-       virtual void OnUserNotice(userrec* user, void* dest, int target_type, const std::string &text, char status)
+       virtual void OnUserNotice(userrec* user, void* dest, int target_type, const std::string &text, char status, const CUList &exempt_list)
        {
                if (target_type == TYPE_USER)
                {
@@ -4595,12 +4822,12 @@ class ModuleSpanningTree : public Module
                                        std::string cname = c->name;
                                        if (status)
                                                cname = status + cname;
-                                       std::deque<TreeServer*> list;
-                                       Utils->GetListOfServersForChannel(c,list);
-                                       unsigned int ucount = list.size();
-                                       for (unsigned int i = 0; i < ucount; i++)
+                                       TreeServerList list;
+                                       Utils->GetListOfServersForChannel(c,list,status,exempt_list);
+
+                                       for (TreeServerList::iterator i = list.begin(); i != list.end(); i++)
                                        {
-                                               TreeSocket* Sock = list[i]->GetSocket();
+                                               TreeSocket* Sock = i->second->GetSocket();
                                                if (Sock)
                                                        Sock->WriteLine(":"+std::string(user->nick)+" NOTICE "+cname+" :"+text);
                                        }
@@ -4620,7 +4847,7 @@ class ModuleSpanningTree : public Module
                }
        }
 
-       virtual void OnUserMessage(userrec* user, void* dest, int target_type, const std::string &text, char status)
+       virtual void OnUserMessage(userrec* user, void* dest, int target_type, const std::string &text, char status, const CUList &exempt_list)
        {
                if (target_type == TYPE_USER)
                {
@@ -4646,12 +4873,12 @@ class ModuleSpanningTree : public Module
                                        std::string cname = c->name;
                                        if (status)
                                                cname = status + cname;
-                                       std::deque<TreeServer*> list;
-                                       Utils->GetListOfServersForChannel(c,list);
-                                       unsigned int ucount = list.size();
-                                       for (unsigned int i = 0; i < ucount; i++)
+                                       TreeServerList list;
+                                       Utils->GetListOfServersForChannel(c,list,status,exempt_list);
+
+                                       for (TreeServerList::iterator i = list.begin(); i != list.end(); i++)
                                        {
-                                               TreeSocket* Sock = list[i]->GetSocket();
+                                               TreeSocket* Sock = i->second->GetSocket();
                                                if (Sock)
                                                        Sock->WriteLine(":"+std::string(user->nick)+" PRIVMSG "+cname+" :"+text);
                                        }
@@ -4693,6 +4920,14 @@ class ModuleSpanningTree : public Module
                        params.push_back(ConvToStr(channel->age));
                        params.push_back(std::string(channel->GetAllPrefixChars(user))+","+std::string(user->nick));
                        Utils->DoOneToMany(ServerInstance->Config->ServerName,"FJOIN",params);
+                       if (channel->GetUserCounter() == 1)
+                       {
+                               /* First user in, sync the modes for the channel */
+                               params.pop_back();
+                               /* This is safe, all inspircd servers default to +nt */
+                               params.push_back("+nt");
+                               Utils->DoOneToMany(ServerInstance->Config->ServerName,"FMODE",params);
+                       }
                }
        }
 
@@ -4842,26 +5077,39 @@ class ModuleSpanningTree : public Module
 
        void OnLine(userrec* source, const std::string &host, bool adding, char linetype, long duration, const std::string &reason)
        {
-               if (IS_LOCAL(source))
+               if (!source)
                {
-                       char type[8];
-                       snprintf(type,8,"%cLINE",linetype);
-                       std::string stype = type;
-                       if (adding)
-                       {
-                               char sduration[MAXBUF];
-                               snprintf(sduration,MAXBUF,"%ld",duration);
-                               std::deque<std::string> params;
-                               params.push_back(host);
-                               params.push_back(sduration);
-                               params.push_back(":"+reason);
-                               Utils->DoOneToMany(source->nick,stype,params);
-                       }
-                       else
+                       /* Server-set lines */
+                       char data[MAXBUF];
+                       snprintf(data,MAXBUF,"%c %s %s %lu %lu :%s", linetype, host.c_str(), ServerInstance->Config->ServerName, (unsigned long)ServerInstance->Time(false),
+                                       (unsigned long)duration, reason.c_str());
+                       std::deque<std::string> params;
+                       params.push_back(data);
+                       Utils->DoOneToMany(ServerInstance->Config->ServerName, "ADDLINE", params);
+               }
+               else
+               {
+                       if (IS_LOCAL(source))
                        {
-                               std::deque<std::string> params;
-                               params.push_back(host);
-                               Utils->DoOneToMany(source->nick,stype,params);
+                               char type[8];
+                               snprintf(type,8,"%cLINE",linetype);
+                               std::string stype = type;
+                               if (adding)
+                               {
+                                       char sduration[MAXBUF];
+                                       snprintf(sduration,MAXBUF,"%ld",duration);
+                                       std::deque<std::string> params;
+                                       params.push_back(host);
+                                       params.push_back(sduration);
+                                       params.push_back(":"+reason);
+                                       Utils->DoOneToMany(source->nick,stype,params);
+                               }
+                               else
+                               {
+                                       std::deque<std::string> params;
+                                       params.push_back(host);
+                                       Utils->DoOneToMany(source->nick,stype,params);
+                               }
                        }
                }
        }
@@ -5006,7 +5254,7 @@ class ModuleSpanningTree : public Module
                                return;
                        (*params)[1] = ":" + (*params)[1];
                        params->insert(params->begin() + 1,ServerInstance->Config->ServerName);
-                       params->insert(params->begin() + 1,ConvToStr(ServerInstance->Time()));
+                       params->insert(params->begin() + 1,ConvToStr(ServerInstance->Time(true)));
                        Utils->DoOneToMany(ServerInstance->Config->ServerName,"FTOPIC",*params);
                }
                else if (event->GetEventID() == "send_mode")
@@ -5031,6 +5279,46 @@ class ModuleSpanningTree : public Module
                        params->insert(params->begin() + 1,ConvToStr(ourTS));
                        Utils->DoOneToMany(ServerInstance->Config->ServerName,"FMODE",*params);
                }
+               else if (event->GetEventID() == "send_mode_explicit")
+               {
+                       if (params->size() < 2)
+                               return;
+                       Utils->DoOneToMany(ServerInstance->Config->ServerName,"MODE",*params);
+               }
+               else if (event->GetEventID() == "send_opers")
+               {
+                       if (params->size() < 1)
+                               return;
+                       (*params)[0] = ":" + (*params)[0];
+                       Utils->DoOneToMany(ServerInstance->Config->ServerName,"OPERNOTICE",*params);
+               }
+               else if (event->GetEventID() == "send_modeset")
+               {
+                       if (params->size() < 2)
+                               return;
+                       (*params)[1] = ":" + (*params)[1];
+                       Utils->DoOneToMany(ServerInstance->Config->ServerName,"MODENOTICE",*params);
+               }
+               else if (event->GetEventID() == "send_snoset")
+               {
+                       if (params->size() < 2)
+                               return;
+                       (*params)[1] = ":" + (*params)[1];
+                       Utils->DoOneToMany(ServerInstance->Config->ServerName,"SNONOTICE",*params);
+               }
+               else if (event->GetEventID() == "send_push")
+               {
+                       if (params->size() < 2)
+                               return;
+                       
+                       userrec *a = ServerInstance->FindNick((*params)[0]);
+                       
+                       if (!a)
+                               return;
+                       
+                       (*params)[1] = ":" + (*params)[1];
+                       Utils->DoOneToOne(ServerInstance->Config->ServerName, "PUSH", *params, a->server);
+               }
        }
 
        virtual ~ModuleSpanningTree()
@@ -5038,6 +5326,8 @@ class ModuleSpanningTree : public Module
                ServerInstance->Log(DEBUG,"Performing unload of spanningtree!");
                /* This will also free the listeners */
                delete Utils;
+               if (SyncTimer)
+                       ServerInstance->Timers->DelTimer(SyncTimer);
        }
 
        virtual Version GetVersion()
@@ -5069,14 +5359,49 @@ class ModuleSpanningTree : public Module
        }
 };
 
-void DoFailOver(Link* x)
+TimeSyncTimer::TimeSyncTimer(InspIRCd *Inst, ModuleSpanningTree *Mod) : InspTimer(43200, Inst->Time()), Instance(Inst), Module(Mod)
+{
+}
+
+void TimeSyncTimer::Tick(time_t TIME)
 {
-       TreeProtocolModule->DoFailOver(x);
+       Module->BroadcastTimeSync();
+       Module->SyncTimer = new TimeSyncTimer(Instance, Module);
+       Instance->Timers->AddTimer(Module->SyncTimer);
+}
+
+void SpanningTreeUtilities::DoFailOver(Link* x)
+{
+       if (x->FailOver.length())
+       {
+               if (x->FailOver == x->Name)
+               {
+                       ServerInstance->SNO->WriteToSnoMask('l',"FAILOVER: Some muppet configured the failover for server \002%s\002 to point at itself. Not following it!", x->Name.c_str());
+                       return;
+               }
+               Link* TryThisOne = this->FindLink(x->FailOver.c_str());
+               if (TryThisOne)
+               {
+                       ServerInstance->SNO->WriteToSnoMask('l',"FAILOVER: Trying failover link for \002%s\002: \002%s\002...", x->Name.c_str(), TryThisOne->Name.c_str());
+                       Creator->ConnectServer(TryThisOne);
+               }
+               else
+               {
+                       ServerInstance->SNO->WriteToSnoMask('l',"FAILOVER: Invalid failover server specified for server \002%s\002, will not follow!", x->Name.c_str());
+               }
+       }
 }
 
-Link* FindLink(const std::string& name)
+Link* SpanningTreeUtilities::FindLink(const std::string& name)
 {
-       return TreeProtocolModule->FindLink(name);
+       for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
+       {
+               if (ServerInstance->MatchText(x->Name.c_str(), name.c_str()))
+               {
+                       return &(*x);
+               }
+       }
+       return NULL;
 }
 
 class ModuleSpanningTreeFactory : public ModuleFactory
@@ -5092,8 +5417,7 @@ class ModuleSpanningTreeFactory : public ModuleFactory
        
        virtual Module * CreateModule(InspIRCd* Me)
        {
-               TreeProtocolModule = new ModuleSpanningTree(Me);
-               return TreeProtocolModule;
+               return new ModuleSpanningTree(Me);
        }
        
 };