Fix a regression from fcb51634669e6ed588e42f2072c6f910e267126f ("fix uid collision...

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

diff --git a/src/modules/m_servprotect.cpp b/src/modules/m_servprotect.cpp
index e9012ac31998a642a1f4f6945f66c6b2eb8deb75..7c4aef2899832ff26aba0ca6cd4eab2c0c21a814 100644 (file)
--- a/src/modules/m_servprotect.cpp
+++ b/src/modules/m_servprotect.cpp
@@ -2,8 +2,8 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
- * See: http://www.inspircd.org/wiki/index.php/Credits
+ *  InspIRCd: (C) 2002-2009 InspIRCd Development Team
+ * See: http://wiki.inspircd.org/Credits
  *
  * This program is free but copyrighted software; see
  *            the file COPYING for details.
@@ -12,12 +12,6 @@
  */
 
 #include "inspircd.h"
-#include <stdio.h>
-#include <string>
-#include "users.h"
-#include "channels.h"
-#include "modules.h"
-#include "configreader.h"
 
 /* $ModDesc: Provides support for Austhex style +k / UnrealIRCD +S services mode */
 
@@ -30,6 +24,14 @@ class ServProtectMode : public ModeHandler
 
        ModeAction OnModeChange(User* source, User* dest, Channel* channel, std::string &parameter, 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 or uline, 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;
        }
 
@@ -61,7 +63,7 @@ class ModuleServProtectMode : public Module
 
        virtual Version GetVersion()
        {
-               return Version(1,2,0,0,VF_COMMON,API_VERSION);
+               return Version("$Id$", VF_COMMON | VF_VENDOR, API_VERSION);
        }
 
        virtual void OnWhois(User* src, User* dst)
@@ -108,7 +110,7 @@ class ModuleServProtectMode : public Module
                if (dst->IsModeSet('k'))
                {
                        src->WriteNumeric(485, "%s :You are not permitted to kill %s services!", src->nick.c_str(), ServerInstance->Config->Network);
-                       ServerInstance->SNO->WriteToSnoMask('A', std::string(src->nick)+" tried to kill service "+dst->nick+" ("+reason+")");
+                       ServerInstance->SNO->WriteGlobalSno('a', std::string(src->nick)+" tried to kill service "+dst->nick+" ("+reason+")");
                        return 1;
                }
                return 0;