Merge pull request #4059 from bylaws/log

Windows: Don't assume the log file was successfully opened
FEX-Emu · Sep 11, 2024 · f20e626 · f20e626
2 parents 2672e06 + 0b0e15c
commit f20e626
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Source/Windows/Common/Logging.cpp b/Source/Windows/Common/Logging.cpp
@@ -17,7 +17,7 @@ static void MsgHandler(LogMan::DebugLevels Level, const char* Message) {
   const auto Output = fextl::fmt::format("[{}][{:X}] {}\n", LogMan::DebugLevelStr(Level), GetCurrentThreadId(), Message);
   if (WineDbgOut) {
     WineDbgOut(Output.c_str());
-  } else {
+  } else if (LogFile) {
     fwrite(Output.c_str(), 1, Output.size(), LogFile);
   }
 }
@@ -26,7 +26,7 @@ static void AssertHandler(const char* Message) {
   const auto Output = fextl::fmt::format("[ASSERT] {}\n", Message);
   if (WineDbgOut) {
     WineDbgOut(Output.c_str());
-  } else {
+  } else if (LogFile) {
     fwrite(Output.c_str(), 1, Output.size(), LogFile);
   }
 }