From 48e20cb5f714fbee83889d55505eb99c3f444cda Mon Sep 17 00:00:00 2001
From: Jeremy Lorelli <jeremy.lorelli.1337@gmail.com>
Date: Fri, 27 May 2022 16:41:57 -0700
Subject: [PATCH] Fix crash when aborting instance import

Also turned a loop var into a reference to avoid copies on each iteration
---
 launcher/InstanceImportTask.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/launcher/InstanceImportTask.cpp b/launcher/InstanceImportTask.cpp
index 4bad7251..56081ed1 100644
--- a/launcher/InstanceImportTask.cpp
+++ b/launcher/InstanceImportTask.cpp
@@ -72,7 +72,8 @@ InstanceImportTask::InstanceImportTask(const QUrl sourceUrl, QWidget* parent)
 
 bool InstanceImportTask::abort()
 {
-    m_filesNetJob->abort();
+    if (m_filesNetJob)
+        m_filesNetJob->abort();
     m_extractFuture.cancel();
 
     return false;
@@ -386,7 +387,7 @@ void InstanceImportTask::processFlame()
     {
         auto results = m_modIdResolver->getResults();
         m_filesNetJob = new NetJob(tr("Mod download"), APPLICATION->network());
-        for(auto result: results.files)
+        for(const auto& result: results.files)
         {
             QString filename = result.fileName;
             if(!result.required)