diff options
| author |  Attila Molnar | 2013-12-15 13:10:33 +0100 |
|---|---|---|
| committer | Attila Molnar | 2013-12-15 13:10:33 +0100 |
| commit | 24fc55bbfd367bf2ef68230173ffe1cb58f744fa (patch) | |
| tree | 087ca6e8040673b95ae1ab92e568a55ba453166e /src/modules/m_spanningtree/svsnick.cpp | |
| parent | Fix m_override refusing to override modes if the user holds halfop or higher ... (diff) | |
m_spanningtree Fix nick TS desync on SVSNICK
Don't accept invalid timestamps
Diffstat (limited to 'src/modules/m_spanningtree/svsnick.cpp')
| -rw-r--r-- | src/modules/m_spanningtree/svsnick.cpp | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/src/modules/m_spanningtree/svsnick.cpp b/src/modules/m_spanningtree/svsnick.cpp index 79dc27ea3..59973202d 100644 --- a/src/modules/m_spanningtree/svsnick.cpp +++ b/src/modules/m_spanningtree/svsnick.cpp @@ -34,17 +34,28 @@ CmdResult CommandSVSNick::Handle(const std::vector<std::string>& parameters, Use if (isdigit(nick[0])) nick = u->uuid; + // Don't update the TS if the nick is exactly the same + if (u->nick == nick) + return CMD_FAILURE; + + time_t NickTS = ConvToInt(parameters[2]); + if (NickTS <= 0) + return CMD_FAILURE; + + ModuleSpanningTree* st = (ModuleSpanningTree*)(Module*)creator; + st->KeepNickTS = true; + u->age = NickTS; + if (!u->ForceNickChange(nick.c_str())) { /* buh. UID them */ if (!u->ForceNickChange(u->uuid.c_str())) { ServerInstance->Users->QuitUser(u, "Nickname collision"); - return CMD_SUCCESS; } } - u->age = atoi(parameters[2].c_str()); + st->KeepNickTS = false; } return CMD_SUCCESS; |