Convert WriteNumeric() calls to pass the parameters of the numeric as method parameters

[user/henk/code/inspircd.git] / src / coremods / core_stats.cpp

diff --git a/src/coremods/core_stats.cpp b/src/coremods/core_stats.cpp
index 468e230f8c255411a58dfebc577689ec54cc562d..9fb232bc0192bb7a2e1d0210f1e580a09042df3d 100644 (file)
--- a/src/coremods/core_stats.cpp
+++ b/src/coremods/core_stats.cpp
@@ -54,11 +54,11 @@ static void GenerateStatsLl(User* user, string_list& results, char c)
 {
        results.push_back(InspIRCd::Format("211 %s nick[ident@%s] sendq cmds_out bytes_out cmds_in bytes_in time_open", user->nick.c_str(), (c == 'l' ? "host" : "ip")));
 
-       const LocalUserList& list = ServerInstance->Users->local_users;
-       for (LocalUserList::const_iterator i = list.begin(); i != list.end(); ++i)
+       const UserManager::LocalList& list = ServerInstance->Users.GetLocalUsers();
+       for (UserManager::LocalList::const_iterator i = list.begin(); i != list.end(); ++i)
        {
                LocalUser* u = *i;
-               results.push_back("211 "+user->nick+" "+u->nick+"["+u->ident+"@"+(c == 'l' ? u->dhost : u->GetIPString())+"] "+ConvToStr(u->eh.getSendQSize())+" "+ConvToStr(u->cmds_out)+" "+ConvToStr(u->bytes_out)+" "+ConvToStr(u->cmds_in)+" "+ConvToStr(u->bytes_in)+" "+ConvToStr(ServerInstance->Time() - u->age));
+               results.push_back("211 "+user->nick+" "+u->nick+"["+u->ident+"@"+(c == 'l' ? u->dhost : u->GetIPString())+"] "+ConvToStr(u->eh.getSendQSize())+" "+ConvToStr(u->cmds_out)+" "+ConvToStr(u->bytes_out)+" "+ConvToStr(u->cmds_in)+" "+ConvToStr(u->bytes_in)+" "+ConvToStr(ServerInstance->Time() - u->signon));
        }
 }
 
@@ -99,6 +99,11 @@ void CommandStats::DoStats(char statschar, User* user, string_list &results)
                                std::string ip = ls->bind_addr;
                                if (ip.empty())
                                        ip.assign("*");
+                               else if (ip.find_first_of(':') != std::string::npos)
+                               {
+                                       ip.insert(ip.begin(), '[');
+                                       ip.insert(ip.end(),  ']');
+                               }
                                std::string type = ls->bind_tag->getString("type", "clients");
                                std::string hook = ls->bind_tag->getString("ssl", "plaintext");
 
@@ -335,7 +340,7 @@ void CommandStats::DoStats(char statschar, User* user, string_list &results)
                                tag->init();
                                std::string umodes;
                                std::string cmodes;
-                               for(char c='A'; c < 'z'; c++)
+                               for(char c='A'; c <= 'z'; c++)
                                {
                                        ModeHandler* mh = ServerInstance->Modes->FindMode(c, MODETYPE_USER);
                                        if (mh && mh->NeedsOper() && tag->AllowedUserModes[c - 'A'])
@@ -378,7 +383,13 @@ void CommandStats::DoStats(char statschar, User* user, string_list &results)
 CmdResult CommandStats::Handle (const std::vector<std::string>& parameters, User *user)
 {
        if (parameters.size() > 1 && parameters[1] != ServerInstance->Config->ServerName)
+       {
+               // Give extra penalty if a non-oper does /STATS <remoteserver>
+               LocalUser* localuser = IS_LOCAL(user);
+               if ((localuser) && (!user->IsOper()))
+                       localuser->CommandFloodPenalty += 2000;
                return CMD_SUCCESS;
+       }
        string_list values;
        char search = parameters[0][0];
        DoStats(search, user, values);