From ffdd357e073819fd617af45c86c2499597380d50 Mon Sep 17 00:00:00 2001 From: brain Date: Mon, 2 Apr 2007 22:13:20 +0000 Subject: [PATCH 1/1] 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 --- src/cmd_user.cpp | 4 ++-- 1 file 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); } } -- 2.39.2