WHO #channel: don't limit list size

It makes no sense to limit the list size when doing WHO for a channel
and not to return all the users in that channel, so I removed the check.
But if there are more than MAX_RPL_WHO(25) replies, the client requesting
the list will be "penalized" one second more (then 2 in total).

This fixes bug #125.
This commit is contained in:
Alexander Barton 2012-09-10 12:43:44 +02:00
parent d2d867ea36
commit 2205227c3b
1 changed files with 8 additions and 5 deletions

View File

@ -848,6 +848,8 @@ IRC_WHO_Channel(CLIENT *Client, CHANNEL *Chan, bool OnlyOps)
assert( Client != NULL ); assert( Client != NULL );
assert( Chan != NULL ); assert( Chan != NULL );
IRC_SetPenalty(Client, 1);
is_member = Channel_IsMemberOf(Chan, Client); is_member = Channel_IsMemberOf(Chan, Client);
/* Secret channel? */ /* Secret channel? */
@ -866,9 +868,6 @@ IRC_WHO_Channel(CLIENT *Client, CHANNEL *Chan, bool OnlyOps)
is_visible = strchr(client_modes, 'i') == NULL; is_visible = strchr(client_modes, 'i') == NULL;
if (is_member || is_visible) { if (is_member || is_visible) {
if (IRC_CheckListTooBig(Client, count, MAX_RPL_WHO, "WHO"))
break;
strcpy(flags, who_flags_status(client_modes)); strcpy(flags, who_flags_status(client_modes));
if (is_ircop) if (is_ircop)
strlcat(flags, "*", sizeof(flags)); strlcat(flags, "*", sizeof(flags));
@ -883,6 +882,11 @@ IRC_WHO_Channel(CLIENT *Client, CHANNEL *Chan, bool OnlyOps)
count++; count++;
} }
} }
/* If there are a lot of clients, augment penalty a bit */
if (count > MAX_RPL_WHO)
IRC_SetPenalty(Client, 1);
return IRC_WriteStrClient(Client, RPL_ENDOFWHO_MSG, Client_ID(Client), return IRC_WriteStrClient(Client, RPL_ENDOFWHO_MSG, Client_ID(Client),
Channel_Name(Chan)); Channel_Name(Chan));
} }
@ -911,6 +915,7 @@ IRC_WHO_Mask(CLIENT *Client, char *Mask, bool OnlyOps)
if (Mask) if (Mask)
ngt_LowerStr(Mask); ngt_LowerStr(Mask);
IRC_SetPenalty(Client, 3);
for (c = Client_First(); c != NULL; c = Client_Next(c)) { for (c = Client_First(); c != NULL; c = Client_Next(c)) {
if (Client_Type(c) != CLIENT_USER) if (Client_Type(c) != CLIENT_USER)
continue; continue;
@ -1014,13 +1019,11 @@ IRC_WHO(CLIENT *Client, REQUEST *Req)
chan = Channel_Search(Req->argv[0]); chan = Channel_Search(Req->argv[0]);
if (chan) { if (chan) {
/* Members of a channel have been requested */ /* Members of a channel have been requested */
IRC_SetPenalty(Client, 1);
return IRC_WHO_Channel(Client, chan, only_ops); return IRC_WHO_Channel(Client, chan, only_ops);
} }
if (strcmp(Req->argv[0], "0") != 0) { if (strcmp(Req->argv[0], "0") != 0) {
/* A mask has been given. But please note this RFC /* A mask has been given. But please note this RFC
* stupidity: "0" is same as no arguments ... */ * stupidity: "0" is same as no arguments ... */
IRC_SetPenalty(Client, 3);
return IRC_WHO_Mask(Client, Req->argv[0], only_ops); return IRC_WHO_Mask(Client, Req->argv[0], only_ops);
} }
} }