Merge pull request #748 from intel/push-2024-05-22

Push 2024 05 22

Author: rdementi

src/cpucounters.cpp

@@ -4432,11 +4432,7 @@ std::string PCM::getCPUFamilyModelString(const uint32 cpu_family_, const uint32
 {
     char buffer[sizeof(int)*4*3+6];
     std::fill(buffer, buffer + sizeof(buffer), 0);
-#ifdef _MSC_VER
-    sprintf_s(buffer,sizeof(buffer),"GenuineIntel-%d-%2X-%X", cpu_family_, cpu_model_, cpu_stepping_);
-#else
-    snprintf(buffer,sizeof(buffer),"GenuineIntel-%d-%2X-%X", cpu_family_, cpu_model_, cpu_stepping_);
-#endif
+    std::snprintf(buffer,sizeof(buffer),"GenuineIntel-%d-%2X-%X", cpu_family_, cpu_model_, cpu_stepping_);
     std::string result(buffer);
     return result;
 }

src/pcm-accel.cpp

@@ -4,7 +4,6 @@
 
 #include "pcm-accel-common.h"
 #ifdef _MSC_VER
-#pragma warning(disable : 4996) // for sprintf
 #include <windows.h>
 #include "windows/windriver.h"
 #else

src/pcm-iio.cpp

@@ -7,7 +7,6 @@
 #include "cpucounters.h"
 
 #ifdef _MSC_VER
-    #pragma warning(disable : 4996) // for sprintf
     #include <windows.h>
     #include "windows/windriver.h"
 #else

src/pcm-latency.cpp

@@ -4,7 +4,6 @@
 // written by Subhiksha Ravisundar
 #include "cpucounters.h"
 #ifdef _MSC_VER
-#pragma warning(disable : 4996) // for sprintf
 #include <windows.h>
 #include "windows/windriver.h"
 #else

src/pcm-lspci.cpp

@@ -4,7 +4,6 @@
 // written by Patrick Lu
 #include "cpucounters.h"
 #ifdef _MSC_VER
-#pragma warning(disable : 4996) // for sprintf
 #include <windows.h>
 #include "windows/windriver.h"
 #else

src/pcm-msr.cpp

@@ -167,8 +167,8 @@ int mainThrows(int argc, char * argv[])
                         uint64 value2 = value;
                         extractBitsPrintHelper(bits, value, dec);
                         char cname[100];
-                        if (core_in_dec) sprintf(cname,"%d",core);
-                        else sprintf(cname,"0x%x",core);
+                        if (core_in_dec) std::snprintf(cname, sizeof(cname), "%d", core);
+                        else std::snprintf(cname, sizeof(cname), "0x%x", core);
                         std::cout << " from MSR " << msr << " on core " << cname << "\n";
                         auto itx = it;
                         itx++;

src/utils.h

@@ -37,6 +37,9 @@
 #include <map>
 #include <unordered_map>
 
+#ifdef __linux__
+#include <unistd.h>
+#endif
 namespace pcm {
     std::string safe_getenv(const char* env);
 }