'tmp' is used to iterate over the list of files, so it will be NULL at the end of the iteration, and g_list_free() will be a no-op. The initial list is stored in 'children', so free that instead. Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> --- osinfo/osinfo_loader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/osinfo/osinfo_loader.c b/osinfo/osinfo_loader.c index b2635dfb..cc5c20f5 100644 --- a/osinfo/osinfo_loader.c +++ b/osinfo/osinfo_loader.c @@ -2060,7 +2060,7 @@ static void osinfo_loader_find_files(OsinfoLoader *loader, tmp = tmp->next; } g_object_unref(ents); - g_list_free(tmp); + g_list_free(children); } else { OSINFO_ERROR(&error, "Unexpected file type"); g_propagate_error(err, error); -- 2.14.3 _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo