summaryrefslogtreecommitdiffstats
path: root/debuggerd
diff options
context:
space:
mode:
authorGreg Kaiser <gkaiser@google.com>2019-01-08 06:29:22 -0800
committerGreg Kaiser <gkaiser@google.com>2019-01-08 06:29:22 -0800
commit3fa9a59ea323e6b59e1c9292e088071b9e85a4b8 (patch)
treea1f37c9629f1f540a7e1e17689b04a8a21f9c87a /debuggerd
parent6cc741ec9ac58ca55a3a137c0c8082683d2a0220 (diff)
downloadsystem_core-3fa9a59ea323e6b59e1c9292e088071b9e85a4b8.tar.gz
system_core-3fa9a59ea323e6b59e1c9292e088071b9e85a4b8.tar.bz2
system_core-3fa9a59ea323e6b59e1c9292e088071b9e85a4b8.zip
debuggerd: Fix return type in error case
With our method returning 'bool', a "return -1" is interpretted as 'true'. We change this to an explicit 'false', as desired. Test: TreeHugger Change-Id: I222858b797bc4242a2dc6d4fe81df3d2586d055a
Diffstat (limited to 'debuggerd')
-rw-r--r--debuggerd/client/debuggerd_client.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/debuggerd/client/debuggerd_client.cpp b/debuggerd/client/debuggerd_client.cpp
index 785df99c5..610b96b22 100644
--- a/debuggerd/client/debuggerd_client.cpp
+++ b/debuggerd/client/debuggerd_client.cpp
@@ -72,7 +72,7 @@ bool debuggerd_trigger_dump(pid_t tid, DebuggerdDumpType dump_type, unsigned int
std::string error;
if (!android::procinfo::GetProcessInfo(tid, &procinfo, &error)) {
LOG(ERROR) << "libdebugged_client: failed to get process info: " << error;
- return -1;
+ return false;
}
pid = procinfo.pid;
}