From f4d46916fab002cefea632dad685e6beb5b4a427 Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Thu, 7 Sep 2017 22:13:28 +1000 Subject: UI: fix memory leak when copy-to-selected failed --- source/blender/editors/interface/interface_ops.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'source/blender/editors/interface') diff --git a/source/blender/editors/interface/interface_ops.c b/source/blender/editors/interface/interface_ops.c index 580ad025086..ce3cf24f292 100644 --- a/source/blender/editors/interface/interface_ops.c +++ b/source/blender/editors/interface/interface_ops.c @@ -497,10 +497,11 @@ static bool copy_to_selected_button(bContext *C, bool all, bool poll) char *path = NULL; bool use_path_from_id; CollectionPointerLink *link; - ListBase lb; + ListBase lb = {NULL}; - if (!UI_context_copy_to_selected_list(C, &ptr, prop, &lb, &use_path_from_id, &path)) - return success; + if (!UI_context_copy_to_selected_list(C, &ptr, prop, &lb, &use_path_from_id, &path)) { + goto finally; + } for (link = lb.first; link; link = link->next) { if (link->ptr.data != ptr.data) { @@ -542,6 +543,7 @@ static bool copy_to_selected_button(bContext *C, bool all, bool poll) } } +finally: MEM_SAFE_FREE(path); BLI_freelistN(&lb); } -- cgit v1.2.3