Re: [E-devel] [PATCH] Reinit main loop if somebody passes a closed fd to epoll

2010-10-20 Thread Gustavo Sverzut Barbieri
On Tue, Oct 19, 2010 at 10:47 PM, Mike McCormack wrote: > On 10/19/2010 10:19 PM, Gustavo Sverzut Barbieri wrote: >> >> On Tue, Oct 19, 2010 at 11:16 AM, Mike McCormack >>  wrote: >>> >>> Hi Guys, >>> >>> Superficially tested... issue is if somebody does: >>> >>> close(fd); >>> ecore_main_fd_hand

Re: [E-devel] [PATCH] Reinit main loop if somebody passes a closed fd to epoll

2010-10-19 Thread Mike McCormack
On 10/19/2010 10:19 PM, Gustavo Sverzut Barbieri wrote: On Tue, Oct 19, 2010 at 11:16 AM, Mike McCormack wrote: Hi Guys, Superficially tested... issue is if somebody does: close(fd); ecore_main_fd_handler_del(fdh); epoll can't remove the fd because it's already closed, so we get an EBADF fr

Re: [E-devel] [PATCH] Reinit main loop if somebody passes a closed fd to epoll

2010-10-19 Thread Gustavo Sverzut Barbieri
On Tue, Oct 19, 2010 at 11:16 AM, Mike McCormack wrote: > Hi Guys, > > Superficially tested... issue is if somebody does: > > close(fd); > ecore_main_fd_handler_del(fdh); > > epoll can't remove the fd because it's already closed, so we get an EBADF > from the kernel. > Work around the broken case