From 4166372d53b44e44a1a2f5a59dad2fb54acf52dc Mon Sep 17 00:00:00 2001
From: Derock <derock@derock.dev>
Date: Wed, 13 Aug 2025 12:52:16 -0400
Subject: [PATCH] core/log: Fix nullptr crash in ThreadLogging

---
 src/core/logging.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/core/logging.cpp b/src/core/logging.cpp
index cb3a2142..21548146 100644
--- a/src/core/logging.cpp
+++ b/src/core/logging.cpp
@@ -458,13 +458,13 @@ void ThreadLogging::onMessage(const LogMessage& msg, bool showInSparse) {
 	}
 
 	if (!this->detailedWriter.write(msg) || (this->detailedFile && !this->detailedFile->flush())) {
+		this->detailedWriter.setDevice(nullptr);
+
 		if (this->detailedFile) {
 			qCCritical(logLogging) << "Detailed logger failed to write. Ending detailed logs.";
+			this->detailedFile->close();
+			this->detailedFile = nullptr;
 		}
-
-		this->detailedWriter.setDevice(nullptr);
-		this->detailedFile->close();
-		this->detailedFile = nullptr;
 	}
 }