Skip to content

Commit

Permalink
Merge pull request #1932 from H3rnand3zzz/cleanup/refactor-conf-strin…
Browse files Browse the repository at this point in the history
…g-list

Fix mem leak in `conf_string_list_add`
  • Loading branch information
jubalh authored Nov 20, 2023
2 parents 62e1de0 + bd2821f commit e143513
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions src/config/conflists.c
Original file line number Diff line number Diff line change
Expand Up @@ -65,14 +65,16 @@ conf_string_list_add(GKeyFile* keyfile, const char* const group, const char* con
}

// Add item to the existing list
gchar** new_list = g_new(gchar*, length + 2);
const gchar** new_list = g_new(const gchar*, length + 2);
for (gsize i = 0; i < length; ++i) {
new_list[i] = g_strdup(list[i]);
new_list[i] = list[i];
}
new_list[length] = g_strdup(item);
new_list[length] = item;
new_list[length + 1] = NULL;

g_key_file_set_string_list(keyfile, group, key, (const gchar* const*)new_list, length + 1);
g_key_file_set_string_list(keyfile, group, key, new_list, length + 1);

g_free(new_list);

return TRUE;
}
Expand Down

0 comments on commit e143513

Please sign in to comment.