]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_spanningtree.cpp
The IPV6 stuff compiles now, with compile-correct ipv6 code. I dont know if this...
[user/henk/code/inspircd.git] / src / modules / m_spanningtree.cpp
index c632fa540bc46f27218d00bc3bc1897edd4952f4..0725d7b51767dab0ea2d9c40a8b27b7418242b6c 100644 (file)
@@ -30,6 +30,7 @@ using namespace std;
 #include "modules.h"
 #include "commands.h"
 #include "commands/cmd_whois.h"
+#include "commands/cmd_stats.h"
 #include "socket.h"
 #include "helperfuncs.h"
 #include "inspircd.h"
@@ -41,11 +42,7 @@ using namespace std;
 #include "cull_list.h"
 #include "aes.h"
 
-#ifdef GCC3
 #define nspace __gnu_cxx
-#else
-#define nspace std
-#endif
 
 /*
  * The server list in InspIRCd is maintained as two structures
@@ -115,6 +112,9 @@ static Server* Srv;
 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;
+
 /* More forward declarations */
 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);
@@ -139,6 +139,53 @@ extern std::vector<ZLine> pzlines;
 extern std::vector<QLine> pqlines;
 extern std::vector<ELine> pelines;
 
+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;
+       }
+};
+
 /* Each server in the tree is represented by one class of
  * type TreeServer. A locally connected TreeServer can
  * have a class of type TreeSocket associated with it, for
@@ -167,8 +214,6 @@ 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 */
-       std::map<userrec*,userrec*> Users;      /* Users on this server */
-       bool DontModifyHash;                    /* When the server is splitting, this is set to true so we dont bash our own iterator to death */
        
  public:
 
@@ -183,7 +228,6 @@ class TreeServer : public classbase
                VersionString = "";
                UserCount = OperCount = 0;
                VersionString = Srv->GetVersion();
-               DontModifyHash = false;
        }
 
        /* We use this constructor only to create the 'root' item, TreeRoot, which
@@ -198,7 +242,6 @@ class TreeServer : public classbase
                VersionString = Srv->GetVersion();
                Route = NULL;
                Socket = NULL; /* Fix by brain */
-               DontModifyHash = false;
                AddHashEntry();
        }
 
