From c8477338c49b9c069d97abc008196916210b0629 Mon Sep 17 00:00:00 2001
From: Kamil Cudnik <kcudnik@gmail.com>
Date: Mon, 16 Nov 2020 19:22:24 +0100
Subject: [PATCH] Fix LGTM localtime function warnings (#707)

Signed-off-by: kcudnik <kcudnik@gmail.com>
---
 lib/src/Recorder.cpp      | 15 ++++++---------
 saisdkdump/saisdkdump.cpp |  3 ++-
 syncd/PortMapParser.cpp   |  2 +-
 3 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/lib/src/Recorder.cpp b/lib/src/Recorder.cpp
index 4a19b52b33e4..5461cbaf941c 100644
--- a/lib/src/Recorder.cpp
+++ b/lib/src/Recorder.cpp
@@ -1,7 +1,3 @@
-#include "swss/table.h"
-#include <vector>
-#include <fstream>
-
 #include "Recorder.h"
 
 #include "meta/sai_serialize.h"
@@ -11,6 +7,8 @@
 #include <inttypes.h>
 
 #include <cstring>
+#include <vector>
+#include <fstream>
 
 using namespace sairedis;
 using namespace saimeta;
@@ -216,7 +214,10 @@ std::string Recorder::getTimestamp()
 
     gettimeofday(&tv, NULL);
 
-    size_t size = strftime(buffer, 32, "%Y-%m-%d.%T.", localtime(&tv.tv_sec));
+    struct tm now;
+    localtime_r(&tv.tv_sec, &now);
+
+    size_t size = strftime(buffer, 32, "%Y-%m-%d.%T.", &now);
 
     snprintf(&buffer[size], 32, "%06ld", tv.tv_usec);
 
@@ -283,7 +284,6 @@ void Recorder::recordQueryAttributeCapabilityResponse(
     recordLine("Q|attribute_capability|" + sai_serialize_status(status) + "|" + joinFieldValues(arguments));
 }
 
-
 void Recorder::recordQueryAttributeEnumValuesCapability(
         _In_ const std::string& key,
         _In_ const std::vector<swss::FieldValueTuple>& arguments)
@@ -514,7 +514,6 @@ void Recorder::recordBulkGenericSet(
     recordLine("S|" + key + joined);
 }
 
-
 void Recorder::recordBulkGenericSetResponse(
         _In_ sai_status_t status,
         _In_ uint32_t objectCount,
@@ -833,7 +832,6 @@ void Recorder::recordRemove(                                            \
     _X(ROUTE_ENTRY,route_entry);            \
     _X(NAT_ENTRY,nat_entry);                \
 
-
 REDIS_DECLARE_EVERY_ENTRY(DECLARE_RECORD_REMOVE_ENTRY)
 
 #define DECLARE_RECORD_CREATE_ENTRY(OT,ot)                                                      \
@@ -940,7 +938,6 @@ void Recorder::recordQueryAttributeCapability(
     recordQueryAttributeCapability(key, values);
 }
 
-
 void Recorder::recordQueryAttributeCapabilityResponse(
         _In_ sai_status_t status,
         _In_ sai_object_type_t objectType,
diff --git a/saisdkdump/saisdkdump.cpp b/saisdkdump/saisdkdump.cpp
index 462d3f881f94..bdcd01f898ed 100644
--- a/saisdkdump/saisdkdump.cpp
+++ b/saisdkdump/saisdkdump.cpp
@@ -111,7 +111,8 @@ int main(int argc, char **argv)
     {
         std::ostringstream strStream;
         time_t t = time(NULL);
-        struct tm *now = localtime(&t);
+        struct tm local_tm;
+        struct tm *now = localtime_r(&t, &local_tm);
         strStream << "/tmp/saisdkdump_" << now->tm_mday << "_" << now->tm_mon + 1 << "_" << now->tm_year + 1900 << "_" << now->tm_hour << "_" << now->tm_min << "_" << now->tm_sec;
         fileName = strStream.str();
         SWSS_LOG_INFO("The dump file is not specified, generated \"%s\" file name", fileName.c_str());
diff --git a/syncd/PortMapParser.cpp b/syncd/PortMapParser.cpp
index d5412fb79180..eaa13e9ee1b0 100644
--- a/syncd/PortMapParser.cpp
+++ b/syncd/PortMapParser.cpp
@@ -7,7 +7,7 @@
 #include <sstream>
 #include <cstring>
 
-// FIXME: introduce common config format for SONiC
+// TODO: introduce common config format for SONiC
 std::shared_ptr<PortMap> PortMapParser::parsePortMap(
         _In_ const std::string& portMapFile)
 {