logging: Fix potential use-after-free in LogPrintStr(...)

1 files changed, 5 insertions, 2 deletions

diff --git a/src/logging.cpp b/src/logging.cpp
index 10a3b1895..604696937 100644
--- a/src/logging.cpp
+++ b/src/logging.cpp
@@ -222,8 +222,11 @@ int BCLog::Logger::LogPrintStr(const std::string &str)
             // reopen the log file, if requested
             if (m_reopen_file) {
                 m_reopen_file = false;
-                if (fsbridge::freopen(m_file_path,"a",m_fileout) != nullptr)
-                    setbuf(m_fileout, nullptr); // unbuffered
+                m_fileout = fsbridge::freopen(m_file_path, "a", m_fileout);
+                if (!m_fileout) {
+                    return ret;
+                }
+                setbuf(m_fileout, nullptr); // unbuffered
             }
 
             ret = FileWriteStr(strTimestamped, m_fileout);