@@ -210,7 +253,6 @@ class TreeServer : public classbase
        {
                VersionString = "";
                UserCount = OperCount = 0;
-               DontModifyHash = false;
                this->SetNextPingTime(time(NULL) + 120);
                this->SetPingFlag();
 
@@ -270,24 +312,24 @@ class TreeServer : public classbase
 
        int QuitUsers(const std::string &reason)
        {
-                log(DEBUG,"Removing all users from server %s",this->ServerName.c_str());
-                const char* reason_s = reason.c_str();
-                std::vector<userrec*> time_to_die;
-                for (user_hash::iterator n = clientlist.begin(); n != clientlist.end(); n++)
-                {
-                        if (!strcmp(n->second->server, this->ServerName.c_str()))
-                        {
-                                time_to_die.push_back(n->second);
-                        }
-                }
-                for (std::vector<userrec*>::iterator n = time_to_die.begin(); n != time_to_die.end(); n++)
-                {
-                        userrec* a = (userrec*)*n;
-                        log(DEBUG,"Kill %s fd=%d",a->nick,a->fd);
-                        if (!IS_LOCAL(a))
-                                kill_link(a,reason_s);
-                }
-                return time_to_die.size();
+               log(DEBUG,"Removing all users from server %s",this->ServerName.c_str());
+               const char* reason_s = reason.c_str();
+               std::vector<userrec*> time_to_die;
+               for (user_hash::iterator n = clientlist.begin(); n != clientlist.end(); n++)
+               {
+                       if (!strcmp(n->second->server, this->ServerName.c_str()))
+                       {
+                               time_to_die.push_back(n->second);
+                       }
+               }
+               for (std::vector<userrec*>::iterator n = time_to_die.begin(); n != time_to_die.end(); n++)
+               {
+                       userrec* a = (userrec*)*n;
+                       log(DEBUG,"Kill %s fd=%d",a->nick,a->fd);
+                       if (!IS_LOCAL(a))
+                               kill_link(a,reason_s);
+               }
+               return time_to_die.size();
        }
 
        /* This method is used to add the structure to the
@@ -564,9 +606,10 @@ class cmd_rconnect : public command_t
        cmd_rconnect (Module* Callback) : command_t("RCONNECT", 'o', 2), Creator(Callback)
        {
                this->source = "m_spanningtree.so";
-       }                
+               syntax = "<remote-server-mask> <servermask>";
+       }
 
-       void Handle (char **parameters, int pcnt, userrec *user)
+       void Handle (const char** parameters, int pcnt, userrec *user)
        {
                WriteServ(user->fd,"NOTICE %s :*** RCONNECT: Sending remote connect to \002%s\002 to connect server \002%s\002.",user->nick,parameters[0],parameters[1]);
                /* Is this aimed at our server? */
@@ -574,7 +617,7 @@ class cmd_rconnect : public command_t
                {
                        /* Yes, initiate the given connect */
                        WriteOpers("*** Remote CONNECT from %s matching \002%s\002, connecting server \002%s\002",user->nick,parameters[0],parameters[1]);
-                       char* para[1];
+                       const char* para[1];
                        para[0] = parameters[1];
                        Creator->OnPreCommand("CONNECT", para, 1, user, true);
                }
@@ -845,14 +888,6 @@ class TreeSocket : public InspSocket
                /* Now we've whacked the kids, whack self */
                num_lost_servers++;
                num_lost_users += Current->QuitUsers(from);
-               /*for (user_hash::iterator u = clientlist.begin(); u != clientlist.end(); u++)
-               {
-                       if (!strcasecmp(u->second->server,Current->GetName().c_str()))
-                       {
-                               Goners->AddItem(u->second,from);
-                               num_lost_users++;
-                       }
-               }*/
        }
 
        /* This is a wrapper function for SquitServer above, which
@@ -890,22 +925,378 @@ class TreeSocket : public InspSocket
                }
        }
 
-       /* FMODE command */
+       /* FMODE command - server mode with timestamp checks */
        bool ForceMode(std::string source, std::deque<std::string> &params)
        {
-               if (params.size() < 2)
-                       return true;
-               userrec* who = new userrec();
-               who->fd = FD_MAGIC_NUMBER;
-               char* modelist[64];
+               /* Chances are this is a 1.0 FMODE without TS */
+               if (params.size() < 3)
+               {
+                       this->WriteLine("ERROR :Version 1.0 FMODE sent to version 1.1 server");
+                       return false;
+               }
+               
+               bool smode = false;
+               std::string sourceserv;
+
+               /* Are we dealing with an FMODE from a user, or from a server? */
+               userrec* who = Srv->FindNick(source);
+               if (who)
+               {
+                       /* FMODE from a user, set sourceserv to the users server name */
+                       sourceserv = who->server;
+               }
+               else
+               {
+                       /* FMODE from a server, create a fake user to receive mode feedback */
+                       who = new userrec();
+                       who->fd = FD_MAGIC_NUMBER;
+                       smode = true;           /* Setting this flag tells us we should free the userrec later */
+                       sourceserv = source;    /* Set sourceserv to the actual source string */
+               }
+               const char* modelist[64];
+               time_t TS = 0;
+               int n = 0;
                memset(&modelist,0,sizeof(modelist));
-               for (unsigned int q = 0; q < params.size(); q++)
+               for (unsigned int q = 0; (q < params.size()) && (q < 64); q++)
+               {
+                       if (q == 1)
+                       {
+                               /* The timestamp is in this position.
+                                * We don't want to pass that up to the
+                                * server->client protocol!
+                                */
+                               TS = atoi(params[q].c_str());
+                       }
+                       else
+                       {
+                               /* Everything else is fine to append to the modelist */
+                               modelist[n++] = params[q].c_str();
+                       }
+                               
+               }
+                /* Extract the TS value of the object, either userrec or chanrec */
+               userrec* dst = Srv->FindNick(params[0]);
+               chanrec* chan = NULL;
+               time_t ourTS = 0;
+               if (dst)
+               {
+                       ourTS = dst->age;
+               }
+               else
+               {
+                       chan = Srv->FindChannel(params[0]);
+                       if (chan)
+                       {
+                               ourTS = chan->age;
+                       }
+               }
+
+               /* TS is equal: Merge the mode changes, use voooodoooooo on modes
+                * with parameters.
+                */
+               if (TS == ourTS)
+               {
+                       log(DEBUG,"Entering TS equality check");
+                       ModeHandler* mh = NULL;
+                       unsigned long paramptr = 3;
+                       std::string to_bounce = "";
+                       std::string to_keep = "";
+                       std::vector<std::string> params_to_keep;
+                       std::string params_to_bounce = "";
+                       bool adding = true;
+                       char cur_change = 1;
+                       char old_change = 0;
+                       char old_bounce_change = 0;
+                       /* Merge modes, basically do special stuff to mode with params */
+                       for (std::string::iterator x = params[2].begin(); x != params[2].end(); x++)
+                       {
+                               switch (*x)
+                               {
+                                       case '-':
+                                               adding = false;
+                                       break;
+                                       case '+':
+                                               adding = true;
+                                       break;
+                                       default:
+                                               if (adding)
+                                               {
+                                                       /* We only care about whats being set,
+                                                        * not whats being unset
+                                                        */
+                                                       mh = ServerInstance->ModeGrok->FindMode(*x, chan ? MODETYPE_CHANNEL : MODETYPE_USER);
+
+                                                       if ((mh->GetNumParams(adding) > 0) && (!mh->IsListMode()))
+                                                       {
+                                                               /* We only want to do special things to
+                                                                * modes with parameters, we are going to rewrite
+                                                                * those parameters
+                                                                */
+                                                               ModePair ret;
+                                                               adding ? cur_change = '+' : cur_change = '-';
+
+                                                               ret = mh->ModeSet(smode ? NULL : who, dst, chan, params[paramptr]);
+
+                                                               /* The mode is set here, check which we should keep */
+                                                               if (ret.first)
+                                                               {
+                                                                       bool which_to_keep = mh->CheckTimeStamp(TS, ourTS, params[paramptr], ret.second, chan);
+
+                                                                       if (which_to_keep == true)
+                                                                       {
+                                                                               /* Keep ours, bounce theirs:
+                                                                                * Send back ours to them and
+                                                                                * drop their mode changs
+                                                                                */
+                                                                               adding ? cur_change = '+' : cur_change = '-';
+                                                                               if (cur_change != old_bounce_change)
+                                                                                       to_bounce += cur_change;
+                                                                               to_bounce += *x;
+                                                                               old_bounce_change = cur_change;
+
+                                                                               if ((mh->GetNumParams(adding) > 0) && (paramptr < params.size()))
+                                                                                       params_to_bounce.append(" ").append(ret.second);
+                                                                       }
+                                                                       else
+                                                                       {
+                                                                               /* Keep theirs: Accept their mode change,
+                                                                                * do nothing else
+                                                                                */
+                                                                               adding ? cur_change = '+' : cur_change = '-';
+                                                                               if (cur_change != old_change)
+                                                                                       to_keep += cur_change;
+                                                                               to_keep += *x;
+                                                                               old_change = cur_change;
+
+                                                                               if ((mh->GetNumParams(adding) > 0) && (paramptr < params.size()))
+                                                                                       params_to_keep.push_back(params[paramptr]);
+                                                                       }
+                                                               }
+                                                               else
+                                                               {
+                                                                       /* Mode isnt set here, we want it */
+                                                                       adding ? cur_change = '+' : cur_change = '-';
+                                                                       if (cur_change != old_change)
+                                                                               to_keep += cur_change;
+                                                                       to_keep += *x;
+                                                                       old_change = cur_change;
+
+                                                                       if ((mh->GetNumParams(adding) > 0) && (paramptr < params.size()))
+                                                                               params_to_keep.push_back(params[paramptr]);
+                                                               }
+
+                                                               paramptr++;
+                                                       }
+                                                       else
+                                                       {
+                                                               mh = ServerInstance->ModeGrok->FindMode(*x, chan ? MODETYPE_CHANNEL : MODETYPE_USER);
+
+                                                               if (mh)
+                                                               {
+                                                                       adding ? cur_change = '+' : cur_change = '-';
+       
+                                                                       /* Just keep this, safe to merge with no checks
+                                                                        * it has no parameters
+                                                                        */
+       
+                                                                       if (cur_change != old_change)
+                                                                               to_keep += cur_change;
+                                                                       to_keep += *x;
+                                                                       old_change = cur_change;
+       
+                                                                       if ((mh->GetNumParams(adding) > 0) && (paramptr < params.size()))
+                                                                       {
+                                                                               log(DEBUG,"Mode removal %d %d",adding, mh->GetNumParams(adding));
+                                                                               params_to_keep.push_back(params[paramptr++]);
+                                                                       }
+                                                               }
+                                                       }
+                                               }
+                                               else
+                                               {
+                                                       mh = ServerInstance->ModeGrok->FindMode(*x, chan ? MODETYPE_CHANNEL : MODETYPE_USER);
+
+                                                       if (mh)
+                                                       {
+                                                               /* Taking a mode away */
+                                                               adding ? cur_change = '+' : cur_change = '-';
+
+                                                               if (cur_change != old_change)
+                                                                       to_keep += cur_change;
+                                                               to_keep += *x;
+                                                               old_change = cur_change;
+
+                                                               if ((mh->GetNumParams(adding) > 0) && (paramptr < params.size()))
+                                                                       params_to_keep.push_back(params[paramptr++]);
+                                                       }
+                                               }
+                                       break;
+                               }
+                       }
+
+                       if (to_bounce.length())
+                       {
+                               std::deque<std::string> newparams;
+                               newparams.push_back(params[0]);
+                               newparams.push_back(ConvToStr(ourTS));
+                               newparams.push_back(to_bounce+params_to_bounce);
+                               DoOneToOne(Srv->GetServerName(),"FMODE",newparams,sourceserv);
+                       }
+
+                       if (to_keep.length())
+                       {
+                               n = 0;
+                               modelist[0] = params[0].c_str();
+                               modelist[1] = to_keep.c_str();
+
+                               for (unsigned int q = 2; (q < params.size()) && (q < 64); q++)
+                                               modelist[q] = params_to_keep[n++].c_str();
+
+                               if (smode)
+                               {
+                                       Srv->SendMode(modelist, n+2, who);
+                               }
+                               else
+                               {
+                                       Srv->CallCommandHandler("MODE", modelist, n+2, who);
+                               }
+
+                               /* HOT POTATO! PASS IT ON! */
+                               DoOneToAllButSender(source,"FMODE",params,sourceserv);
+                       }
+               }
+               else
+               /* U-lined servers always win regardless of their TS */
+               if ((TS > ourTS) && (!Srv->IsUlined(source)))
+               {
+                       /* Bounce the mode back to its sender.* We use our lower TS, so the other end
+                        * SHOULD accept it, if its clock is right.
+                        *
+                        * NOTE: We should check that we arent bouncing anything thats already set at this end.
+                        * If we are, bounce +ourmode to 'reinforce' it. This prevents desyncs.
+                        * e.g. They send +l 50, we have +l 10 set. rather than bounce -l 50, we bounce +l 10.
+                        *
+                        * Thanks to jilles for pointing out this one-hell-of-an-issue before i even finished
+                        * writing the code. It took me a while to come up with this solution.
+                        *
+                        * XXX: BE SURE YOU UNDERSTAND THIS CODE FULLY BEFORE YOU MESS WITH IT.
+                        */
+
+                       std::deque<std::string> newparams;      /* New parameter list we send back */
+                       newparams.push_back(params[0]);         /* Target, user or channel */
+                       newparams.push_back(ConvToStr(ourTS));  /* Timestamp value of the target */
+                       newparams.push_back("");                /* This contains the mode string. For now
+                                                                * it's empty, we fill it below.
+                                                                */
+
+                       /* Intelligent mode bouncing. Don't just invert, reinforce any modes which are already
+                        * set to avoid a desync here.
+                        */
+                       std::string modebounce = "";
+                       bool adding = true;
+                       unsigned int t = 3;
+                       ModeHandler* mh = NULL;
+                       char cur_change = 1;
+                       char old_change = 0;
+                       for (std::string::iterator x = params[2].begin(); x != params[2].end(); x++)
+                       {
+                               /* Iterate over all mode chars in the sent set */
+                               switch (*x)
+                               {
+                                       /* Adding or subtracting modes? */
+                                       case '-':
+                                               adding = false;
+                                       break;
+                                       case '+':
+                                               adding = true;
+                                       break;
+                                       default:
+                                               /* Find the mode handler for this mode */
+                                               mh = ServerInstance->ModeGrok->FindMode(*x, chan ? MODETYPE_CHANNEL : MODETYPE_USER);
+
+                                               /* Got a mode handler?
+                                                * This also prevents us bouncing modes we have no handler for.
+                                                */
+                                               if (mh)
+                                               {
+                                                       ModePair ret;
+                                                       std::string p = "";
+
+                                                       /* Does the mode require a parameter right now?
+                                                        * If it does, fetch it if we can
+                                                        */
+                                                       if ((mh->GetNumParams(adding) > 0) && (t < params.size()))
+                                                               p = params[t++];
+
+                                                       /* Call the ModeSet method to determine if its set with the
+                                                        * given parameter here or not.
+                                                        */
+                                                       ret = mh->ModeSet(smode ? NULL : who, dst, chan, p);
+
+                                                       /* XXX: Really. Dont ask.
+                                                        * Determine from if its set combined with what the current
+                                                        * 'state' is (adding or not) as to wether we should 'invert'
+                                                        * or 'reinforce' the mode change
+                                                        */
+                                                       (!ret.first ? (adding ? cur_change = '-' : cur_change = '+') : (!adding ? cur_change = '-' : cur_change = '+'));
+
+                                                       /* Quickly determine if we have 'flipped' from + to -,
+                                                        * or - to +, to prevent unneccessary +/- chars in the
+                                                        * output string that waste bandwidth
+                                                        */
+                                                       if (cur_change != old_change)
+                                                               modebounce += cur_change;
+                                                       old_change = cur_change;
+
+                                                       /* Add the mode character to the output string */
+                                                       modebounce += mh->GetModeChar();
+
+                                                       /* We got a parameter back from ModeHandler::ModeSet,
+                                                        * are we supposed to be sending one out right now?
+                                                        */
+                                                       if (ret.second.length())
+                                                       {
+                                                               if (mh->GetNumParams(cur_change == '+') > 0)
+                                                                       /* Yes we're supposed to be sending out
+                                                                        * the parameter. Make sure it goes
+                                                                        */
+                                                                       newparams.push_back(ret.second);
+                                                       }
+
+                                               }
+                                       break;
+                               }
+                       }
+                       
+                       /* Update the parameters for FMODE with the new 'bounced' string */
+                       newparams[2] = modebounce;
+                       /* Only send it back the way it came, no need to send it anywhere else */
+                       DoOneToOne(Srv->GetServerName(),"FMODE",newparams,sourceserv);
+                       log(DEBUG,"FMODE bounced intelligently, our TS less than theirs and the other server is NOT a uline.");
+               }
+               else
                {
-                       modelist[q] = (char*)params[q].c_str();
+                       log(DEBUG,"Allow modes, TS lower for sender");
+                       /* The server was ulined, but something iffy is up with the TS.
+                        * Sound the alarm bells!
+                        */
+                       if ((Srv->IsUlined(sourceserv)) && (TS > ourTS))
+                       {
+                               WriteOpers("\2WARNING!\2 U-Lined server '%s' has bad TS for '%s' (accepted change): \2SYNC YOUR CLOCKS\2 to avoid this notice",sourceserv.c_str(),params[0].c_str());
+                       }
+                       /* Allow the mode, route it to either server or user command handling */
+                       if (smode)
+                               Srv->SendMode(modelist,n,who);
+                       else
+                               Srv->CallCommandHandler("MODE", modelist, n, who);
+
+                       /* HOT POTATO! PASS IT ON! */
+                       DoOneToAllButSender(source,"FMODE",params,sourceserv);
                }
-               Srv->SendMode(modelist,params.size(),who);
-               DoOneToAllButSender(source,"FMODE",params,source);
-               DELETE(who);
+               /* Are we supposed to free the userrec? */
+               if (smode)
+                       DELETE(who);
+
                return true;
        }
 
