diff options
author | Richard Hughes <richard@hughsie.com> | 2015-12-27 21:28:23 +0000 |
---|---|---|
committer | Richard Hughes <richard@hughsie.com> | 2015-12-28 08:55:14 +0000 |
commit | 1e38c97c85474f8b67ccf98473bd19db1f2f7a42 (patch) | |
tree | 1e72c1854a8d4491f1863188b51b413e3077a683 | |
parent | 646e3a1e5c6be58be920d15455919cb5a8502682 (diff) | |
download | appstream-glib-1e38c97c85474f8b67ccf98473bd19db1f2f7a42.tar.gz |
trivial: Fix a small memory leak when parsing debs
-rw-r--r-- | libappstream-builder/asb-package-deb.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libappstream-builder/asb-package-deb.c b/libappstream-builder/asb-package-deb.c index d74bfe2..d854788 100644 --- a/libappstream-builder/asb-package-deb.c +++ b/libappstream-builder/asb-package-deb.c @@ -51,7 +51,6 @@ asb_package_deb_ensure_simple (AsbPackage *pkg, GError **error) { const gchar *argv[4] = { "dpkg", "--field", "fn", NULL }; gchar *tmp; - gchar **vr; guint i; guint j; g_autofree gchar *output = NULL; @@ -77,6 +76,7 @@ asb_package_deb_ensure_simple (AsbPackage *pkg, GError **error) continue; } if (g_str_has_prefix (lines[i], "Version: ")) { + g_auto(GStrv) vr = NULL; vr = g_strsplit (lines[i] + 9, "-", 2); tmp = g_strstr_len (vr[0], -1, ":"); if (tmp == NULL) { @@ -88,10 +88,10 @@ asb_package_deb_ensure_simple (AsbPackage *pkg, GError **error) asb_package_set_version (pkg, tmp + 1); } asb_package_set_release (pkg, vr[1]); - g_strfreev (vr); continue; } if (g_str_has_prefix (lines[i], "Depends: ")) { + g_auto(GStrv) vr = NULL; vr = g_strsplit (lines[i] + 9, ", ", -1); for (j = 0; vr[j] != NULL; j++) { tmp = g_strstr_len (vr[j], -1, " "); |