On Sun, 6 Nov 2011 12:14:06 +0900 Bluezery <[email protected]> said:
yes! you're right. bad thread safety changes i think.
> Dear all,
>
>
> When ecore_main_fd_handler_add() failed to add poll on fd, it should
> return NULL.
> To prevent NULL Dereference. I made "fdh" NULL after free.
> Please review this patch.
>
> Index: src/lib/ecore/ecore_main.c
> ===================================================================
> --- src/lib/ecore/ecore_main.c (리비전 64791)
> +++ src/lib/ecore/ecore_main.c (작업 사본)
> @@ -998,6 +998,7 @@ ecore_main_fd_handler_add(int
> int err = errno;
> ERR("Failed to add poll on fd %d (errno = %d: %s)!", fd, err,
> strerror(err));
> free(fdh);
> + fdh = NULL;
> goto unlock;
> }
> fdh->read_active = EINA_FALSE;
>
> ------------------------------------------------------------------------------
> RSA(R) Conference 2012
> Save $700 by Nov 18
> Register now
> http://p.sf.net/sfu/rsa-sfdev2dev1
> _______________________________________________
> enlightenment-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
--
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler) [email protected]
------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
