Fix not checking for server names case insensitively.

diff --git a/src/coremods/core_info/cmd_admin.cpp b/src/coremods/core_info/cmd_admin.cpp
index 44827555d5488ee7db133022d1765cd69e4247c0..58a799d7704c1f3fdb426091ebfa8f42b92c4bcb 100644 (file)
--- a/src/coremods/core_info/cmd_admin.cpp
+++ b/src/coremods/core_info/cmd_admin.cpp
@@ -32,7 +32,7 @@ CommandAdmin::CommandAdmin(Module* parent)
  */
 CmdResult CommandAdmin::Handle(User* user, const Params& parameters)
 {
-       if (parameters.size() > 0 && parameters[0] != ServerInstance->Config->ServerName)
+       if (parameters.size() > 0 && !irc::equals(parameters[0], ServerInstance->Config->ServerName))
                return CMD_SUCCESS;
        user->WriteRemoteNumeric(RPL_ADMINME, InspIRCd::Format("Administrative info for %s", ServerInstance->Config->ServerName.c_str()));
        if (!AdminName.empty())

diff --git a/src/coremods/core_info/cmd_info.cpp b/src/coremods/core_info/cmd_info.cpp
index ee079493488e81005f653023b9978589192f3956..431175f7a10fc038b755e9018ccea97b5416f55d 100644 (file)
--- a/src/coremods/core_info/cmd_info.cpp
+++ b/src/coremods/core_info/cmd_info.cpp
@@ -78,7 +78,7 @@ static const char* const lines[] = {
  */
 CmdResult CommandInfo::Handle(User* user, const Params& parameters)
 {
-       if (parameters.size() > 0 && parameters[0] != ServerInstance->Config->ServerName)
+       if (parameters.size() > 0 && !irc::equals(parameters[0], ServerInstance->Config->ServerName))
                return CMD_SUCCESS;
 
        int i=0;

diff --git a/src/coremods/core_info/cmd_modules.cpp b/src/coremods/core_info/cmd_modules.cpp
index 7212e9525331e173403aca1dec4665d4122e99bc..d68c8b9fdc00ccdeae035b2fd1a75ebf2f64bdea 100644 (file)
--- a/src/coremods/core_info/cmd_modules.cpp
+++ b/src/coremods/core_info/cmd_modules.cpp
@@ -42,7 +42,7 @@ CmdResult CommandModules::Handle(User* user, const Params& parameters)
 {
        // Don't ask remote servers about their modules unless the local user asking is an oper
        // 2.0 asks anyway, so let's handle that the same way
-       bool for_us = (parameters.empty() || parameters[0] == ServerInstance->Config->ServerName);
+       bool for_us = (parameters.empty() || irc::equals(parameters[0], ServerInstance->Config->ServerName));
        if ((!for_us) || (!IS_LOCAL(user)))
        {
                if (!user->IsOper())

diff --git a/src/coremods/core_info/cmd_motd.cpp b/src/coremods/core_info/cmd_motd.cpp
index e4c57553947cac246fa3a881b2d92aa9432b76e9..eb6460f00bfeeb130aca2a82f31439b69a869192 100644 (file)
--- a/src/coremods/core_info/cmd_motd.cpp
+++ b/src/coremods/core_info/cmd_motd.cpp
@@ -31,7 +31,7 @@ CommandMotd::CommandMotd(Module* parent)
  */
 CmdResult CommandMotd::Handle(User* user, const Params& parameters)
 {
-       if (parameters.size() > 0 && parameters[0] != ServerInstance->Config->ServerName)
+       if (parameters.size() > 0 && !irc::equals(parameters[0], ServerInstance->Config->ServerName))
        {
                // Give extra penalty if a non-oper queries the /MOTD of a remote server
                LocalUser* localuser = IS_LOCAL(user);

diff --git a/src/coremods/core_info/cmd_time.cpp b/src/coremods/core_info/cmd_time.cpp
index 73987cf0183c24ba8f1fc91ab0745676be29a787..725ef6cb5232dd1ed26245acdc7071167bb71889 100644 (file)
--- a/src/coremods/core_info/cmd_time.cpp
+++ b/src/coremods/core_info/cmd_time.cpp
@@ -29,7 +29,7 @@ CommandTime::CommandTime(Module* parent)
 
 CmdResult CommandTime::Handle(User* user, const Params& parameters)
 {
-       if (parameters.size() > 0 && parameters[0] != ServerInstance->Config->ServerName)
+       if (parameters.size() > 0 && !irc::equals(parameters[0], ServerInstance->Config->ServerName))
                return CMD_SUCCESS;
 
        user->WriteRemoteNumeric(RPL_TIME, ServerInstance->Config->ServerName, InspIRCd::TimeString(ServerInstance->Time()));

diff --git a/src/coremods/core_stats.cpp b/src/coremods/core_stats.cpp
index 6a4427aef8caeef674b34a1acf6e20004693e684..6576aa514c505dc4b3b2e0469317f26b9ab4866b 100644 (file)
--- a/src/coremods/core_stats.cpp
+++ b/src/coremods/core_stats.cpp
@@ -378,7 +378,7 @@ void CommandStats::DoStats(Stats::Context& stats)
 
 CmdResult CommandStats::Handle(User* user, const Params& parameters)
 {
-       if (parameters.size() > 1 && parameters[1] != ServerInstance->Config->ServerName)
+       if (parameters.size() > 1 && !irc::equals(parameters[1], ServerInstance->Config->ServerName))
        {
                // Give extra penalty if a non-oper does /STATS <remoteserver>
                LocalUser* localuser = IS_LOCAL(user);

diff --git a/src/modules/m_check.cpp b/src/modules/m_check.cpp
index 1d030acc6f31bd59341e6874be812473b9419fab..2d06df199ccbc6b76840290d7f44f4fd203d7953 100644 (file)
--- a/src/modules/m_check.cpp
+++ b/src/modules/m_check.cpp
@@ -142,7 +142,7 @@ class CommandCheck : public Command
 
        CmdResult Handle(User* user, const Params& parameters) CXX11_OVERRIDE
        {
-               if (parameters.size() > 1 && parameters[1] != ServerInstance->Config->ServerName)
+               if (parameters.size() > 1 && !irc::equals(parameters[1], ServerInstance->Config->ServerName))
                        return CMD_SUCCESS;
 
                User *targuser;

diff --git a/src/modules/m_opermotd.cpp b/src/modules/m_opermotd.cpp
index 49409a1c05f5e90f33fe1f429a48d1a51d1a6cc3..591dc97a8ad0d476b520bf8e7af3debfbf4691ce 100644 (file)
--- a/src/modules/m_opermotd.cpp
+++ b/src/modules/m_opermotd.cpp
@@ -36,7 +36,7 @@ class CommandOpermotd : public Command
 
        CmdResult Handle(User* user, const Params& parameters) CXX11_OVERRIDE
        {
-               if ((parameters.empty()) || (parameters[0] == ServerInstance->Config->ServerName))
+               if ((parameters.empty()) || (irc::equals(parameters[0], ServerInstance->Config->ServerName)))
                        ShowOperMOTD(user);
                return CMD_SUCCESS;
        }