On Tue, 09 Jun 2015 10:38:03 +0200 Stefan Schmidt <ste...@osg.samsung.com> said:

> Hello.
> 
> I just run through some of the coverity issues and spotted some where i 
> would the actual code author to have a look and either fix them or mark 
> as false positive.
> 
> First of all the 4 issues with potential high impact:
> 
> 1304728 Resource leak 
> <https://scan6.coverity.com:8443/docs/en/cov_checker_ref.html#static_checker_RESOURCE_LEAK>
>  
> in eo_lexer.c (q66)
> 1297407 Resource leak 
> <https://scan6.coverity.com:8443/docs/en/cov_checker_ref.html#static_checker_RESOURCE_LEAK>
>  
> in eina_btlog.c (raster)

if you know what the binary does/is for... this is zero impact :)

> 1294210 Uninitialized scalar variable 
> <https://scan6.coverity.com:8443/docs/en/cov_checker_ref.html#static_checker_UNINIT>
>  
> in ector_software_rasterrizer.c (cedric)
> 1267458 Read from pointer after free 
> <https://scan6.coverity.com:8443/docs/en/cov_checker_ref.html#static_checker_USE_AFTER_FREE>
>  
> in edje_pick.c (sachiel, maybe someone else if he is not around)
> 
> And three more that jumped into my eye:
> 
> 1304560 Bad bit shift operation 
> <https://scan6.coverity.com:8443/docs/en/cov_checker_ref.html#static_checker_BAD_SHIFT>
>  
> in evas_gl_core.c (Dongyeon Kim)
> 1267475 Missing unlock 
> <https://scan6.coverity.com:8443/docs/en/cov_checker_ref.html#static_checker_LOCK>
>  
> in eina_thread_queue.c (raster)

and i think coverity is wrong here. it THINKS sem_wait() is a lock... it's not.

http://linux.die.net/man/3/sem_wait

> 1106338 Thread deadlock 
> <https://scan6.coverity.com:8443/docs/en/cov_checker_ref.html#static_checker_ORDER_REVERSAL>
>  
> in evas_image_scalecache.c (cedric)
> 
> There are more if you don't want to stop with these. :)
> 
> regards
> Stefan Schmidt
> 
> 
> ------------------------------------------------------------------------------
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    ras...@rasterman.com


------------------------------------------------------------------------------
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to