From 914f2f3b01a2166710302e7287f82687afabb503 Mon Sep 17 00:00:00 2001 From: zhupu Date: Tue, 15 Jul 2025 14:43:17 +0800 Subject: [PATCH] securec memory func fix Signed-off-by: zhupu --- bundle.json | 6 ------ frameworks/localhandler/dfx_signal_local_handler.cpp | 11 ++--------- 2 files changed, 2 insertions(+), 15 deletions(-) diff --git a/bundle.json b/bundle.json index 3a919fcc6..29e0feb39 100644 --- a/bundle.json +++ b/bundle.json @@ -117,12 +117,6 @@ "header_base": "//base/hiviewdfx/faultloggerd/interfaces/innerkits/signal_handler/include" } }, - { - "name": "//base/hiviewdfx/faultloggerd/interfaces/innerkits/signal_handler:dfxsignalhandler" - }, - { - "name": "//base/hiviewdfx/faultloggerd/interfaces/innerkits/signal_handler:dfxsignalhandler_noflto" - }, { "name": "//base/hiviewdfx/faultloggerd/interfaces/innerkits/stack_printer:libstack_printer", "header": { diff --git a/frameworks/localhandler/dfx_signal_local_handler.cpp b/frameworks/localhandler/dfx_signal_local_handler.cpp index 34637c3de..a368ad823 100644 --- a/frameworks/localhandler/dfx_signal_local_handler.cpp +++ b/frameworks/localhandler/dfx_signal_local_handler.cpp @@ -110,7 +110,6 @@ void DFX_SignalLocalHandler(int sig, siginfo_t *si, void *context) pthread_mutex_lock(&g_signalHandlerMutex); if (sig != SIGALRM) { (void)memset_s(&g_request, sizeof(g_request), 0, sizeof(g_request)); - g_request.type = static_cast(sig); g_request.tid = gettid(); g_request.pid = getpid(); g_request.uid = FAULTLOGGERD_UID; @@ -121,14 +120,8 @@ void DFX_SignalLocalHandler(int sig, siginfo_t *si, void *context) GetThreadNameByTid(g_request.tid, g_request.threadName, sizeof(g_request.threadName)); GetProcessName(g_request.processName, sizeof(g_request.processName)); - int ret = memcpy_s(&(g_request.siginfo), sizeof(siginfo_t), si, sizeof(siginfo_t)); - if (ret < 0) { - DFXLOGE("memcpy_s siginfo fail, ret=%{public}d", ret); - } - ret = memcpy_s(&(g_request.context), sizeof(ucontext_t), context, sizeof(ucontext_t)); - if (ret < 0) { - DFXLOGE("memcpy_s context fail, ret=%{public}d", ret); - } + (void)memcpy_s(&(g_request.siginfo), sizeof(siginfo_t), si, sizeof(siginfo_t)); + (void)memcpy_s(&(g_request.context), sizeof(ucontext_t), context, sizeof(ucontext_t)); } #if defined(__aarch64__) || defined(__loongarch_lp64) DoCrashHandler(&sig); -- Gitee