free some leaks reported by valgrind in src/bin tests.
SVN revision: 41117
This commit is contained in:
parent
728b003fee
commit
3871aed422
|
@ -1222,9 +1222,12 @@ efreet_desktop_command_progress_get(Efreet_Desktop *desktop, Eina_List *files,
|
||||||
if (command->num_pending == 0)
|
if (command->num_pending == 0)
|
||||||
{
|
{
|
||||||
Eina_List *execs;
|
Eina_List *execs;
|
||||||
|
char *exec;
|
||||||
|
|
||||||
execs = efreet_desktop_command_build(command);
|
execs = efreet_desktop_command_build(command);
|
||||||
ret = efreet_desktop_command_execs_process(command, execs);
|
ret = efreet_desktop_command_execs_process(command, execs);
|
||||||
eina_list_free(execs);
|
EINA_LIST_FREE(execs, exec)
|
||||||
|
free(exec);
|
||||||
efreet_desktop_command_free(command);
|
efreet_desktop_command_free(command);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -1949,7 +1949,10 @@ efreet_menu_handle_legacy_dir_helper(Efreet_Menu_Internal *root,
|
||||||
|
|
||||||
/* if the .desktop has categories it isn't legacy */
|
/* if the .desktop has categories it isn't legacy */
|
||||||
if (efreet_desktop_category_count_get(desktop) != 0)
|
if (efreet_desktop_category_count_get(desktop) != 0)
|
||||||
|
{
|
||||||
|
efreet_desktop_free(desktop);
|
||||||
continue;
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
/* XXX: This will disappear when the .desktop is free'd */
|
/* XXX: This will disappear when the .desktop is free'd */
|
||||||
efreet_desktop_category_add(desktop, "Legacy");
|
efreet_desktop_category_add(desktop, "Legacy");
|
||||||
|
@ -2547,9 +2550,12 @@ efreet_menu_entry_new(void)
|
||||||
EAPI void
|
EAPI void
|
||||||
efreet_menu_free(Efreet_Menu *entry)
|
efreet_menu_free(Efreet_Menu *entry)
|
||||||
{
|
{
|
||||||
|
Efreet_Menu *sub;
|
||||||
|
|
||||||
IF_RELEASE(entry->name);
|
IF_RELEASE(entry->name);
|
||||||
IF_RELEASE(entry->icon);
|
IF_RELEASE(entry->icon);
|
||||||
entry->entries = eina_list_free(entry->entries);
|
EINA_LIST_FREE(entry->entries, sub)
|
||||||
|
efreet_menu_free(sub);
|
||||||
IF_RELEASE(entry->id);
|
IF_RELEASE(entry->id);
|
||||||
if (entry->desktop) efreet_desktop_free(entry->desktop);
|
if (entry->desktop) efreet_desktop_free(entry->desktop);
|
||||||
FREE(entry);
|
FREE(entry);
|
||||||
|
@ -3420,7 +3426,10 @@ efreet_menu_app_dir_scan(Efreet_Menu_Internal *internal, const char *path, const
|
||||||
if (eina_list_search_unsorted(internal->app_pool,
|
if (eina_list_search_unsorted(internal->app_pool,
|
||||||
EINA_COMPARE_CB(efreet_menu_cb_md_compare_ids),
|
EINA_COMPARE_CB(efreet_menu_cb_md_compare_ids),
|
||||||
buf2))
|
buf2))
|
||||||
|
{
|
||||||
|
if (desktop) efreet_desktop_free(desktop);
|
||||||
continue;
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
menu_desktop = efreet_menu_desktop_new();
|
menu_desktop = efreet_menu_desktop_new();
|
||||||
menu_desktop->desktop = desktop;
|
menu_desktop->desktop = desktop;
|
||||||
|
|
Loading…
Reference in New Issue