Decide that it wasn't quite appropriate :(

[user/henk/code/inspircd.git] / src / cmd_oper.cpp

diff --git a/src/cmd_oper.cpp b/src/cmd_oper.cpp
index 8e57329d4702899a37f0652c3ca431743f930357..7cbf4b79f62d10318dc3f18f975ada330072171f 100644 (file)
--- a/src/cmd_oper.cpp
+++ b/src/cmd_oper.cpp
@@ -110,7 +110,14 @@ void cmd_oper::Handle (char **parameters, int pcnt, userrec *user)
                                        Config->ConfValue("type","host",j,HostName,&Config->config_f);
                                        if (*HostName)
                                                ChangeDisplayedHost(user,HostName);
-                                       strlcpy(user->oper,TypeName,NICKMAX);
+                                       if (!isnick(TypeName))
+                                       {
+                                               WriteServ(user->fd,"491 %s :Invalid oper type (oper types must follow the same syntax as nicknames)",user->nick);
+                                               WriteOpers("*** CONFIGURATION ERROR! Oper type invalid for OperType '%s'",OperType);
+                                               log(DEFAULT,"OPER: Failed oper attempt by %s!%s@%s: credentials valid, but oper type erroneous.",user->nick,user->ident,user->host);
+                                               return;
+                                       }
+                                       strlcpy(user->oper,TypeName,NICKMAX-1);
                                        found = true;
                                        fail2 = false;
                                        break;