bool badchan;
public:
- ModuleChannelNames() : rememberer(ServerInstance->IsChannel)
+ ModuleChannelNames() : rememberer(ServerInstance->IsChannel), badchan(false)
+ {
+ }
+
+ void init()
{
ServerInstance->IsChannel = &myhandler;
- badchan = false;
Implementation eventlist[] = { I_OnRehash, I_OnUserKick };
- ServerInstance->Modules->Attach(eventlist, this, 2);
+ ServerInstance->Modules->Attach(eventlist, this, sizeof(eventlist)/sizeof(Implementation));
OnRehash(NULL);
}
ServerInstance->SendGlobalMode(modes, ServerInstance->FakeClient);
}
const UserMembList* users = c->GetUsers();
- for(UserMembCIter j = users->begin(); j != users->end(); ++j)
+ for(UserMembCIter j = users->begin(); j != users->end(); )
+ {
if (IS_LOCAL(j->first))
- c->KickUser(ServerInstance->FakeClient, j->first, "Channel name no longer valid");
+ {
+ // KickUser invalidates the iterator
+ UserMembCIter it = j++;
+ c->KickUser(ServerInstance->FakeClient, it->first, "Channel name no longer valid");
+ }
+ else
+ ++j;
+ }
}
badchan = false;
}
virtual void OnRehash(User* user)
{
- ConfigReader Conf;
- std::string denyToken = Conf.ReadValue("channames", "denyrange", 0);
- std::string allowToken = Conf.ReadValue("channames", "allowrange", 0);
+ ConfigTag* tag = ServerInstance->Config->ConfValue("channames");
+ std::string denyToken = tag->getString("denyrange");
+ std::string allowToken = tag->getString("allowrange");
allowedmap.set();
irc::portparser denyrange(denyToken, false);