e_module: fix bug detected by static analysis tool
authorDoyoun Kang <doyoun.kang@samsung.com>
Mon, 17 Jul 2017 12:15:48 +0000 (21:15 +0900)
committerDoyoun Kang <doyoun.kang@samsung.com>
Tue, 18 Jul 2017 02:25:21 +0000 (11:25 +0900)
Change-Id: If0e7ff20cd80d67897d66c14b7d60a7ae8eef4f3

src/bin/e_module.c

index 4b8a1e954447f74a9df2875109e0a015346f8c05..8e3eedf5ee69cdce6eff527ae72d3071e0033ef3 100644 (file)
@@ -478,9 +478,12 @@ init_done:
         E_Config_Module *module;
 
         module = E_NEW(E_Config_Module, 1);
-        module->name = eina_stringshare_add(m->name);
-        module->enabled = 0;
-        e_config->modules = eina_list_append(e_config->modules, module);
+        if (module)
+          {
+             module->name = eina_stringshare_add(m->name);
+             module->enabled = 0;
+             e_config->modules = eina_list_append(e_config->modules, module);
+          }
         e_config_save_queue();
      }
    if (modpath) eina_stringshare_del(modpath);
@@ -536,6 +539,7 @@ e_module_enable(E_Module *m)
                   e_config_save_queue();
 
                   ev = E_NEW(E_Event_Module_Update, 1);
+                  if (!ev) break;
                   ev->name = eina_stringshare_ref(em->name);
                   ev->enabled = 1;
                   ecore_event_add(E_EVENT_MODULE_UPDATE, ev,
@@ -580,6 +584,7 @@ e_module_disable(E_Module *m)
              e_config_save_queue();
 
              ev = E_NEW(E_Event_Module_Update, 1);
+             if (!ev) break;
              ev->name = eina_stringshare_ref(em->name);
              ev->enabled = 0;
              ecore_event_add(E_EVENT_MODULE_UPDATE, ev,
@@ -799,6 +804,7 @@ _e_module_dialog_disable_create(const char *title, const char *body, E_Module *m
 #endif
 
    dd = E_NEW(Disable_Dialog, 1);
+   if (!dd) return;
    dd->title = strdup(title);
    dd->body = strdup(body);
    dd->m = m;