Hello. 

1. _pool_tex_dynamic_new function, it didn’t set pt to NULL when
secsym_eglCreateImage function failed.
In this case, it returns wrong pt pointer and it has possibility to make
crash.
So I add free pt code and return NULL code into _pool_tex_dynamic_new
function.

2. I modified eng_image_data_get of gl engine.

If Evas_GL_Image's texture creation failed and evas_gl_image's cache image
was droped, 
Im->im can be NULL. So I add check code. 
Example: evas_gl_common_image_content_hint_set
     1) EVAS_IMAGE_CONTENT_HINT_DYNAMIC , it drop cache image
     2) if evas_gl_common_texture_dynamic_new failed
     3) then, im->im =NULL, im->tex=NULL
In this situation, if application call's evas_object_image_data_get
function, 
It make crash in evas_cache_image_load_data function.

3. I think function's related with evas_object's engine data have to be
return NULL if it failed.
If function's returns null, evas object code can handle error more easily.  
But evas object's code was implemented differently each case. 
Does my suggestion right?
I add engine data null check code to evas_object_image based on upper
consumtion.
If it is wrong , the patch code related with evas object image have to be
removed.
If it is right , I will survey other evas object type also.  

Thanks. 

 
----------------------------------------------
Jiyoun Park
 
Mobile S/W Platform Lab
DMC R&D Center
SAMSUNG ELECTRONICS CO. ,LTD
 
TEL: +82-31-279-0619
Mobile: +82-10-9871-0703
[email protected]
----------------------------------------------





Attachment: evas_gl_texture_creation_buf_fix.patch
Description: Binary data

------------------------------------------------------------------------------
EMC VNX: the world's simplest storage, starting under $10K
The only unified storage solution that offers unified management 
Up to 160% more powerful than alternatives and 25% more efficient. 
Guaranteed. http://p.sf.net/sfu/emc-vnx-dev2dev
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to