9pfs/solanum-vs-hackint-and-charybdis
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix double-free when removing a ban.

Browse source 
del_id() should not free the ban anymore, its caller does that now.
This commit is contained in:
Jilles Tjoelker 2011-09-14 00:52:56 +02:00
parent f890420014
commit 93fbe9c349
1 changed files with 0 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/chmode.c
View file

@ -290,7 +290,6 @@ del_id(struct Channel *chptr, const char *banid, rb_dlink_list * list, long mode
if(irccmp(banid, banptr->banstr) == 0) if(irccmp(banid, banptr->banstr) == 0)
{ {
rb_dlinkDelete(&banptr->node, list); rb_dlinkDelete(&banptr->node, list);
free_ban(banptr);
/* invalidate the can_send() cache */ /* invalidate the can_send() cache */
if(mode_type == CHFL_BAN || mode_type == CHFL_QUIET || mode_type == CHFL_EXCEPTION) if(mode_type == CHFL_BAN || mode_type == CHFL_QUIET || mode_type == CHFL_EXCEPTION)