diff options
| author |  Sadie Powell | 2022-04-17 11:02:42 +0100 |
|---|---|---|
| committer | Sadie Powell | 2022-04-17 11:14:39 +0100 |
| commit | 80e80132826569444e690a177f5bfc5f603fada1 (patch) | |
| tree | 497e86b783dc6beb1828a07c1ea99b8383b8ea10 /src/modules/m_exemptchanops.cpp | |
| parent | Refactor CleanMask to cover more edge cases. (diff) | |
Remove ValidateParam and rename CanonicalizeParam.
There's basically no safe way to handle a malformed list mode
sent by a remote server without causing a desync. Its probably
for the best if we just only apply validation to locally added
list modes entries.
Diffstat (limited to 'src/modules/m_exemptchanops.cpp')
| -rw-r--r-- | src/modules/m_exemptchanops.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/modules/m_exemptchanops.cpp b/src/modules/m_exemptchanops.cpp index 89c1688fc..c384e723f 100644 --- a/src/modules/m_exemptchanops.cpp +++ b/src/modules/m_exemptchanops.cpp @@ -81,8 +81,9 @@ public: return MOD_RES_DENY; } - bool ValidateParam(User* user, Channel* chan, const std::string& parameter) override + bool ValidateParam(LocalUser* user, Channel* chan, std::string& parameter) override { + // We only enforce the format restriction against local users to avoid causing a desync. std::string restriction; std::string prefix; if (!ParseEntry(parameter, restriction, prefix)) |