diff options
| author |  Sadie Powell | 2023-07-02 11:49:47 +0100 |
|---|---|---|
| committer | Sadie Powell | 2023-07-02 11:52:25 +0100 |
| commit | 0dfaffa488753cfb92dd5a6909bbe8e14277343a (patch) | |
| tree | bb0ed3ce7e62e41d2c9f0a6190134604df62c880 /src/modules/m_cloak_static.cpp | |
| parent | Fix parsing <sqlauth:allowpattern> in v3 compat code. (diff) | |
Ensure that <cloak:cloak> isn't too long in cloak_static.
Diffstat (limited to 'src/modules/m_cloak_static.cpp')
| -rw-r--r-- | src/modules/m_cloak_static.cpp | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/src/modules/m_cloak_static.cpp b/src/modules/m_cloak_static.cpp index 7c56654f6..4109b4291 100644 --- a/src/modules/m_cloak_static.cpp +++ b/src/modules/m_cloak_static.cpp @@ -27,9 +27,9 @@ private: std::string cloak; public: - StaticMethod(const Cloak::Engine* engine, const std::shared_ptr<ConfigTag>& tag) ATTR_NOT_NULL(2) + StaticMethod(const Cloak::Engine* engine, const std::shared_ptr<ConfigTag>& tag, const std::string& c) ATTR_NOT_NULL(2) : Cloak::Method(engine, tag) - , cloak(tag->getString("cloak")) + , cloak(c) { } @@ -63,7 +63,14 @@ public: Cloak::MethodPtr Create(const std::shared_ptr<ConfigTag>& tag, bool primary) override { - return std::make_shared<StaticMethod>(this, tag); + const std::string cloak = tag->getString("cloak"); + if (cloak.empty() || cloak.length() > ServerInstance->Config->Limits.MaxHost) + { + throw ModuleException(creator, INSP_FORMAT("Your static cloak must be between 1 and {} characters long, at {}", + ServerInstance->Config->Limits.MaxHost, tag->source.str())); + } + + return std::make_shared<StaticMethod>(this, tag, cloak); } }; |