From d0a6b9c6eab666136af529953ebd585986a19848 Mon Sep 17 00:00:00 2001 From: Attila Molnar Date: Mon, 13 Jan 2014 14:01:16 +0100 Subject: m_mlock Only deny the mlocked mode from being changed instead of denying the entire mode change if it contains a single mlocked mode Fixes issue #615 reported by @BlacklightShining --- src/modules/m_mlock.cpp | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/modules/m_mlock.cpp b/src/modules/m_mlock.cpp index 719701d02..d1df81354 100644 --- a/src/modules/m_mlock.cpp +++ b/src/modules/m_mlock.cpp @@ -31,7 +31,7 @@ public: void init() { - ServerInstance->Modules->Attach(I_OnPreMode, this); + ServerInstance->Modules->Attach(I_OnRawMode, this); ServerInstance->Modules->AddService(this->mlock); } @@ -40,12 +40,7 @@ public: return Version("Implements the ability to have server-side MLOCK enforcement.", VF_VENDOR); } - void Prioritize() - { - ServerInstance->Modules->SetPriority(this, I_OnPreMode, PRIORITY_FIRST); - } - - ModResult OnPreMode(User* source, User* dest, Channel* channel, const std::vector& parameters) + ModResult OnRawMode(User* source, Channel* channel, const char mode, const std::string& parameter, bool adding, int pcnt) { if (!channel) return MOD_RES_PASSTHRU; @@ -57,11 +52,11 @@ public: if (!mlock_str) return MOD_RES_PASSTHRU; - std::string::size_type p = parameters[1].find_first_of(*mlock_str); + std::string::size_type p = mlock_str->find(mode); if (p != std::string::npos) { source->WriteNumeric(742, "%s %c %s :MODE cannot be set due to channel having an active MLOCK restriction policy", - channel->name.c_str(), parameters[1][p], mlock_str->c_str()); + channel->name.c_str(), mode, mlock_str->c_str()); return MOD_RES_DENY; } -- cgit v1.2.3