Merge pull request #976 from SaberUK/master+fix-xline-db

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

diff --git a/src/wildcard.cpp b/src/wildcard.cpp
index b64d2d6e89b633f21e30e8748a6d5f48f038a8b2..6711f953aa9b7a311d0c91a6a741769ebf437422 100644 (file)
--- a/src/wildcard.cpp
+++ b/src/wildcard.cpp
@@ -19,15 +19,12 @@
  */
 
 
-/* $Core */
-
 #include "inspircd.h"
-#include "hashcomp.h"
-#include "inspstring.h"
 
-static bool match_internal(const unsigned char *str, const unsigned char *mask, unsigned const char *map)
+static bool MatchInternal(const unsigned char* str, const unsigned char* mask, unsigned const char* map)
 {
-       unsigned char *cp = NULL, *mp = NULL;
+       unsigned char* cp = NULL;
+       unsigned char* mp = NULL;
        unsigned char* string = (unsigned char*)str;
        unsigned char* wild = (unsigned char*)mask;
 
@@ -74,45 +71,38 @@ static bool match_internal(const unsigned char *str, const unsigned char *mask,
        return !*wild;
 }
 
-/********************************************************************
- * Below here is all wrappers around match_internal
- ********************************************************************/
+// Below here is all wrappers around MatchInternal
 
-bool InspIRCd::Match(const std::string &str, const std::string &mask, unsigned const char *map)
+bool InspIRCd::Match(const std::string& str, const std::string& mask, unsigned const char* map)
 {
        if (!map)
                map = national_case_insensitive_map;
 
-       return match_internal((const unsigned char *)str.c_str(), (const unsigned char *)mask.c_str(), map);
+       return MatchInternal((const unsigned char*)str.c_str(), (const unsigned char*)mask.c_str(), map);
 }
 
-bool InspIRCd::Match(const char *str, const char *mask, unsigned const char *map)
+bool InspIRCd::Match(const char* str, const char* mask, unsigned const char* map)
 {
        if (!map)
                map = national_case_insensitive_map;
-       return match_internal((const unsigned char *)str, (const unsigned char *)mask, map);
+
+       return MatchInternal((const unsigned char*)str, (const unsigned char*)mask, map);
 }
 
-bool InspIRCd::MatchCIDR(const std::string &str, const std::string &mask, unsigned const char *map)
+bool InspIRCd::MatchCIDR(const std::string& str, const std::string& mask, unsigned const char* map)
 {
        if (irc::sockets::MatchCIDR(str, mask, true))
                return true;
 
-       if (!map)
-               map = national_case_insensitive_map;
-
        // Fall back to regular match
        return InspIRCd::Match(str, mask, map);
 }
 
-bool InspIRCd::MatchCIDR(const char *str, const char *mask, unsigned const char *map)
+bool InspIRCd::MatchCIDR(const char* str, const char* mask, unsigned const char* map)
 {
        if (irc::sockets::MatchCIDR(str, mask, true))
                return true;
 
-       if (!map)
-               map = national_case_insensitive_map;
-
        // Fall back to regular match
        return InspIRCd::Match(str, mask, map);
 }
@@ -123,7 +113,7 @@ bool InspIRCd::MatchMask(const std::string& masks, const std::string& hostname,
        std::string mask;
        while (masklist >> mask)
        {
-               if (InspIRCd::Match(hostname, mask, ascii_case_insensitive_map) || 
+               if (InspIRCd::Match(hostname, mask, ascii_case_insensitive_map) ||
                        InspIRCd::MatchCIDR(ipaddr, mask, ascii_case_insensitive_map))
                {
                        return true;