Skip to content

Commit

Permalink
Fix few deltarpm related memory leaks
Browse files Browse the repository at this point in the history
  • Loading branch information
kontura committed Oct 6, 2023
1 parent f15fdce commit 2d19e0b
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion src/deltarpms.c
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,7 @@ cr_deltapackage_from_drpm_base(const char *filename,

deltapackage->nevr = cr_safe_string_chunk_insert_null(
deltapackage->chunk, str);
free(str);

ret = drpm_get_string(delta, DRPM_TAG_SEQUENCE, &str);
if (ret != DRPM_ERR_OK) {
Expand All @@ -141,6 +142,7 @@ cr_deltapackage_from_drpm_base(const char *filename,

deltapackage->sequence = cr_safe_string_chunk_insert_null(
deltapackage->chunk, str);
free(str);

drpm_destroy(&delta);

Expand Down Expand Up @@ -328,7 +330,8 @@ cr_delta_thread(gpointer data, gpointer udata)
cr_DeltaTargetPackage *old = lelem->data;

g_debug("Generating delta %s -> %s", old->path, tpkg->path);
cr_drpm_create(old, tpkg, user_data->outdeltadir, &tmp_err);
char * drpm_path = cr_drpm_create(old, tpkg, user_data->outdeltadir, &tmp_err);
free(drpm_path);
if (tmp_err) {
g_warning("Cannot generate delta %s -> %s : %s",
old->path, tpkg->path, tmp_err->message);
Expand All @@ -338,6 +341,8 @@ cr_delta_thread(gpointer data, gpointer udata)
if (++x == user_data->num_deltas)
break;
}

g_slist_free_full(local_candidates, (GDestroyNotify) cr_deltatargetpackage_free);
}

g_debug("Deltas for \"%s\" (%"G_GINT64_FORMAT") generated",
Expand Down

0 comments on commit 2d19e0b

Please sign in to comment.