Fix bug spotted by KingTarquin, if an empty string is given to /stats with "STATS...

[user/henk/code/inspircd.git] / src / cidr.cpp

diff --git a/src/cidr.cpp b/src/cidr.cpp
index 90654ee820e70da4a5d95c27bd4c8dc00868cb49..669ccd8d27f6df9fd685c1b29728f407c4ea2460 100644 (file)
--- a/src/cidr.cpp
+++ b/src/cidr.cpp
@@ -11,10 +11,9 @@
  * ---------------------------------------------------
  */
 
-/* $Core: libIRCDcidr */
+/* $Core */
 
 #include "inspircd.h"
-#include "wildcard.h"
 
 /* Used when comparing CIDR masks for the modulus bits left over.
  * A lot of ircd's seem to do this:
@@ -91,7 +90,7 @@ bool irc::sockets::MatchCIDR(const std::string &address, const std::string &cidr
                         * symbols, and recursively call MatchCIDR without
                         * username matching enabled to match the host part.
                         */
-                       return (match(address.substr(0, username_addr_pos), cidr_mask.substr(0, username_mask_pos)) &&
+                       return (InspIRCd::Match(address.substr(0, username_addr_pos), cidr_mask.substr(0, username_mask_pos), ascii_case_insensitive_map) &&
                                        MatchCIDR(address.substr(username_addr_pos + 1), cidr_mask.substr(username_mask_pos + 1), false));
                }
                else
@@ -100,6 +99,11 @@ bool irc::sockets::MatchCIDR(const std::string &address, const std::string &cidr
                        cidr_copy = cidr_mask.substr(username_mask_pos + 1);
                }
        }
+       else
+       {
+               address_copy.assign(address);
+               cidr_copy.assign(cidr_mask);
+       }
 
        in_addr  address_in4;
        in_addr  mask_in4;
@@ -109,6 +113,7 @@ bool irc::sockets::MatchCIDR(const std::string &address, const std::string &cidr
        if (bits_chars != std::string::npos)
        {
                bits = atoi(cidr_copy.substr(bits_chars + 1).c_str());
+               cidr_copy.erase(bits_chars, cidr_copy.length() - bits_chars);
        }
        else
        {