@@ -965,7 +1356,7 @@ class TreeSocket : public InspSocket
                memset(&mode_users,0,sizeof(mode_users));
                mode_users[0] = first;
                mode_users[1] = modestring;
-               strcpy(mode_users[1],"+");
+               strcpy(first,"+");
                unsigned int modectr = 2;
                
                userrec* who = NULL;
@@ -1000,7 +1391,7 @@ class TreeSocket : public InspSocket
                for (unsigned int usernum = 2; usernum < params.size(); usernum++)
                {
                        /* process one channel at a time, applying modes. */
-                       char* usr = const_cast<char*>(params[usernum].c_str());
+                       char* usr = (char*)params[usernum].c_str();
                        /* Safety check just to make sure someones not sent us an FJOIN full of spaces
                         * (is this even possible?) */
                        if (usr && *usr)
@@ -1037,7 +1428,12 @@ class TreeSocket : public InspSocket
                                                {
                                                        /* We also always let u-lined clients win, no matter what the TS value */
                                                        log(DEBUG,"Our our channel newer than theirs, accepting their modes");
-                                                       Srv->SendMode(mode_users,modectr,who);
+                                                       Srv->SendMode((const char**)mode_users,modectr,who);
+                                                       if (ourTS != TS)
+                                                       {
+                                                               log(DEFAULT,"Channel TS for %s changed from %lu to %lu",us,ourTS,TS);
+                                                               us->age = TS;
+                                                       }
                                                }
                                                else
                                                {
@@ -1048,6 +1444,10 @@ class TreeSocket : public InspSocket
                                                        *mode_users[1] = '-';
                                                        for (unsigned int x = 0; x < modectr; x++)
                                                        {
+                                                               if (x == 1)
+                                                               {
+                                                                       params.push_back(ConvToStr(us->age));
+                                                               }
                                                                params.push_back(mode_users[x]);
                                                        }
                                                        // tell everyone to bounce the modes. bad modes, bad!
@@ -1067,7 +1467,12 @@ class TreeSocket : public InspSocket
                        if (ourTS >= TS)
                        {
                                log(DEBUG,"Our our channel newer than theirs, accepting their modes");
-                               Srv->SendMode(mode_users,modectr,who);
+                               Srv->SendMode((const char**)mode_users,modectr,who);
+                               if (ourTS != TS)
+                               {
+                                       log(DEFAULT,"Channel TS for %s changed from %lu to %lu",us,ourTS,TS);
+                                       us->age = TS;
+                               }
                        }
                        else
                        {
@@ -1076,6 +1481,10 @@ class TreeSocket : public InspSocket
                                *mode_users[1] = '-';
                                for (unsigned int x = 0; x < modectr; x++)
                                {
+                                       if (x == 1)
+                                       {
+                                               params.push_back(ConvToStr(us->age));
+                                       }
                                        params.push_back(mode_users[x]);
                                }
                                DoOneToMany(Srv->GetServerName(),"FMODE",params);
@@ -1086,7 +1495,7 @@ class TreeSocket : public InspSocket
 
        bool SyncChannelTS(std::string source, std::deque<std::string> &params)
        {
-               if (params.size() == 2)
+               if (params.size() >= 2)
                {
                        chanrec* c = Srv->FindChannel(params[0]);
                        if (c)
@@ -1144,16 +1553,16 @@ class TreeSocket : public InspSocket
                clientlist[tempnick]->server = FindServerNamePtr(source.c_str());
                strlcpy(clientlist[tempnick]->ident, params[4].c_str(),IDENTMAX);
                strlcpy(clientlist[tempnick]->fullname, params[7].c_str(),MAXGECOS);
-               clientlist[tempnick]->registered = 7;
+               clientlist[tempnick]->registered = REG_ALL;
                clientlist[tempnick]->signon = age;
                
                for (std::string::iterator v = params[5].begin(); v != params[5].end(); v++)
                {
                        clientlist[tempnick]->modes[(*v)-65] = 1;
                }
-               inet_aton(params[6].c_str(),&clientlist[tempnick]->ip4);
+               insp_aton(params[6].c_str(),&clientlist[tempnick]->ip4);
 
-               WriteOpers("*** Client connecting at %s: %s!%s@%s [%s]",clientlist[tempnick]->server,clientlist[tempnick]->nick,clientlist[tempnick]->ident,clientlist[tempnick]->host, inet_ntoa(clientlist[tempnick]->ip4));
+               WriteOpers("*** Client connecting at %s: %s!%s@%s [%s]",clientlist[tempnick]->server,clientlist[tempnick]->nick,clientlist[tempnick]->ident,clientlist[tempnick]->host, insp_ntoa(clientlist[tempnick]->ip4));
 
                params[7] = ":" + params[7];
                DoOneToAllButSender(source,"NICK",params,source);
@@ -1177,7 +1586,7 @@ class TreeSocket : public InspSocket
        {
                log(DEBUG,"Sending FJOINs to other server for %s",c->name);
                char list[MAXBUF];
-               std::string individual_halfops = ":"+Srv->GetServerName()+" FMODE "+c->name;
+               std::string individual_halfops = ":"+Srv->GetServerName()+" FMODE "+c->name+" "+ConvToStr(c->age);
                
                size_t dlen, curlen;
                dlen = curlen = snprintf(list,MAXBUF,":%s FJOIN %s %lu",Srv->GetServerName().c_str(),c->name,(unsigned long)c->age);
@@ -1187,6 +1596,8 @@ class TreeSocket : public InspSocket
                CUList *ulist = c->GetUsers();
                std::vector<userrec*> specific_halfop;
                std::vector<userrec*> specific_voice;
+               std::string modes = "";
+               std::string params = "";
 
                for (CUList::iterator i = ulist->begin(); i != ulist->end(); i++)
                {
@@ -1230,11 +1641,15 @@ class TreeSocket : public InspSocket
                                numusers = 0;
                                for (unsigned int y = 0; y < specific_voice.size(); y++)
                                {
-                                       this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" +v "+specific_voice[y]->nick);
+                                       modes.append("v");
+                                       params.append(specific_voice[y]->nick).append(" ");
+                                       //this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" "+ConvToStr(c->age)+" +v "+specific_voice[y]->nick);
                                }
                                for (unsigned int y = 0; y < specific_halfop.size(); y++)
                                {
-                                       this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" +h "+specific_halfop[y]->nick);
+                                       modes.append("h");
+                                       params.append(specific_halfop[y]->nick).append(" ");
+                                       //this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" "+ConvToStr(c->age)+" +h "+specific_halfop[y]->nick);
                                }
                        }
                }
@@ -1243,14 +1658,27 @@ class TreeSocket : public InspSocket
                        this->WriteLine(list);
                        for (unsigned int y = 0; y < specific_voice.size(); y++)
                        {
-                               this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" +v "+specific_voice[y]->nick);
+                               modes.append("v");
+                               params.append(specific_voice[y]->nick).append(" ");
+                               //this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" "+ConvToStr(c->age)+" +v "+specific_voice[y]->nick);
                        }
                        for (unsigned int y = 0; y < specific_halfop.size(); y++)
                        {
-                               this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" +h "+specific_halfop[y]->nick);
+                               modes.append("h");
+                               params.append(specific_halfop[y]->nick).append(" ");
+                               //this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" "+ConvToStr(c->age)+" +h "+specific_halfop[y]->nick);
                        }
                }
