Changed definition of CallHandler and CallCommandHandler for the ability to return false on bad command.

This allows us to terminate server links that send commands we dont know (e.g. admin loads module A after linking, we dont have module A, when it sends module A's command, abort)


git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@3414 e03df62e-2008-0410-955e-edbf42e46eb7

1 files changed, 5 insertions, 7 deletions

diff --git a/src/command_parse.cpp b/src/command_parse.cpp
index 18915b819..77fef66d7 100644
--- a/src/command_parse.cpp
+++ b/src/command_parse.cpp
@@ -288,7 +288,7 @@ bool CommandParser::IsValidCommand(std::string &commandname, int pcnt, userrec *
 
 // calls a handler function for a command
 
-void CommandParser::CallHandler(std::string &commandname,char **parameters, int pcnt, userrec *user)
+bool CommandParser::CallHandler(std::string &commandname,char **parameters, int pcnt, userrec *user)
 {
 	nspace::hash_map<std::string,command_t*>::iterator n = cmdlist.find(commandname);
 
@@ -300,23 +300,21 @@ void CommandParser::CallHandler(std::string &commandname,char **parameters, int
 			{
 				if (n->second->flags_needed)
 				{
-					if ((user->HasPermission(commandname)) || (is_uline(user->server)))
+					if ((user->HasPermission(commandname)) || (!IS_LOCAL(user)))
 					{
 						n->second->Handle(parameters,pcnt,user);
-					}
-					else
-					{
-						if (!IS_LOCAL(user))
-							WriteOpers("*** \2WARNING\2: Command '%s' not allowed for oper '%s', dropped.",commandname.c_str(),user->nick);
+						return true;
 					}
 				}
 				else
 				{
 					n->second->Handle(parameters,pcnt,user);
+					return true;
 				}
 			}
 		}
 	}
+	return false;
 }
 
 int CommandParser::ProcessParameters(char **command_p,char *parameters)