diff options
| author |  brain | 2007-04-02 22:13:20 +0000 |
|---|---|---|
| committer | brain | 2007-04-02 22:13:20 +0000 |
| commit | ffdd357e073819fd617af45c86c2499597380d50 (patch) | |
| tree | a6b72a51a0734c5c74910dada01fba2f931cec73 /src/cmd_user.cpp | |
| parent | This needs some general QA-ing. Add support to new parser (introduced in 1.1)... (diff) | |
A quick glance at ircu indicates that when an empty gecos is given in USER, it is replaced by 'No info', rather than our way of saying 'not enough parameters', so we'll do this as it seems to make
sense. The RFC is unclear here.
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@6732 e03df62e-2008-0410-955e-edbf42e46eb7
Diffstat (limited to 'src/cmd_user.cpp')
| -rw-r--r-- | src/cmd_user.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/cmd_user.cpp b/src/cmd_user.cpp index fc5ed7bb1..1a0e392c6 100644 --- 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); } } |