raster pushed a commit to branch master.

http://git.enlightenment.org/core/enlightenment.git/commit/?id=8803e7fed2ffd67981d0fc517c73ad9ae9ca2eb9

commit 8803e7fed2ffd67981d0fc517c73ad9ae9ca2eb9
Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com>
Date:   Sat Nov 30 11:51:50 2013 +0900

    double free coverty complaint - it doesn't know the action re-execs
    
    this will shut coverty up (hopefully). it doesn't know that a re-exec
    of the process will mean the code never gets out of the if ()...
    unless the restart action is never found... and that never happens.
---
 src/modules/conf_theme/e_int_config_theme.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/modules/conf_theme/e_int_config_theme.c 
b/src/modules/conf_theme/e_int_config_theme.c
index dcaf771..6e351a4 100644
--- a/src/modules/conf_theme/e_int_config_theme.c
+++ b/src/modules/conf_theme/e_int_config_theme.c
@@ -756,6 +756,7 @@ _basic_apply_data(E_Config_Dialog *cfd __UNUSED__, 
E_Config_Dialog_Data *cfdata)
              elm_config_all_flush();
              elm_config_save();
              free(name);
+             name = NULL;
              a = e_action_find("restart");
              if ((a) && (a->func.go)) a->func.go(NULL, NULL);
           }

-- 


Reply via email to