From 2b688869e29930df43e1476f43d816dad3b66180 Mon Sep 17 00:00:00 2001
From: Mikhail Babynichev <i@kotrik.ru>
Date: Tue, 5 Apr 2022 22:28:29 +0300
Subject: [PATCH] fix: that defer should be after err checking

---
 api/download/download.go | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/api/download/download.go b/api/download/download.go
index a572a98..5051c3a 100644
--- a/api/download/download.go
+++ b/api/download/download.go
@@ -117,14 +117,14 @@ func downloadBeatmap(c *downloader.Client, b *housekeeper.CachedBeatmap, house *
 	fCbm, errF := b.File()
 	if errF == nil {
 		stat, err := fCbm.Stat()
+		if err != nil {
+			goto DOWNLOAD_MAP
+		}
 		defer func() {
 			// We need to wrap this inside a function because this way the arguments
 			// to DownloadCompleted are actually evaluated during the defer call.
 			b.DownloadCompleted(uint64(stat.Size()), house)
 		}()
-		if err != nil {
-			goto DOWNLOAD_MAP
-		}
 
 		// FILE EXISTS!
 		log.Println("[⬇️][👌] Map found in cache, no need to download!", b.String())
-- 
GitLab