From 3492221fcff84291e6b51fb2d9b0bb185a1e5856 Mon Sep 17 00:00:00 2001 From: Liu Changcheng Date: Thu, 6 Apr 2017 11:20:14 +0800 Subject: [PATCH] check send_signal result before further dealing If send_signal failed, debuggerd_trigger_dump should return instead of further dealing Change-Id: I0a1ac4255344d1da7a21917ad559b9de4d5a4fee Signed-off-by: Liu Changcheng Signed-off-by: Jerry Liu --- debuggerd/client/debuggerd_client.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/debuggerd/client/debuggerd_client.cpp b/debuggerd/client/debuggerd_client.cpp index 3b8485362..2be13c611 100644 --- a/debuggerd/client/debuggerd_client.cpp +++ b/debuggerd/client/debuggerd_client.cpp @@ -140,7 +140,9 @@ bool debuggerd_trigger_dump(pid_t pid, unique_fd output_fd, DebuggerdDumpType du } bool backtrace = dump_type == kDebuggerdBacktrace; - send_signal(pid, backtrace); + if (!send_signal(pid, backtrace)) { + return false; + } rc = TEMP_FAILURE_RETRY(recv(set_timeout(sockfd.get()), &response, sizeof(response), MSG_TRUNC)); if (rc == 0) {