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, 1 deletions

diff --git a/src/modules/m_spanningtree.cpp b/src/modules/m_spanningtree.cpp
index b33e28af8..f927ac044 100644
--- a/src/modules/m_spanningtree.cpp
+++ b/src/modules/m_spanningtree.cpp
@@ -2513,7 +2513,11 @@ class TreeSocket : public InspSocket
 						{
 							strparams[q] = (char*)params[q].c_str();
 						}
-						Srv->CallCommandHandler(command.c_str(), strparams, params.size(), who);
+						if (!Srv->CallCommandHandler(command.c_str(), strparams, params.size(), who))
+						{
+							this->WriteLine("ERROR :Unrecognised command -- possibly loaded mismatched modules");
+							return false;
+						}
 					}
 					else
 					{