diff options
| author | Peter Powell <petpow@saberuk.com> | 2018-12-17 19:08:41 +0000 |
|---|---|---|
| committer | Peter Powell <petpow@saberuk.com> | 2018-12-17 19:26:59 +0000 |
| commit | bf0bf05ac07a4bd0afeba5a276ef86308f0f9e54 (patch) | |
| tree | 1cee2d712715a5ff7cc56fed523509404af4a6fd | |
| parent | 28eeee38bd912445f376d0261580173889c2137a (diff) | |
Fix not propagating rehashes properly across the network.
The REHASH command is not routed like a normal command. When a
rehash that needs to be forwarded is received the local server
forwards it indirectly by calling the local REHASH handler which
then calls the OnPreRehash hook. This results in the OnPreRehash
hook being called whilst loopCall is true.
The bug which this check was added to fix seems to no longer be
present so no alternate fix for that seems necessary.
Closes #1537.
| -rw-r--r-- | src/modules/m_spanningtree/main.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/modules/m_spanningtree/main.cpp b/src/modules/m_spanningtree/main.cpp index 0ff180a83..2a4e927a4 100644 --- a/src/modules/m_spanningtree/main.cpp +++ b/src/modules/m_spanningtree/main.cpp @@ -582,9 +582,6 @@ void ModuleSpanningTree::OnUserKick(User* source, Membership* memb, const std::s void ModuleSpanningTree::OnPreRehash(User* user, const std::string ¶meter) { - if (loopCall) - return; // Don't generate a REHASH here if we're in the middle of processing a message that generated this one - ServerInstance->Logs->Log(MODNAME, LOG_DEBUG, "OnPreRehash called with param %s", parameter.c_str()); // Send out to other servers |