diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/coremods/core_channel/cmode_l.cpp | 5 | ||||
-rw-r--r-- | src/coremods/core_channel/core_channel.cpp | 5 | ||||
-rw-r--r-- | src/coremods/core_channel/core_channel.h | 1 |
3 files changed, 9 insertions, 2 deletions
diff --git a/src/coremods/core_channel/cmode_l.cpp b/src/coremods/core_channel/cmode_l.cpp index eb16fd182..e71eb500e 100644 --- a/src/coremods/core_channel/cmode_l.cpp +++ b/src/coremods/core_channel/cmode_l.cpp @@ -24,6 +24,7 @@ ModeChannelLimit::ModeChannelLimit(Module* Creator) : ParamMode<ModeChannelLimit, LocalIntExt>(Creator, "limit", 'l') + , minlimit(0) { } @@ -35,8 +36,8 @@ bool ModeChannelLimit::ResolveModeConflict(std::string &their_param, const std:: ModeAction ModeChannelLimit::OnSet(User* user, Channel* chan, std::string& parameter) { - int limit = ConvToInt(parameter); - if (limit < 0) + size_t limit = ConvToNum<size_t>(parameter); + if (limit < minlimit) return MODEACTION_DENY; ext.set(chan, limit); diff --git a/src/coremods/core_channel/core_channel.cpp b/src/coremods/core_channel/core_channel.cpp index af71e2ced..161f618d1 100644 --- a/src/coremods/core_channel/core_channel.cpp +++ b/src/coremods/core_channel/core_channel.cpp @@ -104,6 +104,11 @@ class CoreModChannel : public Module, public CheckExemption::EventListener exempts[restriction] = prefix; } exemptions.swap(exempts); + + // In 2.0 we allowed limits of 0 to be set. This is non-standard behaviour + // and will be removed in the next major release. + limitmode.minlimit = optionstag->getBool("allowzerolimit", true) ? 0 : 1; + banmode.DoRehash(); } diff --git a/src/coremods/core_channel/core_channel.h b/src/coremods/core_channel/core_channel.h index fa600cd17..c500add07 100644 --- a/src/coremods/core_channel/core_channel.h +++ b/src/coremods/core_channel/core_channel.h @@ -168,6 +168,7 @@ class ModeChannelKey : public ParamMode<ModeChannelKey, LocalStringExt> class ModeChannelLimit : public ParamMode<ModeChannelLimit, LocalIntExt> { public: + size_t minlimit; ModeChannelLimit(Module* Creator); bool ResolveModeConflict(std::string& their_param, const std::string& our_param, Channel* channel) CXX11_OVERRIDE; void SerializeParam(Channel* chan, intptr_t n, std::string& out); |