(struct handle_list).name was unnecessarily made const in: commit caa9eda091d6 ("trace-cmd split: Store instances in local list") Remove it. Also fix bad indentation. Signed-off-by: Pierre Gondois <pierre.gondois@xxxxxxx> --- Notes: v6: - New patch: Remove unnecessary const for (struct handle_list).name tracecmd/trace-split.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tracecmd/trace-split.c b/tracecmd/trace-split.c index 2c311b3d..cb6242d8 100644 --- a/tracecmd/trace-split.c +++ b/tracecmd/trace-split.c @@ -53,9 +53,9 @@ struct cpu_data { struct handle_list { struct list_head list; - const char *name; + char *name; int index; - struct tracecmd_input *handle; + struct tracecmd_input *handle; /* Identify the top instance in the input trace. */ bool was_top_instance; @@ -114,7 +114,7 @@ static void free_handles(struct list_head *list) while (!list_empty(list)) { item = container_of(list->next, struct handle_list, list); list_del(&item->list); - free((char *)item->name); + free(item->name); tracecmd_close(item->handle); free(item); } -- 2.25.1