diff options
| author | Greg Kaiser <gkaiser@google.com> | 2019-01-08 06:29:22 -0800 |
|---|---|---|
| committer | Greg Kaiser <gkaiser@google.com> | 2019-01-08 06:29:22 -0800 |
| commit | 3fa9a59ea323e6b59e1c9292e088071b9e85a4b8 (patch) | |
| tree | a1f37c9629f1f540a7e1e17689b04a8a21f9c87a /debuggerd | |
| parent | 6cc741ec9ac58ca55a3a137c0c8082683d2a0220 (diff) | |
| download | system_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.cpp | 2 |
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; } |
