aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWilmer van der Gaast <wilmer@gaast.net>2010-09-06 00:30:40 +0100
committerWilmer van der Gaast <wilmer@gaast.net>2010-09-06 00:30:40 +0100
commitd6657ce1d4fa016bfa0e013f29351229e2cc23d1 (patch)
tree37de800c370c689aa9029fbac16c02d8279639e0
parented320e88b1677266f70b8b163c638bebe2a81e0a (diff)
Fix NULL pointer dereference when removing nicks from groupchats.
-rw-r--r--irc_im.c2
-rw-r--r--protocols/bee_chat.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/irc_im.c b/irc_im.c
index 7fa70350..6c8b7f70 100644
--- a/irc_im.c
+++ b/irc_im.c
@@ -529,7 +529,7 @@ static gboolean bee_irc_chat_remove_user( bee_t *bee, struct groupchat *c, bee_u
irc_t *irc = bee->ui_data;
irc_channel_t *ic = c->ui_data;
- if( ic == NULL )
+ if( ic == NULL || bu == NULL )
return FALSE;
/* TODO: Possible bug here: If a module removes $user here instead of just
diff --git a/protocols/bee_chat.c b/protocols/bee_chat.c
index 0314cae5..349e0547 100644
--- a/protocols/bee_chat.c
+++ b/protocols/bee_chat.c
@@ -195,7 +195,7 @@ void imcb_chat_remove_buddy( struct groupchat *c, const char *handle, const char
bu = bee_user_by_handle( bee, ic, handle );
}
- if( bee->ui->chat_remove_user )
+ if( bee->ui->chat_remove_user && bu )
bee->ui->chat_remove_user( bee, c, bu );
}