devilhorns pushed a commit to branch master.

commit 292d23321c6fc17af93d36fb4dae3108585a9645
Author: Chris Michael <[email protected]>
Date:   Tue Jul 2 09:52:19 2013 +0100

    Check for valid out_file before trying to write eet data to it.
    
    NB: Fixes klockwork issue.
    
    Signed-off-by: Chris Michael <[email protected]>
---
 src/bin/edje/edje_pick.c | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/src/bin/edje/edje_pick.c b/src/bin/edje/edje_pick.c
index 42e672c..ad2208c 100644
--- a/src/bin/edje/edje_pick.c
+++ b/src/bin/edje/edje_pick.c
@@ -1367,19 +1367,21 @@ main(int argc, char **argv)
           fl->list = eina_list_append(fl->list, fnt->f);
      }
 
-   {
-      /* Write Fonts from all files */
-      Eet_Data_Descriptor *_font_list_edd = NULL;
-      Eet_Data_Descriptor *_font_edd;
-
-      _edje_data_font_list_desc_make(&_font_list_edd, &_font_edd);
-      bytes = eet_data_write(out_file->ef, _font_list_edd,
-                             "edje_source_fontmap", fl, comp_mode);
-      VERBOSE(EINA_LOG_INFO("Wrote <%d> bytes for fontmap.\n", bytes));
-
-      eet_data_descriptor_free(_font_list_edd);
-      eet_data_descriptor_free(_font_edd);
-   }
+   if (out_file)
+     {
+        /* Write Fonts from all files */
+        Eet_Data_Descriptor *_font_list_edd = NULL;
+        Eet_Data_Descriptor *_font_edd;
+
+        _edje_data_font_list_desc_make(&_font_list_edd, &_font_edd);
+        bytes = eet_data_write(out_file->ef, _font_list_edd,
+                               "edje_source_fontmap", fl, comp_mode);
+        VERBOSE(EINA_LOG_INFO("Wrote <%d> bytes for fontmap.\n", bytes));
+
+        eet_data_descriptor_free(_font_list_edd);
+        eet_data_descriptor_free(_font_edd);
+     }
+
    free(fl);
 
    if (sets)

-- 

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

Build for Windows Store.

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

Reply via email to