Fix various inconsistencies/crashes related to having multiple opers configured for the same user through combinations of os_oper/config/m_sql_oper

1 files changed, 7 insertions, 0 deletions

diff --git a/src/config.cpp b/src/config.cpp
index 18a6c5d39..b6972bc42 100644
--- a/src/config.cpp
+++ b/src/config.cpp
@@ -502,6 +502,13 @@ Conf::Conf() : Block("")
 		if (!na)
 			continue;
 
+		if (!na->nc || na->nc->o)
+		{
+			// If the account is already an oper it might mean two oper blocks for the same nick, or
+			// something else has configured them as an oper (like a module)
+			continue;
+		}
+
 		na->nc->o = o;
 		Log() << "Tied oper " << na->nc->display << " to type " << o->ot->GetName();
 	}