From 6b0e4e6817afad721cb2b0d840e983cf0fdac1b6 Mon Sep 17 00:00:00 2001
From: ChunkyProgrammer <78101139+ChunkyProgrammer@users.noreply.github.com>
Date: Tue, 30 Apr 2024 19:45:29 -0400
Subject: [PATCH] Put temp.delete inside ensure block

---
 src/invidious/user/imports.cr | 47 +++++++++++++++++++----------------
 1 file changed, 25 insertions(+), 22 deletions(-)

diff --git a/src/invidious/user/imports.cr b/src/invidious/user/imports.cr
index 84794991..163a8c74 100644
--- a/src/invidious/user/imports.cr
+++ b/src/invidious/user/imports.cr
@@ -298,30 +298,33 @@ struct Invidious::User
             # Ensure max size of 4MB
             io_sized = IO::Sized.new(file_io, 0x400000)
 
-            temp = File.tempfile(".db") do |tempfile|
-              begin
-                File.write(tempfile.path, io_sized.gets_to_end)
-              rescue
-                return false
-              end
-
-              DB.open("sqlite3://" + tempfile.path) do |db|
-                user.watched += db.query_all("SELECT url FROM streams", as: String)
-                  .map(&.lchop("https://www.youtube.com/watch?v="))
-
-                user.watched.uniq!
-                Invidious::Database::Users.update_watch_history(user)
-
-                user.subscriptions += db.query_all("SELECT url FROM subscriptions", as: String)
-                  .map(&.lchop("https://www.youtube.com/channel/"))
-
-                user.subscriptions.uniq!
-                user.subscriptions = get_batch_channels(user.subscriptions)
-
-                Invidious::Database::Users.update_subscriptions(user)
+            begin
+              temp = File.tempfile(".db") do |tempfile|
+                begin
+                  File.write(tempfile.path, io_sized.gets_to_end)
+                rescue
+                  return false
+                end
+
+                DB.open("sqlite3://" + tempfile.path) do |db|
+                  user.watched += db.query_all("SELECT url FROM streams", as: String)
+                    .map(&.lchop("https://www.youtube.com/watch?v="))
+
+                  user.watched.uniq!
+                  Invidious::Database::Users.update_watch_history(user)
+
+                  user.subscriptions += db.query_all("SELECT url FROM subscriptions", as: String)
+                    .map(&.lchop("https://www.youtube.com/channel/"))
+
+                  user.subscriptions.uniq!
+                  user.subscriptions = get_batch_channels(user.subscriptions)
+
+                  Invidious::Database::Users.update_subscriptions(user)
+                end
               end
+            ensure
+              temp.delete if !temp.nil?
             end
-            temp.delete
           end
         end
       end