On Thu, Jun 21, 2012 at 4:34 AM, Enlightenment SVN
<no-re...@enlightenment.org> wrote:
> Log:
> eio: Fix memory leaks spotted by clang static analyzer

I think this patch should be reverted and was a complete false
positive detection by clang. eio_file_set always call
ecore_thread_run. If ecore_thread_run fails to create and queue the
new task in the thread worker pool, it will always call the cancel_cb.
This cancel callback here do cleanup and destroy all the Eio structure
related to this task.

So after after a call to eio_file_set, either it succeeded and you can
play with the content of the Eio task structure, or you should not
touch it at all because it was destroyed. Question is what are you
trying to fix with this patch ?
-- 
Cedric BAIL

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to