forked from enlightenment/efl
Fix memleak reported by Coverity: If we fail to allocate a new
internal menu, then free the previously allocated memory for xml file. NB: Fixed Coverity CID 1039183 Signed-off-by: Chris Michael <cp.michael@samsung.com>
This commit is contained in:
parent
c04cfde0e5
commit
aeb4b7f7b9
|
@ -1452,7 +1452,11 @@ efreet_menu_merge(Efreet_Menu_Internal *parent, Efreet_Xml *xml, const char *pat
|
||||||
}
|
}
|
||||||
|
|
||||||
internal = efreet_menu_internal_new(parent);
|
internal = efreet_menu_internal_new(parent);
|
||||||
if (!internal) return 0;
|
if (!internal)
|
||||||
|
{
|
||||||
|
efreet_xml_del(merge_xml);
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
efreet_menu_path_set(internal, path);
|
efreet_menu_path_set(internal, path);
|
||||||
efreet_menu_handle_menu(internal, merge_xml);
|
efreet_menu_handle_menu(internal, merge_xml);
|
||||||
efreet_menu_concatenate(parent, internal);
|
efreet_menu_concatenate(parent, internal);
|
||||||
|
|
Loading…
Reference in New Issue