Fix for bug #605 reported by MacGuyver, if a line is expired when we are adding another with the same hostmask, replace the existing line with the new one

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@10589 e03df62e-2008-0410-955e-edbf42e46eb7

1 files changed, 10 insertions, 1 deletions

diff --git a/src/xline.cpp b/src/xline.cpp
index 0c6965fe3..bc1cf9183 100644
--- a/src/xline.cpp
+++ b/src/xline.cpp
@@ -170,7 +170,16 @@ bool XLineManager::AddLine(XLine* line, User* user)
 	ServerInstance->BanCache->RemoveEntries(line->type, false); // XXX perhaps remove ELines here?
 
 	if (DelLine(line->Displayable(), line->type, user, true))
-		return false;
+	{
+		/* Line exists, check if its an expired line */
+		ContainerIter x = lookup_lines.find(line->type);
+		LookupIter i = x->second.find(line->Displayable());
+
+		if (i->second->duration && ServerInstance->Time() > i->second->expiry)
+			ExpireLine(x, i);
+		else
+			return false;
+	}
 
 	/*ELine* item = new ELine(ServerInstance, ServerInstance->Time(), duration, source, reason, ih.first.c_str(), ih.second.c_str());*/
 	XLineFactory* xlf = GetFactory(line->type);