Fix SetModeParam to use std::string and handle edge cases.

[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 f77bdbc6acbda57287a5a628b6e19d96a175e4da..73462a20a686e842abd5882f16949d5738b94ef3 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,11 +24,11 @@ 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 
+               /* 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.
+                * 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!
                 */
@@ -69,7 +63,7 @@ class ModuleServProtectMode : public Module
 
        virtual Version GetVersion()
        {
-               return Version("$Id$",VF_COMMON,API_VERSION);
+               return Version("$Id$", VF_COMMON | VF_VENDOR, API_VERSION);
        }
 
        virtual void OnWhois(User* src, User* dst)
@@ -116,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->WriteToSnoMask('a', std::string(src->nick)+" tried to kill service "+dst->nick+" ("+reason+")");
                        return 1;
                }
                return 0;