-               this->WriteLine(":"+Srv->GetServerName()+" SYNCTS "+c->name+" "+ConvToStr(c->age));
+               //std::string modes = "";
+               //std::string params = "";
+                for (BanList::iterator b = c->bans.begin(); b != c->bans.end(); b++)
+                {
+                       modes.append("b");
+                       params.append(b->data).append(" ");
+                }
+               /* XXX: Send each channel mode and its params -- we'll need a method for this in ModeHandler? */
+                //FOREACH_MOD(I_OnSyncChannel,OnSyncChannel(c->second,(Module*)TreeProtocolModule,(void*)this));
+               this->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" "+ConvToStr(c->age)+" +"+chanmodes(c,true)+modes+" "+params);
        }
 
        /* Send G, Q, Z and E lines */
@@ -1314,18 +1742,11 @@ class TreeSocket : public InspSocket
                for (chan_hash::iterator c = chanlist.begin(); c != chanlist.end(); c++, iterations++)
                {
                        SendFJoins(Current, c->second);
-                       snprintf(data,MAXBUF,":%s FMODE %s +%s",sn,c->second->name,chanmodes(c->second,true));
-                       this->WriteLine(data);
                        if (*c->second->topic)
                        {
                                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);
                        }
-                       for (BanList::iterator b = c->second->bans.begin(); b != c->second->bans.end(); b++)
-                       {
-                               snprintf(data,MAXBUF,":%s FMODE %s +b %s",sn,c->second->name,b->data);
-                               this->WriteLine(data);
-                       }
                        FOREACH_MOD(I_OnSyncChannel,OnSyncChannel(c->second,(Module*)TreeProtocolModule,(void*)this));
                        list.clear();
                        c->second->GetExtList(list);
