diff options
| author |  Sadie Powell | 2022-12-10 14:49:37 +0000 |
|---|---|---|
| committer | Sadie Powell | 2022-12-10 14:50:11 +0000 |
| commit | 7a6addf671190d32b441f9eef3042e561ea7e042 (patch) | |
| tree | 2e3f0dceabd7d119f4fe19b44f3e0788e41b1a3f /src | |
| parent | Move <oper:host> check into OnPreOperLogin. (diff) | |
Slightly improve the logging in cmd_oper.
Diffstat (limited to 'src')
| -rw-r--r-- | src/coremods/core_oper/cmd_oper.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/coremods/core_oper/cmd_oper.cpp b/src/coremods/core_oper/cmd_oper.cpp index fa9399127..10215e2d4 100644 --- a/src/coremods/core_oper/cmd_oper.cpp +++ b/src/coremods/core_oper/cmd_oper.cpp @@ -47,11 +47,8 @@ CmdResult CommandOper::HandleLocal(LocalUser* user, const Params& parameters) match_user = true; match_pass = ServerInstance->PassCompare(user, tag->getString("password"), parameters[1], tag->getString("hash")); - if (match_pass) - { - user->OperLogin(ifo); + if (match_pass && user->OperLogin(ifo)) return CmdResult::SUCCESS; - } } std::string fields; @@ -59,6 +56,8 @@ CmdResult CommandOper::HandleLocal(LocalUser* user, const Params& parameters) fields.append("username "); if (!match_pass) fields.append("password "); + if (fields.empty()) + fields.append("module "); fields.erase(fields.length() - 1, 1); // Tell them they failed (generically) and lag them up to help prevent brute-force attacks |