fix: api wrapper so detection with pyroscope names (#91)

Author: korniltsev

profiler/src/ProfilerEngine/Datadog.Profiler.Native/OpSysTools.cpp

@@ -430,20 +430,16 @@ bool OpSysTools::IsSafeToStartProfiler(double coresThreshold, double& cpuLimit)
     // For linux, we check that the wrapper library is loaded and the default `dl_iterate_phdr` is
     // the one provided by our library.
 
-    std::string wrapperLibraryName = "Datadog.Linux.ApiWrapper.x64.so";
+    const std::string wrapperLibraryName = "Datadog.Linux.ApiWrapper.x64.so";
+    const std::string wrapperLibraryNamePyroscope = "Pyroscope.Linux.ApiWrapper.x64.so";
     const std::string customFnName = "dl_iterate_phdr";
     auto* dlIteratePhdr = reinterpret_cast<void*>(::dl_iterate_phdr);
 
     Dl_info info;
     auto res = dladdr(dlIteratePhdr, &info);
     if (res == 0 || info.dli_fname == nullptr)
     {
-        wrapperLibraryName = "Pyroscope.Linux.ApiWrapper.x64.so";
-        res = dladdr(dlIteratePhdr, &info);
-    }
-    if (res == 0 || info.dli_fname == nullptr)
-    {
-        Log::Warn("Profiling is disabled: Unable to check if the library '", wrapperLibraryName, "'",
+        Log::Warn("Profiling is disabled: Unable to check if the library '", wrapperLibraryNamePyroscope, "'",
                   " is correctly loaded and/or the function '", customFnName, "' is correctly wrapped.",
                   "Please contact the support for help with the following details:\n",
                   "Call to dladdr: ", res, "\n",
@@ -453,11 +449,12 @@ bool OpSysTools::IsSafeToStartProfiler(double coresThreshold, double& cpuLimit)
 
     auto sharedObjectPath = fs::path(info.dli_fname);
 
-    if (sharedObjectPath.filename() != wrapperLibraryName)
+    if (sharedObjectPath.filename() != wrapperLibraryName
+        && sharedObjectPath.filename() != wrapperLibraryNamePyroscope )
     {
         // We assume that the profiler library is in the same folder as the wrapper library
         auto currentModulePath = fs::path(shared::GetCurrentModuleFileName());
-        auto wrapperLibrary = currentModulePath.parent_path() / wrapperLibraryName;
+        auto wrapperLibrary = currentModulePath.parent_path() / wrapperLibraryNamePyroscope;
         auto wrapperLibraryPath = wrapperLibrary.string();
 
         // Check if process is running is a secure-execution mode
@@ -466,7 +463,7 @@ bool OpSysTools::IsSafeToStartProfiler(double coresThreshold, double& cpuLimit)
         // Get LD_PRELOAD env var content
         auto envVarValue = shared::GetEnvironmentValue(WStr("LD_PRELOAD"));
 
-        Log::Warn("Profiling is disabled: It appears the wrapper library '", wrapperLibraryName, "' is not correctly loaded.\n",
+        Log::Warn("Profiling is disabled: It appears the wrapper library '", wrapperLibraryNamePyroscope, "' is not correctly loaded.\n",
                   "Possible reason(s):\n",
                   "* The LD_PRELOAD environment variable might not contain the path '", wrapperLibraryPath, "'. Try adding ",
                   "'", wrapperLibraryPath, "' to LD_PRELOAD environment variable.\n",