diff options
| author | Wilmer van der Gaast <wilmer@gaast.net> | 2010-09-06 00:30:40 +0100 | 
|---|---|---|
| committer | Wilmer van der Gaast <wilmer@gaast.net> | 2010-09-06 00:30:40 +0100 | 
| commit | d6657ce1d4fa016bfa0e013f29351229e2cc23d1 (patch) | |
| tree | 37de800c370c689aa9029fbac16c02d8279639e0 | |
| parent | ed320e88b1677266f70b8b163c638bebe2a81e0a (diff) | |
Fix NULL pointer dereference when removing nicks from groupchats.
| -rw-r--r-- | irc_im.c | 2 | ||||
| -rw-r--r-- | protocols/bee_chat.c | 2 | 
2 files changed, 2 insertions, 2 deletions
@@ -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 );  }  | 
