Fixed the issue where the Crash Stace Trace of the C++ part was overwritten by the Golang stack when starting the Golang module. (#2171)

* fix crash trace

* add ENTERPRISE switch

* add ENTERPRISE switch

* fix format

* fix comments

Author: linrunqi08

Diff for: core/application/Application.cpp

@@ -125,7 +125,7 @@ void Application::Init() {
         AlarmManager::GetInstance()->SendAlarm(LOGTAIL_CRASH_STACK_ALARM, backTraceStr);
     }
     if (BOOL_FLAG(ilogtail_disable_core)) {
-        InitCrashBackTrace();
+        ResetCrashBackTrace();
     }
 #endif
     // override process related params if designated by user explicitly

Diff for: core/common/CrashBackTraceUtil.cpp

@@ -124,13 +124,14 @@ bool MinidumpCallbackFunc(const wchar_t* dump_path,
 }
 #endif
 
-void InitCrashBackTrace() {
-#if defined(__ANDROID__)
-#elif defined(__linux__)
-    g_crashBackTraceFilePtr = fopen((GetCrashStackFileName()).c_str(), "w");
+void ResetCrashBackTrace() {
+#if defined(__linux__)
     if (g_crashBackTraceFilePtr == NULL) {
-        APSARA_LOG_ERROR(sLogger, ("unable to open stack back trace file", strerror(errno)));
-        return;
+        g_crashBackTraceFilePtr = fopen((GetCrashStackFileName()).c_str(), "w");
+        if (g_crashBackTraceFilePtr == NULL) {
+            APSARA_LOG_ERROR(sLogger, ("unable to open stack back trace file", strerror(errno)));
+            return;
+        }
     }
     signal(SIGSEGV, CrashBackTrace); // SIGSEGV    11       Core Invalid memory reference
     signal(SIGABRT, CrashBackTrace); // SIGABRT     6       Core Abort signal from
@@ -140,9 +141,11 @@ void InitCrashBackTrace() {
     _mkdir("dumps");
     g_handler.reset(new google_breakpad::ExceptionHandler(
         L"dumps\\", FilterCallbackFunc, MinidumpCallbackFunc, NULL, google_breakpad::ExceptionHandler::HANDLER_ALL));
+#elif defined(__ANDROID__)
 #endif
 }
 
+
 std::string GetCrashBackTrace() {
     auto stackFilePath = GetCrashStackFileName();
     FILE* pStackFile = fopen(stackFilePath.c_str(), "rb");

Diff for: core/common/CrashBackTraceUtil.h

@@ -19,8 +19,7 @@
 
 namespace logtail {
 
-void InitCrashBackTrace();
-
+void ResetCrashBackTrace();
 std::string GetCrashBackTrace();
 
 } // namespace logtail

Diff for: core/go_pipeline/LogtailPlugin.cpp

@@ -19,6 +19,7 @@
 #include "app_config/AppConfig.h"
 #include "collection_pipeline/CollectionPipelineManager.h"
 #include "collection_pipeline/queue/SenderQueueManager.h"
+#include "common/CrashBackTraceUtil.h"
 #include "common/DynamicLibHelper.h"
 #include "common/HashUtil.h"
 #include "common/JsonUtil.h"
@@ -39,6 +40,7 @@ DEFINE_FLAG_BOOL(enable_sls_metrics_format, "if enable format metrics in SLS met
 DECLARE_FLAG_STRING(ALIYUN_LOG_FILE_TAGS);
 DECLARE_FLAG_INT32(file_tags_update_interval);
 DECLARE_FLAG_STRING(agent_host_id);
+DECLARE_FLAG_BOOL(ilogtail_disable_core);
 
 using namespace std;
 using namespace logtail;
@@ -477,6 +479,11 @@ bool LogtailPlugin::LoadPluginBase() {
     } else {
         LOG_INFO(sLogger, ("Go plugin system init", "succeeded"));
         mPluginValid = true;
+#ifdef __ENTERPRISE__
+        if (BOOL_FLAG(ilogtail_disable_core)) {
+            ResetCrashBackTrace();
+        }
+#endif
     }
     return mPluginValid;
 }