aniani/irssi
1
0
Fork 0
mirror of https://github.com/irssi/irssi.git synced 2024-09-08 04:26:01 -04:00
Code

Merge pull request #832 from ailin-nemui/netsplit-crash

Browse Source 
try to make sure the server is still good enough to call ischannel when printing netsplit/join
This commit is contained in:
ailin-nemui 2018-02-12 10:54:42 +01:00 committed by GitHub
commit ccfb2dabcf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 25 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/fe-common/irc/fe-netjoin.c
View File

@ -245,17 +245,24 @@ static void print_netjoins(NETJOIN_SERVER_REC *server, const char *filter_channe
message before it. */
static void sig_print_starting(TEXT_DEST_REC *dest)
{
GSList *tmp, *next;
NETJOIN_SERVER_REC *rec;
if (printing_joins)
return;
for (tmp = joinservers; tmp != NULL; tmp = next) {
NETJOIN_SERVER_REC *server = tmp->data;
if (!IS_IRC_SERVER(dest->server))
return;
next = tmp->next;
if (server->netjoins != NULL)
print_netjoins(server, NULL);
rec = netjoin_find_server(IRC_SERVER(dest->server));
if (rec != NULL && rec->netjoins != NULL) {
/* if netjoins exists, the server rec should be
still valid. otherwise, calling server->ischannel
may not be safe. */
if (dest->target != NULL &&
!server_ischannel((SERVER_REC *) rec->server, dest->target))
return;
print_netjoins(rec, NULL);
}
}

17
src/fe-common/irc/fe-netsplit.c
View File

@ -247,16 +247,23 @@ static int check_server_splits(IRC_SERVER_REC *server)
message before it. */
static void sig_print_starting(TEXT_DEST_REC *dest)
{
GSList *tmp;
IRC_SERVER_REC *rec;
if (printing_splits)
return;
for (tmp = servers; tmp != NULL; tmp = tmp->next) {
IRC_SERVER_REC *rec = tmp->data;
if (!IS_IRC_SERVER(dest->server))
return;
if (IS_IRC_SERVER(rec) && rec->split_servers != NULL)
print_splits(rec, NULL);
rec = IRC_SERVER(dest->server);
if (rec->split_servers != NULL) {
/* if split_servers exists, the server rec should be
still valid. otherwise, calling server->ischannel
may not be safe. */
if (dest->target != NULL && !server_ischannel((SERVER_REC *) rec, dest->target))
return;
print_splits(rec, NULL);
}
}