Disable all connect classes, reenable them as we find them in the config. This stops users using a connect class removed from the conf (or will in a second)

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

1 files changed, 6 insertions, 1 deletions

diff --git a/src/configreader.cpp b/src/configreader.cpp
index 5031f6506..c15b5c0e7 100644
--- a/src/configreader.cpp
+++ b/src/configreader.cpp
@@ -421,17 +421,20 @@ bool InitConnect(ServerConfig* conf, const char*)
 	}
 
 goagain:
-	/* change this: only delete a class with refcount 0 */
 	for (ClassVector::iterator i = conf->Classes.begin(); i != conf->Classes.end(); i++)
 	{
 		ConnectClass *c = *i;
 
+		/* only delete a class with refcount 0 */
 		if (c->RefCount == 0)
 		{
 			conf->GetInstance()->Log(DEFAULT, "Removing connect class, refcount is 0!");
 			conf->Classes.erase(i);
 			goto goagain; // XXX fucking hell.. how better to  do this
 		}
+
+		/* also mark all existing classes disabled, if they still exist in the conf, they will be reenabled. */
+		c->SetDisabled(true);
 	}
 
 	return true;
@@ -467,6 +470,8 @@ bool DoConnect(ServerConfig* conf, const char*, char**, ValueList &values, int*)
 		ConnectClass* c = *item;
 		if ((*name && (c->GetName() == name)) || (*allow && (c->GetHost() == allow)) || (*deny && (c->GetHost() == deny)))
 		{
+			/* reenable class so users can be shoved into it :P */
+			c->SetDisabled(false);
 			conf->GetInstance()->Log(DEFAULT, "Not adding class, it already exists!");
 			return true;
 		}