Move ident lookups to the OnSetUserIP hook.

[user/henk/code/inspircd.git] / src / modules / m_chanlog.cpp

diff --git a/src/modules/m_chanlog.cpp b/src/modules/m_chanlog.cpp
index 85e7ca2eb7b9d9eb4900f47db02fb7b5d088d690..afe213f6141f441e96d717da751ebe640203ed67 100644 (file)
--- a/src/modules/m_chanlog.cpp
+++ b/src/modules/m_chanlog.cpp
@@ -33,8 +33,7 @@ class ModuleChanLog : public Module
        {
                std::string snomasks;
                std::string channel;
-
-               logstreams.clear();
+               ChanLogTargets newlogs;
 
                ConfigTagList tags = ServerInstance->Config->ConfTags("chanlog");
                for (ConfigIter i = tags.first; i != tags.second; ++i)
@@ -44,16 +43,16 @@ class ModuleChanLog : public Module
 
                        if (channel.empty() || snomasks.empty())
                        {
-                               ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Malformed chanlog tag, ignoring");
-                               continue;
+                               throw ModuleException("Malformed chanlog tag at " + i->second->getTagLocation());
                        }
 
                        for (std::string::const_iterator it = snomasks.begin(); it != snomasks.end(); it++)
                        {
-                               logstreams.insert(std::make_pair(*it, channel));
+                               newlogs.insert(std::make_pair(*it, channel));
                                ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "Logging %c to %s", *it, channel.c_str());
                        }
                }
+               logstreams.swap(newlogs);
 
        }