From: Igor Kulaychuk <i.kulaychuk@samsung.com>
Date: Wed, 12 Dec 2018 19:48:25 +0000 (+0300)
Subject: Fix localtime thread-safety issues in Logger
X-Git-Tag: submit/tizen_5.0_unified/20181217.133158^2~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0bb630131b6b3d4c7eba35ef53446c6e5132e451;p=sdk%2Ftools%2Fnetcoredbg.git

Fix localtime thread-safety issues in Logger
---

diff --git a/src/debug/netcoredbg/logger.cpp b/src/debug/netcoredbg/logger.cpp
index dfa91d9..84ee7d3 100644
--- a/src/debug/netcoredbg/logger.cpp
+++ b/src/debug/netcoredbg/logger.cpp
@@ -87,14 +87,25 @@ const std::string FileLogger::infoStr = "INFO";
 const std::string FileLogger::warnStr = "WARN";
 const std::string FileLogger::errorStr = "ERROR";
 
+static void get_local_time(std::tm *tm_snapshot)
+{
+    auto system_now = std::chrono::system_clock::now();
+    std::time_t time = std::chrono::system_clock::to_time_t(system_now);
+#if (defined(WIN32) || defined(_WIN32) || defined(__WIN32__))
+    ::localtime_s(tm_snapshot, &time);
+#else
+    ::localtime_r(&time, tm_snapshot); // POSIX
+#endif
+}
 
 FileLogger::FileLogger()
 {
-    auto time = std::time(nullptr);
     std::string tmpPath = GetTempFolder();
     std::ostringstream oss;
 
-    oss << std::put_time(std::localtime(&time), "%Y_%m_%d__%H_%M_%S");
+    std::tm tm_snapshot;
+    get_local_time(&tm_snapshot);
+    oss << std::put_time(&tm_snapshot, "%Y_%m_%d__%H_%M_%S");
 
     logFile = fopen(std::string(tmpPath + filenameBase + oss.str() + ".log").c_str(), "w+");
 }
@@ -121,9 +132,10 @@ const std::string& FileLogger::LevelToString(LogLevel level)
 
 std::string FileLogger::FormatMessageString(LogLevel level, const std::string &str)
 {
-    auto time = std::time(nullptr);
     std::ostringstream oss;
-    oss << std::put_time(std::localtime(&time), "%y-%m-%d %OH:%OM:%OS") << " " << LevelToString(level) << " " << str << std::endl;
+    std::tm tm_snapshot;
+    get_local_time(&tm_snapshot);
+    oss << std::put_time(&tm_snapshot, "%y-%m-%d %OH:%OM:%OS") << " " << LevelToString(level) << " " << str << std::endl;
 
     return oss.str();
 }