Disallow empty string for nick

This commit is contained in:
Simon Ser 2021-11-18 09:44:33 +01:00
parent dcc1eff130
commit ac4696a507

View File

@ -653,7 +653,7 @@ func (dc *downstreamConn) handleMessageUnregistered(ctx context.Context, msg *ir
if err := parseMessageParams(msg, &nick); err != nil { if err := parseMessageParams(msg, &nick); err != nil {
return err return err
} }
if strings.ContainsAny(nick, illegalNickChars) { if nick == "" || strings.ContainsAny(nick, illegalNickChars) {
return ircError{&irc.Message{ return ircError{&irc.Message{
Command: irc.ERR_ERRONEUSNICKNAME, Command: irc.ERR_ERRONEUSNICKNAME,
Params: []string{dc.nick, nick, "contains illegal characters"}, Params: []string{dc.nick, nick, "contains illegal characters"},
@ -1492,7 +1492,7 @@ func (dc *downstreamConn) handleMessageRegistered(ctx context.Context, msg *irc.
} }
} }
if strings.ContainsAny(nick, illegalNickChars) { if nick == "" || strings.ContainsAny(nick, illegalNickChars) {
return ircError{&irc.Message{ return ircError{&irc.Message{
Command: irc.ERR_ERRONEUSNICKNAME, Command: irc.ERR_ERRONEUSNICKNAME,
Params: []string{dc.nick, rawNick, "contains illegal characters"}, Params: []string{dc.nick, rawNick, "contains illegal characters"},