From 4bc2a28760070d25c76240779593ba06870d6ee9 Mon Sep 17 00:00:00 2001
From: tildearrow <me@tildearrow.org>
Date: Tue, 11 Jul 2023 19:30:13 -0500
Subject: [PATCH] GUI: fix memory leak in backup task

issue #1233
---
 src/gui/gui.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/gui/gui.cpp b/src/gui/gui.cpp
index 6eefb6913..1d3737f0a 100644
--- a/src/gui/gui.cpp
+++ b/src/gui/gui.cpp
@@ -5849,13 +5849,12 @@ bool FurnaceGUI::loop() {
               if (outFile!=NULL) {
                 if (fwrite(w->getFinalBuf(),1,w->size(),outFile)!=w->size()) {
                   logW("did not write backup entirely: %s!",strerror(errno));
-                  w->finish();
                 }
                 fclose(outFile);
               } else {
                 logW("could not save backup: %s!",strerror(errno));
-                w->finish();
               }
+              w->finish();
 
               // delete previous backup if there are too many
               delFirstBackup(backupBaseName);