From d6bec26bc7c329c0d3d6d84203ecd12a26dac91b Mon Sep 17 00:00:00 2001 From: brain Date: Thu, 2 Oct 2008 23:14:12 +0000 Subject: Comment on potentially confusing code git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@10616 e03df62e-2008-0410-955e-edbf42e46eb7 --- src/modules/m_servprotect.cpp | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'src/modules') diff --git a/src/modules/m_servprotect.cpp b/src/modules/m_servprotect.cpp index 687c45a94..f77bdbc6a 100644 --- a/src/modules/m_servprotect.cpp +++ b/src/modules/m_servprotect.cpp @@ -30,6 +30,14 @@ class ServProtectMode : public ModeHandler ModeAction OnModeChange(User* source, User* dest, Channel* channel, std::string ¶meter, bool adding, bool) { + /* Because this returns MODEACTION_DENY all the time, there is only ONE + * way to add this mode and that is at client introduction in the UID command, + * as this calls OnModeChange for each mode but disregards the return values. + * The mode cannot be manually added or removed, not even by a server or by a remote + * user, which prevents its (ab)use as a kiddie 'god mode' on such networks. + * I'm sure if someone really wants to do that they can make a copy of this module + * that does the job. It won't be me though! + */ return MODEACTION_DENY; } -- cgit v1.2.3