[patch] Accessing freed memory crash

[Mikulas Patocka]

On Sat, 12 Aug 2006, Leonard den Ottolander wrote:

> Hello Mikulas,
>
> On Fri, 2006-08-11 at 01:49 +0200, Mikulas Patocka wrote:
>> check_selects calls a callback when there is an event on handle. The
>> callback itself can remove the event with delete_select_channel --- when
>> it happens, check_selects accesses defunct p->next pointer and crash may
>> happen.
>
> Committed using a do/while. Thanks for spotting and fixing.
>
> Leonard.

Hi

I think the code you committed is wrong. Imagine this: you have one event 
in select list and that event is set in select_set. On the first pass, you 
call callback and set retry to TRUE. Callback removes the event. You 
return to "do" cycle, now select_list is empty, you never get to 
retry=FALSE statement, and you loop forever with retry == TRUE.

Mikulas

SelectList *p;
gboolean retry = FALSE;

if (disabled_channels)
     return;

do
     for (p = select_list; p; p = p->next)
         if (FD_ISSET (p->fd, select_set)) {
             FD_CLR (p->fd, select_set);
             (*p->callback)(p->fd, p->info);
             retry = TRUE;
             break;
         } else
             retry = FALSE;
while (retry);