Added some missing parameter checking in m_swhois

[user/henk/code/inspircd.git] / src / modules / m_chgident.cpp

diff --git a/src/modules/m_chgident.cpp b/src/modules/m_chgident.cpp
index e3067fe5622d84eccc04ed2cf71e80728642b8c1..168adcc93c2f571c2ca129953c7de2f3e2541dc2 100644 (file)
--- a/src/modules/m_chgident.cpp
+++ b/src/modules/m_chgident.cpp
@@ -40,12 +40,19 @@ class cmd_chgident : public command_t
 
                if (!*parameters[1])
                {
-                       user->WriteServ("NOTICE %s :*** CHGIDENT: Needs non-zero length ident", user->nick);
+                       user->WriteServ("NOTICE %s :*** CHGIDENT: Ident must be specified", user->nick);
                        return CMD_FAILURE;
                }
-               if(!ServerInstance->IsIdent(parameters[1]))
+               
+               if (strlen(parameters[1]) > IDENTMAX)
                {
-                       user->WriteServ("NOTICE %s :*** Invalid characters in ident", user->nick);
+                       user->WriteServ("NOTICE %s :*** CHGIDENT: Ident is too long", user->nick);
+                       return CMD_FAILURE;
+               }
+               
+               if (!ServerInstance->IsIdent(parameters[1]))
+               {
+                       user->WriteServ("NOTICE %s :*** CHGIDENT: Invalid characters in ident", user->nick);
                        return CMD_FAILURE;
                }
 
@@ -81,5 +88,5 @@ public:
        
 };
 
-MODULE_INIT(ModuleChgIdent);
+MODULE_INIT(ModuleChgIdent)