Remove dummy API_VERSION from Version constructor

[user/henk/code/inspircd.git] / src / modules / m_password_hash.cpp

diff --git a/src/modules/m_password_hash.cpp b/src/modules/m_password_hash.cpp
index 011a592e7bab1cde54a8619b8180620f74ad83b1..617983ec4df49a56422a8876ddb8969730687b62 100644 (file)
--- a/src/modules/m_password_hash.cpp
+++ b/src/modules/m_password_hash.cpp
@@ -12,7 +12,6 @@
  */
 
 /* $ModDesc: Allows for hashed oper passwords */
-/* $ModDep: m_hash.h */
 
 #include "inspircd.h"
 #include "m_hash.h"
@@ -29,6 +28,7 @@ class CommandMkpasswd : public Command
        CommandMkpasswd(Module* Creator, hashymodules &h, std::deque<std::string> &n) : Command(Creator, "MKPASSWD", 2), hashers(h), names(n)
        {
                syntax = "<hashtype> <any-text>";
+               Penalty = 5;
        }
 
        void MakeHash(User* user, const char* algo, const char* stuff)
@@ -37,10 +37,10 @@ class CommandMkpasswd : public Command
                hashymodules::iterator x = hashers.find(algo);
                if (x != hashers.end())
                {
-                       /* Yup, reset it first (Always ALWAYS do this) */
-                       HashResetRequest(creator, x->second).Send();
+                       HashRequest hash(creator, x->second, stuff);
                        /* Now attempt to generate a hash */
-                       user->WriteServ("NOTICE %s :%s hashed password for %s is %s",user->nick.c_str(), algo, stuff, HashSumRequest(creator, x->second, stuff).Send() );
+                       user->WriteServ("NOTICE %s :%s hashed password for %s is %s",
+                               user->nick.c_str(), algo, stuff, hash.hex().c_str());
                }
                else if (names.empty())
                {
@@ -57,9 +57,6 @@ class CommandMkpasswd : public Command
        CmdResult Handle (const std::vector<std::string>& parameters, User *user)
        {
                MakeHash(user, parameters[0].c_str(), parameters[1].c_str());
-               // this hashing could take some time, increasing server load.
-               // Slow down the user if they are trying to flood mkpasswd requests
-               user->IncreasePenalty(5);
 
                return CMD_SUCCESS;
        }
@@ -96,7 +93,7 @@ class ModuleOperHash : public Module
                                /* Make a request to it for its name, its implementing
                                 * HashRequest so we know its safe to do this
                                 */
-                               std::string name = HashNameRequest(this, *m).Send();
+                               std::string name = HashNameRequest(this, *m).response;
                                /* Build a map of them */
                                hashers[name.c_str()] = *m;
                                names.push_back(name);
@@ -117,12 +114,11 @@ class ModuleOperHash : public Module
        }
 
 
-       virtual void OnLoadModule(Module* mod, const std::string& name)
+       virtual void OnLoadModule(Module* mod)
        {
                if (ServerInstance->Modules->ModuleHasInterface(mod, "HashRequest"))
                {
-                       ServerInstance->Logs->Log("m_password-hash",DEBUG, "Post-load registering hasher: %s", name.c_str());
-                       std::string sname = HashNameRequest(this, mod).Send();
+                       std::string sname = HashNameRequest(this, mod).response;
                        hashers[sname.c_str()] = mod;
                        names.push_back(sname);
                        if (!diduseiface)
@@ -141,10 +137,8 @@ class ModuleOperHash : public Module
                /* Is this a valid hash name? (case insensitive) */
                if (x != hashers.end())
                {
-                       /* Reset the hashing module */
-                       HashResetRequest(this, x->second).Send();
                        /* Compare the hash in the config to the generated hash */
-                       if (!strcasecmp(data.c_str(), HashSumRequest(this, x->second, input.c_str()).Send()))
+                       if (!strcasecmp(data.c_str(), HashRequest(this, x->second, input).hex().c_str()))
                                return MOD_RES_ALLOW;
                        /* No match, and must be hashed, forbid */
                        else
@@ -157,7 +151,7 @@ class ModuleOperHash : public Module
 
        virtual Version GetVersion()
        {
-               return Version("Allows for hashed oper passwords",VF_VENDOR,API_VERSION);
+               return Version("Allows for hashed oper passwords",VF_VENDOR);
        }
 };