'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 <cferg...@redhat.com>
---
 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@redhat.com
https://www.redhat.com/mailman/listinfo/libosinfo

Reply via email to