A quick glance at ircu indicates that when an empty gecos is given in USER, it is...

sense. The RFC is unclear here.

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

diff --git a/src/cmd_user.cpp b/src/cmd_user.cpp
index fc5ed7bb1afd0b93a45ce1a7e988a8ee46ab9fed..1a0e392c6ad2eb24caf5bb4bb0ab60e9bb82923a 100644 (file)
--- a/src/cmd_user.cpp
+++ b/src/cmd_user.cpp
@@ -27,7 +27,7 @@ CmdResult cmd_user::Handle (const char** parameters, int pcnt, userrec *user)
        /* A user may only send the USER command once */
        if (!(user->registered & REG_USER))
        {
-               if (!*parameters[3] || !ServerInstance->IsIdent(parameters[0]))
+               if (!ServerInstance->IsIdent(parameters[0]))
                {
                        // This kinda Sucks, According to the RFC thou, its either this,
                        // or "You have already registered" :p -- Craig
@@ -42,7 +42,7 @@ CmdResult cmd_user::Handle (const char** parameters, int pcnt, userrec *user)
                         * IDENTMAX here.
                         */
                        strlcpy(user->ident, parameters[0], IDENTMAX);
-                       strlcpy(user->fullname,parameters[3],MAXGECOS);
+                       strlcpy(user->fullname, *parameters[3] ? parameters[3] : "No info", MAXGECOS);
                        user->registered = (user->registered | REG_USER);
                }
        }