[Bf-blender-cvs] [981380cf6d0] blender-v2.92-release: VSE: Fix memory leak when building proxies

Richard Antalik noreply at git.blender.org
Tue Jan 26 17:51:13 CET 2021


Commit: 981380cf6d086517cc1b5dfc60e9bde6253e5bd7
Author: Richard Antalik
Date:   Tue Jan 26 17:33:03 2021 +0100
Branches: blender-v2.92-release
https://developer.blender.org/rB981380cf6d086517cc1b5dfc60e9bde6253e5bd7

VSE: Fix memory leak when building proxies

With no valid strips selected, file_list is not freed.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D10191

===================================================================

M	source/blender/editors/space_sequencer/sequencer_proxy.c

===================================================================

diff --git a/source/blender/editors/space_sequencer/sequencer_proxy.c b/source/blender/editors/space_sequencer/sequencer_proxy.c
index b208f320591..b9698492aa5 100644
--- a/source/blender/editors/space_sequencer/sequencer_proxy.c
+++ b/source/blender/editors/space_sequencer/sequencer_proxy.c
@@ -168,13 +168,13 @@ static void seq_proxy_build_job(const bContext *C, ReportList *reports)
   }
   SEQ_CURRENT_END;
 
+  BLI_gset_free(file_list, MEM_freeN);
+
   if (!selected) {
     BKE_reportf(reports, RPT_WARNING, "Select movie or image strips");
     return;
   }
 
-  BLI_gset_free(file_list, MEM_freeN);
-
   if (selected && !WM_jobs_is_running(wm_job)) {
     G.is_break = false;
     WM_jobs_start(CTX_wm_manager(C), wm_job);



More information about the Bf-blender-cvs mailing list