X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_password_hash.cpp;h=12769acd4d07eb28a49496fa590db166df68f7a3;hb=7107ec12d8640d35cfe3d5002db1bc1deb33625d;hp=52c8f241cb77382df7fdd016a053f0f8d4479c40;hpb=0da6b3a13def40e8fd002b9fc60f955467f6372d;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_password_hash.cpp b/src/modules/m_password_hash.cpp
index 52c8f241c..12769acd4 100644
--- a/src/modules/m_password_hash.cpp
+++ b/src/modules/m_password_hash.cpp
@@ -23,15 +23,13 @@ typedef std::map<irc::string, Module*> hashymodules;
  */
 class CommandMkpasswd : public Command
 {
-	Module* Sender;
 	hashymodules &hashers;
 	std::deque<std::string> &names;
  public:
-	CommandMkpasswd (InspIRCd* Instance, Module* S, hashymodules &h, std::deque<std::string> &n)
-		: Command(Instance,"MKPASSWD", 0, 2), Sender(S), hashers(h), names(n)
+	CommandMkpasswd(Module* Creator, hashymodules &h, std::deque<std::string> &n) : Command(Creator, "MKPASSWD", 2), hashers(h), names(n)
 	{
-		this->source = "m_password_hash.so";
 		syntax = "<hashtype> <any-text>";
+		Penalty = 5;
 	}
 
 	void MakeHash(User* user, const char* algo, const char* stuff)
@@ -41,9 +39,9 @@ class CommandMkpasswd : public Command
 		if (x != hashers.end())
 		{
 			/* Yup, reset it first (Always ALWAYS do this) */
-			HashResetRequest(Sender, x->second).Send();
+			HashResetRequest(creator, x->second).Send();
 			/* Now attempt to generate a hash */
-			user->WriteServ("NOTICE %s :%s hashed password for %s is %s",user->nick.c_str(), algo, stuff, HashSumRequest(Sender, x->second, stuff).Send() );
+			user->WriteServ("NOTICE %s :%s hashed password for %s is %s",user->nick.c_str(), algo, stuff, HashSumRequest(creator, x->second, stuff).Send() );
 		}
 		else if (names.empty())
 		{
@@ -60,26 +58,23 @@ 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_LOCALONLY;
+		return CMD_SUCCESS;
 	}
 };
 
 class ModuleOperHash : public Module
 {
 
-	CommandMkpasswd* mycommand;
+	CommandMkpasswd cmd;
 	hashymodules hashers; /* List of modules which implement HashRequest */
 	std::deque<std::string> names; /* Module names which implement HashRequest */
 
 	bool diduseiface; /* If we've called UseInterface yet. */
  public:
 
-	ModuleOperHash(InspIRCd* Me)
-		: Module(Me)
+	ModuleOperHash()
+		: cmd(this, hashers, names)
 	{
 		diduseiface = false;
 
@@ -109,8 +104,7 @@ class ModuleOperHash : public Module
 			diduseiface = true;
 		}
 
-		mycommand = new CommandMkpasswd(ServerInstance, this, hashers, names);
-		ServerInstance->AddCommand(mycommand);
+		ServerInstance->AddCommand(&cmd);
 		Implementation eventlist[] = { I_OnPassCompare, I_OnLoadModule };
 		ServerInstance->Modules->Attach(eventlist, this, 2);
 	}
@@ -137,7 +131,7 @@ class ModuleOperHash : public Module
 		}
 	}
 
-	virtual int OnPassCompare(Extensible* ex, const std::string &data, const std::string &input, const std::string &hashtype)
+	virtual ModResult OnPassCompare(Extensible* ex, const std::string &data, const std::string &input, const std::string &hashtype)
 	{
 		/* First, lets see what hash theyre using on this oper */
 		hashymodules::iterator x = hashers.find(hashtype.c_str());
@@ -149,18 +143,19 @@ class ModuleOperHash : public 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()))
-				return 1;
+				return MOD_RES_ALLOW;
 			/* No match, and must be hashed, forbid */
-			else return -1;
+			else
+				return MOD_RES_DENY;
 		}
 
 		/* Not a hash, fall through to strcmp in core */
-		return 0;
+		return MOD_RES_PASSTHRU;
 	}
 
 	virtual Version GetVersion()
 	{
-		return Version("$Id$",VF_VENDOR,API_VERSION);
+		return Version("Allows for hashed oper passwords",VF_VENDOR,API_VERSION);
 	}
 };