devilhorns pushed a commit to branch master.

commit 4e103fd858f449b2fa6b0449cbc92c89184cd818
Author: Chris Michael <[email protected]>
Date:   Mon Jul 8 08:30:26 2013 +0100

    Fix mem leak reported by Coverity (don't just free fdata if size > 0).
    
    NB: Fixes Coverity CID1039665
    
    Signed-off-by: Chris Michael <[email protected]>
---
 src/lib/evas/canvas/evas_font_dir.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/evas/canvas/evas_font_dir.c 
b/src/lib/evas/canvas/evas_font_dir.c
index 5eea308..00836a3 100644
--- a/src/lib/evas/canvas/evas_font_dir.c
+++ b/src/lib/evas/canvas/evas_font_dir.c
@@ -633,9 +633,9 @@ evas_font_load(Evas *eo_evas, Evas_Font_Description *fdesc, 
const char *source,
                                 if ((fdata) && (fsize > 0))
                                   {
                                      ok = 
evas->engine.func->font_memory_add(evas->engine.data.output, font, source, nm, 
size, fdata, fsize, wanted_rend);
-                                     free(fdata);
                                   }
                                 eet_close(ef);
+                                 free(fdata);
                              }
                         }
                       else

-- 

------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev

Reply via email to