X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_timedbans.cpp;h=693e72019fa3a1aa102627a572d438fe4ac25b69;hb=8e89fe75f9467969bce1dc6930befc6ef273edf6;hp=1ff193c4dce2ec603a4eb6f85f2a6c2c115f0d6f;hpb=68730d4c9701b34c962302e6410908865fb2ba28;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_timedbans.cpp b/src/modules/m_timedbans.cpp
index 1ff193c4d..693e72019 100644
--- a/src/modules/m_timedbans.cpp
+++ b/src/modules/m_timedbans.cpp
@@ -33,7 +33,7 @@ timedbans TimedBanList;
 class CommandTban : public Command
 {
  public:
- CommandTban (InspIRCd* Instance) : Command(Instance,"TBAN", 0, 3)
+	CommandTban (InspIRCd* Instance) : Command(Instance,"TBAN", 0, 3)
 	{
 		this->source = "m_timedbans.so";
 		syntax = "<channel> <duration> <banmask>";
@@ -91,7 +91,8 @@ class CommandTban : public Command
 					T.expire = expire;
 					TimedBanList.push_back(T);
 					channel->WriteAllExcept(user, true, '@', tmp, "NOTICE %s :%s added a timed ban on %s lasting for %ld seconds.", channel->name, user->nick, mask.c_str(), duration);
-					channel->WriteAllExcept(user, true, '%', tmp, "NOTICE %s :%s added a timed ban on %s lasting for %ld seconds.", channel->name, user->nick, mask.c_str(), duration);
+					if (ServerInstance->Config->AllowHalfop)
+						channel->WriteAllExcept(user, true, '%', tmp, "NOTICE %s :%s added a timed ban on %s lasting for %ld seconds.", channel->name, user->nick, mask.c_str(), duration);
 					return CMD_SUCCESS;
 				}
 				return CMD_FAILURE;
@@ -125,7 +126,6 @@ class ModuleTimedBans : public Module
 		TimedBanList.clear();
 	}
 
-
 	virtual int OnDelBan(User* source, Channel* chan, const std::string &banmask)
 	{
 		irc::string listitem = banmask.c_str();
@@ -145,43 +145,41 @@ class ModuleTimedBans : public Module
 
 	virtual void OnBackgroundTimer(time_t curtime)
 	{
-		bool again = true;
-		while (again)
+		timedbans::iterator safei;
+		for (timedbans::iterator i = TimedBanList.begin(); i < TimedBanList.end();)
 		{
-			again = false;
-			for (timedbans::iterator i = TimedBanList.begin(); i < TimedBanList.end(); i++)
+			/* Safe copy of iterator, so we can erase as we iterate */
+			safei = i;
+			++i;
+
+			if (curtime > safei->expire)
 			{
-				if (curtime > i->expire)
+				Channel* cr = ServerInstance->FindChan(safei->channel);
+				if (cr)
 				{
-					Channel* cr = ServerInstance->FindChan(i->channel);
-					again = true;
-					if (cr)
-					{
-						const char *setban[3];
-						setban[0] = i->channel.c_str();
-						setban[1] = "-b";
-						setban[2] = i->mask.c_str();
-
-						/* Send mode remotely*/
-						std::deque<std::string> n;
-						n.push_back(setban[0]);
-						n.push_back("-b");
-						n.push_back(setban[2]);
-						ServerInstance->SendMode(setban,3, ServerInstance->FakeClient);
-						Event rmode((char *)&n, NULL, "send_mode");
-						rmode.Send(ServerInstance);
-						CUList empty;
-						cr->WriteAllExcept(ServerInstance->FakeClient, true, '@', empty, "NOTICE %s :*** Timed ban on %s expired.", cr->name, i->mask.c_str());
-						if (ServerInstance->Config->AllowHalfop)
-							cr->WriteAllExcept(ServerInstance->FakeClient, true, '%', empty, "NOTICE %s :*** Timed ban on %s expired.", cr->name, i->mask.c_str());
-					}
-					else
-					{
-						/* Where the hell did our channel go?! */
-						TimedBanList.erase(i);
-					}
-					// we used to delete the item here, but we dont need to as the servermode above does it for us,
-					break;
+					const char *setban[3];
+					std::string mask = safei->mask;
+
+					setban[0] = safei->channel.c_str();
+					setban[1] = "-b";
+					setban[2] = mask.c_str();
+
+					CUList empty;
+					cr->WriteAllExcept(ServerInstance->FakeClient, true, '@', empty, "NOTICE %s :*** Timed ban on %s expired.", cr->name, safei->mask.c_str());
+					if (ServerInstance->Config->AllowHalfop)
+						cr->WriteAllExcept(ServerInstance->FakeClient, true, '%', empty, "NOTICE %s :*** Timed ban on %s expired.", cr->name, safei->mask.c_str());
+
+					/* Removes the ban item for us, no ::erase() needed */
+					ServerInstance->PI->SendModeStr(safei->channel, std::string("-b ") + setban[2]);
+					ServerInstance->SendMode(setban, 3, ServerInstance->FakeClient);
+
+					if (ServerInstance->Modes->GetLastParse().empty())
+						TimedBanList.erase(safei);
+				}
+				else
+				{
+					/* Where the hell did our channel go?! */
+					TimedBanList.erase(safei);
 				}
 			}
 		}
@@ -189,7 +187,7 @@ class ModuleTimedBans : public Module
 	
 	virtual Version GetVersion()
 	{
-		return Version(1, 1, 0, 0, VF_COMMON | VF_VENDOR, API_VERSION);
+		return Version(1, 2, 0, 0, VF_COMMON | VF_VENDOR, API_VERSION);
 	}
 };