@@ -1344,9 +1765,9 @@ class TreeSocket : public InspSocket
                int iterations = 0;
                for (user_hash::iterator u = clientlist.begin(); u != clientlist.end(); u++, iterations++)
                {
-                       if (u->second->registered == 7)
+                       if (u->second->registered == REG_ALL)
                        {
-                               snprintf(data,MAXBUF,":%s NICK %lu %s %s %s %s +%s %s :%s",u->second->server,(unsigned long)u->second->age,u->second->nick,u->second->host,u->second->dhost,u->second->ident,u->second->FormatModes(),inet_ntoa(u->second->ip4),u->second->fullname);
+                               snprintf(data,MAXBUF,":%s NICK %lu %s %s %s %s +%s %s :%s",u->second->server,(unsigned long)u->second->age,u->second->nick,u->second->host,u->second->dhost,u->second->ident,u->second->FormatModes(),insp_ntoa(u->second->ip4),u->second->fullname);
                                this->WriteLine(data);
                                if (*u->second->oper)
                                {
@@ -1496,6 +1917,43 @@ class TreeSocket : public InspSocket
                return false;
        }
 
+       bool Stats(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
+                */
+               if (params.size() > 1)
+               {
+                       if (Srv->MatchText(Srv->GetServerName(), params[1]))
+                       {
+                               /* It's for our server */
+                               string_list results;
+                               userrec* source = Srv->FindNick(prefix);
+                               if (source)
+                               {
+                                       std::deque<std::string> par;
+                                       par.push_back(prefix);
+                                       par.push_back("");
+                                       DoStats(*(params[0].c_str()), source, results);
+                                       for (size_t i = 0; i < results.size(); i++)
+                                       {
+                                               par[1] = "::" + results[i];
+                                               DoOneToOne(Srv->GetServerName(), "PUSH",par, source->server);
+                                       }
+                               }
+                       }
+                       else
+                       {
+                               /* Pass it on */
+                               userrec* source = Srv->FindNick(prefix);
+                               if (source)
+                                       DoOneToOne(prefix, "STATS", params, params[1]);
+                       }
+               }
+               return true;
+       }
+
+
        /* Because the core won't let users or even SERVERS set +o,
         * we use the OPERTYPE command to do this.
         */
@@ -1802,7 +2260,7 @@ class TreeSocket : public InspSocket
                        if (params.size() == 1)
                        {
                                userrec* x = Srv->FindNick(params[0]);
-                               if ((x) && (x->fd > -1))
+                               if ((x) && (IS_LOCAL(x)))
                                {
                                        userrec* x = Srv->FindNick(params[0]);
                                        log(DEBUG,"Got IDLE");
@@ -1828,14 +2286,14 @@ class TreeSocket : public InspSocket
                        {
                                std::string who_did_the_whois = params[0];
                                userrec* who_to_send_to = Srv->FindNick(who_did_the_whois);
-                               if ((who_to_send_to) && (who_to_send_to->fd > -1))
+                               if ((who_to_send_to) && (IS_LOCAL(who_to_send_to)))
                                {
                                        log(DEBUG,"Got final IDLE");
                                        // an incoming reply to a whois we sent out
                                        std::string nick_whoised = prefix;
                                        unsigned long signon = atoi(params[1].c_str());
                                        unsigned long idle = atoi(params[2].c_str());
-                                       if ((who_to_send_to) && (who_to_send_to->fd > -1))
+                                       if ((who_to_send_to) && (IS_LOCAL(who_to_send_to)))
                                                do_whois(who_to_send_to,u,signon,idle,nick_whoised.c_str());
                                }
                                else
@@ -1860,15 +2318,7 @@ class TreeSocket : public InspSocket
 
                if (IS_LOCAL(u))
                {
-                       // push the raw to the user
-                       if (Srv->IsUlined(prefix))
-                       {
-                               ::Write(u->fd,"%s",params[1].c_str());
-                       }
-                       else
-                       {
-                               log(DEBUG,"PUSH from non-ulined server dropped into the bit-bucket:  :%s PUSH %s :%s",prefix.c_str(),params[0].c_str(),params[1].c_str());
-                       }
+                       ::Write(u->fd,"%s",params[1].c_str());
                }
                else
                {
@@ -2094,72 +2544,13 @@ class TreeSocket : public InspSocket
                return false;
        }
 
-       void Split(std::string line, bool stripcolon, std::deque<std::string> &n)
+       void Split(std::string line, std::deque<std::string> &n)
        {
-               // we don't do anything with a line > 2048
-               if (line.length() > 2048)
-               {
-                       log(DEBUG,"Line too long!");
-                       return;
-               }
-               if (!strchr(line.c_str(),' '))
-               {
-                       n.push_back(line);
-                       return;
-               }
-               std::stringstream s(line);
-               int count = 0;
-               char param[1024];
-               char* pptr = param;
-
                n.clear();
-               int item = 0;
-               while (!s.eof())
-               {
-                       char c = 0;
-                       s.get(c);
-                       if (c == ' ')
-                       {
-                               *pptr = 0;
-                               if (*param)
-                                       n.push_back(param);
-                               *param = count = 0;
-                               pptr = param;
-                               item++;
-                       }
-                       else
-                       {
-                               if (!s.eof())
-                               {
-                                       *pptr++ = c;
-                                       count++;
-                               }
-                               if ((*param == ':') && (count == 1) && (item > 0))
-                               {
-                                       *param = count = 0;
-                                       pptr = param;
-                                       while (!s.eof())
-                                       {
-                                               s.get(c);
-                                               if (!s.eof())
-                                               {
-                                                       *pptr++ = c;
-                                                       count++;
-                                               }
-                                       }
-                                       *pptr = 0;
-                                       n.push_back(param);
-                                       *param = count = 0;
-                                       pptr = param;
-                               }
-                       }
-               }
-               *pptr = 0;
-               if (*param)
-               {
+               irc::tokenstream tokens(line);
+               std::string param;
+               while ((param = tokens.GetToken()) != "")
                        n.push_back(param);
-               }
-
                return;
        }
 
@@ -2176,7 +2567,7 @@ class TreeSocket : public InspSocket
                
                log(DEBUG,"IN: %s", line.c_str());
                
-               this->Split(line.c_str(),true,params);
+               this->Split(line.c_str(),params);
                        
                if ((params[0][0] == ':') && (params.size() > 1))
                {
@@ -2369,9 +2760,9 @@ class TreeSocket : public InspSocket
                                {
                                        return this->ForceJoin(prefix,params);
                                }
-                               else if (command == "SYNCTS")
+                               else if (command == "STATS")
                                {
-                                       return this->SyncChannelTS(prefix,params);
+                                       return this->Stats(prefix, params);
                                }
                                else if (command == "SERVER")
                                {
@@ -2564,10 +2955,10 @@ class TreeSocket : public InspSocket
                                                }
                                                // its a user
                                                target = who->server;
-                                               char* strparams[127];
+                                               const char* strparams[127];
                                                for (unsigned int q = 0; q < params.size(); q++)
                                                {
-                                                       strparams[q] = (char*)params[q].c_str();
+                                                       strparams[q] = params[q].c_str();
                                                }
                                                if (!Srv->CallCommandHandler(command.c_str(), strparams, params.size(), who))
                                                {
@@ -2652,7 +3043,7 @@ class TreeSocket : public InspSocket
                        /* XXX: Fixme: blocks for a very short amount of time,
                         * we should cache these on rehash/startup
                         */
-                       if (CleanAndResolve(resolved_host,i->IPAddr.c_str(),true))
+                       if (CleanAndResolve(resolved_host,i->IPAddr.c_str(),true,1))
                        {
                                if (std::string(resolved_host) == ip)
                                {
@@ -2706,14 +3097,14 @@ bool DoOneToAllButSenderRaw(std::string data, std::string omit, std::string pref
        TreeServer* omitroute = BestRouteTo(omit);
        if ((command == "NOTICE") || (command == "PRIVMSG"))
        {
-               if ((params.size() >= 2) && (*(params[0].c_str()) != '$'))
+               if (params.size() >= 2)
                {
                        /* Prefixes */
                        if ((*(params[0].c_str()) == '@') || (*(params[0].c_str()) == '%') || (*(params[0].c_str()) == '+'))
                        {
                                params[0] = params[0].substr(1, params[0].length()-1);
                        }
-                       if (*(params[0].c_str()) != '#')
+                       if ((*(params[0].c_str()) != '#') && (*(params[0].c_str()) != '$'))
                        {
                                // special routing for private messages/notices
                                userrec* d = Srv->FindNick(params[0]);
@@ -2726,6 +3117,14 @@ bool DoOneToAllButSenderRaw(std::string data, std::string omit, std::string pref
                                        return true;
                                }
                        }
+                       else if (*(params[0].c_str()) == '$')
+                       {
+                               std::deque<std::string> par;
+                               par.push_back(params[0]);
+                               par.push_back(":"+params[1]);
+                               DoOneToAllButSender(prefix,command.c_str(),par,omitroute->GetName());
+                               return true;
+                       }
                        else
                        {
                                log(DEBUG,"Channel privmsg going to chan %s",params[0].c_str());
@@ -2979,14 +3378,14 @@ class ModuleSpanningTree : public Module
                return serverlist.size();
        }
 
-       void HandleLinks(char** parameters, int pcnt, userrec* user)
+       void HandleLinks(const char** parameters, int pcnt, userrec* user)
        {
                ShowLinks(TreeRoot,user,0);
                WriteServ(user->fd,"365 %s * :End of /LINKS list.",user->nick);
                return;
        }
 
-       void HandleLusers(char** parameters, int pcnt, userrec* user)
+       void HandleLusers(const char** parameters, int pcnt, userrec* user)
        {
                unsigned int n_users = usercnt();
 
@@ -2997,9 +3396,12 @@ class ModuleSpanningTree : public Module
                        max_global = n_users;
 
                WriteServ(user->fd,"251 %s :There are %d users and %d invisible on %d servers",user->nick,n_users-usercount_invisible(),usercount_invisible(),this->CountServs());
-               WriteServ(user->fd,"252 %s %d :operator(s) online",user->nick,usercount_opers());
-               WriteServ(user->fd,"253 %s %d :unknown connections",user->nick,usercount_unknown());
-               WriteServ(user->fd,"254 %s %d :channels formed",user->nick,chancount());
+               if (usercount_opers())
+                       WriteServ(user->fd,"252 %s %d :operator(s) online",user->nick,usercount_opers());
+               if (usercount_unknown())
+                       WriteServ(user->fd,"253 %s %d :unknown connections",user->nick,usercount_unknown());
+               if (chancount())
+                       WriteServ(user->fd,"254 %s %d :channels formed",user->nick,chancount());
                WriteServ(user->fd,"254 %s :I have %d clients and %d servers",user->nick,local_count(),this->CountLocalServs());
                WriteServ(user->fd,"265 %s :Current Local Users: %d  Max: %d",user->nick,local_count(),max_local);
                WriteServ(user->fd,"266 %s :Current Global Users: %d  Max: %d",user->nick,n_users,max_global);
@@ -3062,6 +3464,26 @@ class ModuleSpanningTree : public Module
                }
        }
 
+       int HandleStats(const char** parameters, int pcnt, userrec* user)
+       {
+               if (pcnt > 1)
+               {
+                       /* Remote STATS, the server is within the 2nd parameter */
+                       std::deque<std::string> params;
+                       params.push_back(parameters[0]);
+                       params.push_back(parameters[1]);
+                       /* Send it out remotely, generate no reply yet */
+                       TreeServer* s = FindServerMask(parameters[1]);
+                       if (s)
+                       {
+                               params[1] = s->GetName();
+                               DoOneToOne(user->nick, "STATS", params, s->GetName());
+                       }
+                       return 1;
+               }
+               return 0;
+       }
+
        // Ok, prepare to be confused.
        // After much mulling over how to approach this, it struck me that
        // the 'usual' way of doing a /MAP isnt the best way. Instead of
@@ -3071,7 +3493,7 @@ class ModuleSpanningTree : public Module
        // (a character matrix), then draw the branches as a series of "L" shapes
        // from the nodes. This is not only friendlier on CPU it uses less stack.
 
-       void HandleMap(char** parameters, int pcnt, userrec* user)
+       void HandleMap(const char** parameters, int pcnt, userrec* user)
        {
                // This array represents a virtual screen which we will
                // "scratch" draw to, as the console device of an irc
@@ -3127,7 +3549,7 @@ class ModuleSpanningTree : public Module
                return;
        }
 
-       int HandleSquit(char** parameters, int pcnt, userrec* user)
+       int HandleSquit(const char** parameters, int pcnt, userrec* user)
        {
                TreeServer* s = FindServerMask(parameters[0]);
                if (s)
@@ -3157,9 +3579,9 @@ class ModuleSpanningTree : public Module
                return 1;
        }
 
-       int HandleTime(char** parameters, int pcnt, userrec* user)
+       int HandleTime(const char** parameters, int pcnt, userrec* user)
        {
-               if ((user->fd > -1) && (pcnt))
+               if ((IS_LOCAL(user)) && (pcnt))
                {
                        TreeServer* found = FindServerMask(parameters[0]);
                        if (found)
@@ -3181,9 +3603,9 @@ class ModuleSpanningTree : public Module
                return 1;
        }
 
-       int HandleRemoteWhois(char** parameters, int pcnt, userrec* user)
+       int HandleRemoteWhois(const char** parameters, int pcnt, userrec* user)
        {
-               if ((user->fd > -1) && (pcnt > 1))
+               if ((IS_LOCAL(user)) && (pcnt > 1))
                {
                        userrec* remote = Srv->FindNick(parameters[1]);
                        if ((remote) && (remote->fd < 0))
@@ -3259,7 +3681,7 @@ class ModuleSpanningTree : public Module
                }
        }
 
-       int HandleVersion(char** parameters, int pcnt, userrec* user)
+       int HandleVersion(const char** parameters, int pcnt, userrec* user)
        {
                // we've already checked if pcnt > 0, so this is safe
                TreeServer* found = FindServerMask(parameters[0]);
@@ -3296,7 +3718,7 @@ class ModuleSpanningTree : public Module
                return 1;
        }
        
-       int HandleConnect(char** parameters, int pcnt, userrec* user)
+       int HandleConnect(const char** parameters, int pcnt, userrec* user)
        {
                for (std::vector<Link>::iterator x = LinkBlocks.begin(); x < LinkBlocks.end(); x++)
                {
@@ -3329,23 +3751,23 @@ class ModuleSpanningTree : public Module
                return 1;
        }
 
-       virtual int OnStats(char statschar, userrec* user)
+       virtual int OnStats(char statschar, userrec* user, string_list &results)
        {
                if (statschar == 'c')
                {
                        for (unsigned int i = 0; i < LinkBlocks.size(); i++)
                        {
-                               WriteServ(user->fd,"213 %s C *@%s * %s %d 0 %c%c%c",user->nick,(LinkBlocks[i].HiddenFromStats ? "<hidden>" : LinkBlocks[i].IPAddr).c_str(),LinkBlocks[i].Name.c_str(),LinkBlocks[i].Port,(LinkBlocks[i].EncryptionKey != "" ? 'e' : '-'),(LinkBlocks[i].AutoConnect ? 'a' : '-'),'s');
-                               WriteServ(user->fd,"244 %s H * * %s",user->nick,LinkBlocks[i].Name.c_str());
+                               results.push_back(Srv->GetServerName()+" 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(Srv->GetServerName()+" 244 "+user->nick+" H * * "+LinkBlocks[i].Name.c_str());
                        }
-                       WriteServ(user->fd,"219 %s %c :End of /STATS report",user->nick,statschar);
-                       WriteOpers("*** Notice: Stats '%c' requested by %s (%s@%s)",statschar,user->nick,user->ident,user->host);
+                       results.push_back(Srv->GetServerName()+" 219 "+user->nick+" "+statschar+" :End of /STATS report");
+                       WriteOpers("*** Notice: %s '%c' requested by %s (%s@%s)",(!strcmp(user->server,Config->ServerName) ? "Stats" : "Remote stats"),statschar,user->nick,user->ident,user->host);
                        return 1;
                }
                return 0;
        }
 
-       virtual int OnPreCommand(const std::string &command, char **parameters, int pcnt, userrec *user, bool validated)
+       virtual int OnPreCommand(const std::string &command, const char** parameters, int pcnt, userrec *user, bool validated)
        {
                /* If the command doesnt appear to be valid, we dont want to mess with it. */
                if (!validated)
@@ -3355,6 +3777,10 @@ class ModuleSpanningTree : public Module
                {
                        return this->HandleConnect(parameters,pcnt,user);
                }
+               else if (command == "STATS")
+               {
+                       return this->HandleStats(parameters,pcnt,user);
+               }
                else if (command == "SQUIT")
                {
                        return this->HandleSquit(parameters,pcnt,user);
@@ -3428,7 +3854,7 @@ class ModuleSpanningTree : public Module
 
        virtual void OnUserInvite(userrec* source,userrec* dest,chanrec* channel)
        {
-               if (source->fd > -1)
+               if (IS_LOCAL(source))
                {
                        std::deque<std::string> params;
                        params.push_back(dest->nick);
@@ -3447,7 +3873,7 @@ class ModuleSpanningTree : public Module
 
        virtual void OnWallops(userrec* user, const std::string &text)
        {
-               if (user->fd > -1)
+               if (IS_LOCAL(user))
                {
                        std::deque<std::string> params;
                        params.push_back(":"+text);
@@ -3460,7 +3886,7 @@ class ModuleSpanningTree : public Module
                if (target_type == TYPE_USER)
                {
                        userrec* d = (userrec*)dest;
-                       if ((d->fd < 0) && (user->fd > -1))
+                       if ((d->fd < 0) && (IS_LOCAL(user)))
                        {
                                std::deque<std::string> params;
                                params.clear();
@@ -3469,9 +3895,9 @@ class ModuleSpanningTree : public Module
                                DoOneToOne(user->nick,"NOTICE",params,d->server);
                        }
                }
-               else
+               else if (target_type == TYPE_CHANNEL)
                {
-                       if (user->fd > -1)
+                       if (IS_LOCAL(user))
                        {
                                chanrec *c = (chanrec*)dest;
                                std::string cname = c->name;
@@ -3488,6 +3914,17 @@ class ModuleSpanningTree : public Module
                                }
                        }
                }
+                else if (target_type == TYPE_SERVER)
+               {
+                       if (IS_LOCAL(user))
+                       {
+                               char* target = (char*)dest;
+                               std::deque<std::string> par;
+                               par.push_back(target);
+                               par.push_back(":"+text);
+                               DoOneToMany(user->nick,"NOTICE",par);
+                       }
+               }
        }
 
        virtual void OnUserMessage(userrec* user, void* dest, int target_type, const std::string &text, char status)
@@ -3497,7 +3934,7 @@ class ModuleSpanningTree : public Module
                        // route private messages which are targetted at clients only to the server
                        // which needs to receive them
                        userrec* d = (userrec*)dest;
-                       if ((d->fd < 0) && (user->fd > -1))
+                       if ((d->fd < 0) && (IS_LOCAL(user)))
                        {
                                std::deque<std::string> params;
                                params.clear();
@@ -3506,9 +3943,9 @@ class ModuleSpanningTree : public Module
                                DoOneToOne(user->nick,"PRIVMSG",params,d->server);
                        }
                }
-               else
+               else if (target_type == TYPE_CHANNEL)
                {
-                       if (user->fd > -1)
+                       if (IS_LOCAL(user))
                        {
                                chanrec *c = (chanrec*)dest;
                                std::string cname = c->name;
@@ -3525,6 +3962,17 @@ class ModuleSpanningTree : public Module
                                }
                        }
                }
+               else if (target_type == TYPE_SERVER)
+               {
+                       if (IS_LOCAL(user))
+                       {
+                               char* target = (char*)dest;
+                               std::deque<std::string> par;
+                               par.push_back(target);
+                               par.push_back(":"+text);
+                               DoOneToMany(user->nick,"PRIVMSG",par);
+                       }
+               }
        }
 
        virtual void OnBackgroundTimer(time_t curtime)
@@ -3536,7 +3984,7 @@ class ModuleSpanningTree : public Module
        virtual void OnUserJoin(userrec* user, chanrec* channel)
        {
                // Only do this for local users
-               if (user->fd > -1)
+               if (IS_LOCAL(user))
                {
                        std::deque<std::string> params;
                        params.clear();
@@ -3565,7 +4013,7 @@ class ModuleSpanningTree : public Module
        virtual void OnChangeHost(userrec* user, const std::string &newhost)
        {
                // only occurs for local clients
-               if (user->registered != 7)
+               if (user->registered != REG_ALL)
                        return;
                std::deque<std::string> params;
                params.push_back(newhost);
@@ -3575,7 +4023,7 @@ class ModuleSpanningTree : public Module
        virtual void OnChangeName(userrec* user, const std::string &gecos)
        {
                // only occurs for local clients
-               if (user->registered != 7)
+               if (user->registered != REG_ALL)
                        return;
                std::deque<std::string> params;
                params.push_back(gecos);
@@ -3584,7 +4032,7 @@ class ModuleSpanningTree : public Module
 
        virtual void OnUserPart(userrec* user, chanrec* channel, const std::string &partmessage)
        {
-               if (user->fd > -1)
+               if (IS_LOCAL(user))
                {
                        std::deque<std::string> params;
                        params.push_back(channel->name);
@@ -3597,7 +4045,7 @@ class ModuleSpanningTree : public Module
        virtual void OnUserConnect(userrec* user)
        {
                char agestr[MAXBUF];
-               if (user->fd > -1)
+               if (IS_LOCAL(user))
                {
                        std::deque<std::string> params;
                        snprintf(agestr,MAXBUF,"%lu",(unsigned long)user->age);
@@ -3607,7 +4055,7 @@ class ModuleSpanningTree : public Module
                        params.push_back(user->dhost);
                        params.push_back(user->ident);
                        params.push_back("+"+std::string(user->FormatModes()));
-                       params.push_back((char*)inet_ntoa(user->ip4));
+                       params.push_back((char*)insp_ntoa(user->ip4));
                        params.push_back(":"+std::string(user->fullname));
                        DoOneToMany(Srv->GetServerName(),"NICK",params);
 
@@ -3623,7 +4071,7 @@ class ModuleSpanningTree : public Module
 
        virtual void OnUserQuit(userrec* user, const std::string &reason)
        {
-               if ((user->fd > -1) && (user->registered == 7))
+               if ((IS_LOCAL(user)) && (user->registered == REG_ALL))
                {
                        std::deque<std::string> params;
                        params.push_back(":"+reason);
@@ -3640,7 +4088,7 @@ class ModuleSpanningTree : public Module
 
        virtual void OnUserPostNick(userrec* user, const std::string &oldnick)
        {
-               if (user->fd > -1)
+               if (IS_LOCAL(user))
                {
                        std::deque<std::string> params;
                        params.push_back(user->nick);
@@ -3650,7 +4098,7 @@ class ModuleSpanningTree : public Module
 
        virtual void OnUserKick(userrec* source, userrec* user, chanrec* chan, const std::string &reason)
        {
-               if ((source) && (source->fd > -1))
+               if ((source) && (IS_LOCAL(source)))
                {
                        std::deque<std::string> params;
                        params.push_back(chan->name);
@@ -3698,7 +4146,7 @@ class ModuleSpanningTree : public Module
        // locally.
        virtual void OnOper(userrec* user, const std::string &opertype)
        {
-               if (user->fd > -1)
+               if (IS_LOCAL(user))
                {
                        std::deque<std::string> params;
                        params.push_back(opertype);
@@ -3708,7 +4156,7 @@ class ModuleSpanningTree : public Module
 
        void OnLine(userrec* source, const std::string &host, bool adding, char linetype, long duration, const std::string &reason)
        {
-               if (source->fd > -1)
+               if (IS_LOCAL(source))
                {
                        char type[8];
                        snprintf(type,8,"%cLINE",linetype);
@@ -3774,7 +4222,7 @@ class ModuleSpanningTree : public Module
 
        virtual void OnMode(userrec* user, void* dest, int target_type, const std::string &text)
        {
-               if ((user->fd > -1) && (user->registered == 7))
+               if ((IS_LOCAL(user)) && (user->registered == REG_ALL))
                {
                        if (target_type == TYPE_USER)
                        {
@@ -3823,12 +4271,12 @@ class ModuleSpanningTree : public Module
                        if (target_type == TYPE_USER)
                        {
                                userrec* u = (userrec*)target;
-                               s->WriteLine(":"+Srv->GetServerName()+" FMODE "+u->nick+" "+modeline);
+                               s->WriteLine(":"+Srv->GetServerName()+" FMODE "+u->nick+" "+ConvToStr(u->age)+" "+modeline);
                        }
                        else
                        {
                                chanrec* c = (chanrec*)target;
-                               s->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" "+modeline);
+                               s->WriteLine(":"+Srv->GetServerName()+" FMODE "+c->name+" "+ConvToStr(c->age)+" "+modeline);
                        }
                }
        }
@@ -3870,6 +4318,22 @@ class ModuleSpanningTree : public Module
                        std::deque<std::string>* params = (std::deque<std::string>*)event->GetData();
                        if (params->size() < 2)
                                return;
+                       // Insert the TS value of the object, either userrec or chanrec
+                       time_t ourTS = 0;
+                       userrec* a = Srv->FindNick((*params)[0]);
+                       if (a)
+                       {
+                               ourTS = a->age;
+                       }
+                       else
+                       {
+                               chanrec* a = Srv->FindChannel((*params)[0]);
+                               if (a)
+                               {
+                                       ourTS = a->age;
+                               }
+                       }
+                       params->insert(params->begin() + 1,ConvToStr(ourTS));
                        DoOneToMany(Srv->GetServerName(),"FMODE",*params);
                }
        }