From 60643f1b7bbc2a21f44ab6663dee03d962ff6d6b Mon Sep 17 00:00:00 2001 From: Haru Date: Tue, 13 Nov 2018 17:40:48 +0100 Subject: Fix an issue that caused some guild channels not to be left when leaving allychat - The issue was caused by an array compaction while iterating through the array members. - A visible effect was that under certain conditions (depending on the alliance size), a character that left a guild could still see allychat messages from some of the allied guilds. Signed-off-by: Haru --- src/map/atcommand.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) (limited to 'src/map/atcommand.c') diff --git a/src/map/atcommand.c b/src/map/atcommand.c index adf617ccd..f46ce0362 100644 --- a/src/map/atcommand.c +++ b/src/map/atcommand.c @@ -9125,14 +9125,12 @@ ACMD(channel) return false; } if (VECTOR_INDEX(sd->channels, k)->type == HCS_TYPE_ALLY) { - do { - for (k = 0; k < VECTOR_LENGTH(sd->channels); k++) { - if (VECTOR_INDEX(sd->channels, k)->type == HCS_TYPE_ALLY) { - channel->leave(VECTOR_INDEX(sd->channels, k), sd); - break; - } + for (k = VECTOR_LENGTH(sd->channels) - 1; k >= 0; k--) { + // Loop downward to avoid issues when channel->leave() compacts the array + if (VECTOR_INDEX(sd->channels, k)->type == HCS_TYPE_ALLY) { + channel->leave(VECTOR_INDEX(sd->channels, k), sd); } - } while (k != VECTOR_LENGTH(sd->channels)); // FIXME + } } else { channel->leave(VECTOR_INDEX(sd->channels, k), sd); } -- cgit v1.2.3-60-g2f50