diff options
| author | 2021-05-24 23:22:08 +0000 | |
|---|---|---|
| committer | 2021-05-25 02:42:07 +0000 | |
| commit | af4ca715bd1820616e04fcc63ad4e64d64df4024 (patch) | |
| tree | c58f18b6142f954877e8486e2430b5cff15363c1 /libs/binder/RpcSession.cpp | |
| parent | f51742734f476e7f9478f7cb56c18b9215ecad43 (diff) | |
binderRpcTest: use waitpid
Actually reap child processes. This gives us stronger guarantees (that
everything can shut down) and it avoids 'kill'.
Bug: 186661301
Test: binderRpcTest
Change-Id: If10f00de845eb8097813b4edbf8e2b8ffdc90c5f
Diffstat (limited to 'libs/binder/RpcSession.cpp')
| -rw-r--r-- | libs/binder/RpcSession.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libs/binder/RpcSession.cpp b/libs/binder/RpcSession.cpp index d5ce324de1..ac7544fde8 100644 --- a/libs/binder/RpcSession.cpp +++ b/libs/binder/RpcSession.cpp @@ -199,7 +199,8 @@ void RpcSession::join(unique_fd client) { state()->getAndExecuteCommand(connection->fd, sp<RpcSession>::fromExisting(this)); if (error != OK) { - ALOGI("Binder connection thread closing w/ status %s", statusToString(error).c_str()); + LOG_RPC_DETAIL("Binder connection thread closing w/ status %s", + statusToString(error).c_str()); break; } } |