Assume worst case for unknown user and host in splitMessage

The default USERLEN of 9 doesn't have a great source, the RFC only says
that nicks are length 9, so my assumption is that usernames are not
longer.
weechat-hashes
C. McEnroe 2020-03-23 13:13:43 -04:00
parent 3bb3a11454
commit cf1545870a
4 changed files with 14 additions and 8 deletions

2
chat.c
View File

@ -64,7 +64,7 @@ enum Color idColors[IDCap] = {
};
uint idNext = Network + 1;
struct Network network;
struct Network network = { .userLen = 9, .hostLen = 63 };
struct Self self = { .color = Default };
static const char *save;

2
chat.h
View File

@ -84,6 +84,8 @@ static inline enum Color hash(const char *str) {
extern struct Network {
char *name;
uint userLen;
uint hostLen;
char *chanTypes;
char *prefixes;
char *prefixModes;

View File

@ -55,14 +55,12 @@ static void echoMessage(char *cmd, uint id, char *params) {
static void splitMessage(char *cmd, uint id, char *params) {
if (!params) return;
// FIXME: Get USERLEN, HOSTLEN from ISUPPORT and assume worst case if
// self.user and self.host are unset?
const char *nick = self.nick;
const char *user = (self.user ? self.user : "*");
const char *host = (self.host ? self.host : "*");
int overhead = snprintf(
NULL, 0, ":%s!%s@%s %s %s :\r\n",
nick, user, host, cmd, idNames[id]
NULL, 0, ":%s!%*s@%*s %s %s :\r\n",
self.nick,
(self.user ? 0 : network.userLen), (self.user ? self.user : "*"),
(self.host ? 0 : network.hostLen), (self.host ? self.host : "*"),
cmd, idNames[id]
);
assert(overhead > 0 && overhead < 512);
int chunk = 512 - overhead;

View File

@ -238,6 +238,12 @@ static void handleReplyISupport(struct Message *msg) {
Network, Cold, tagTime(msg),
"You arrive in %s", msg->params[i]
);
} else if (!strcmp(key, "USERLEN")) {
if (!msg->params[i]) continue;
network.userLen = strtoul(msg->params[i], NULL, 10);
} else if (!strcmp(key, "HOSTLEN")) {
if (!msg->params[i]) continue;
network.hostLen = strtoul(msg->params[i], NULL, 10);
} else if (!strcmp(key, "CHANTYPES")) {
if (!msg->params[i]) continue;
set(&network.chanTypes, msg->params[i]);