summaryrefslogtreecommitdiff
path: root/libs/binder/RpcState.cpp
diff options
context:
space:
mode:
author Steven Moreland <smoreland@google.com> 2021-05-05 23:36:46 +0000
committer Steven Moreland <smoreland@google.com> 2021-05-05 23:41:04 +0000
commitb6a1e7eb34980400bdbabcab756c5a244b1f525b (patch)
tree4f681c34a0ecc1747f15d979c449c462c5686638 /libs/binder/RpcState.cpp
parente8393349fe438e65babd434f0a37c976ef0ff3ed (diff)
libbinder: remove RpcState deadlock debug logs
'dump' internally takes a lock, and there were two calls to it here which would actually deadlock. Simply removing them to move fuzzer forward. Bug: 182938024 Test: fuzzer Change-Id: Ic07193955f0794fd1fc11d7e89c65b469a43323c
Diffstat (limited to 'libs/binder/RpcState.cpp')
-rw-r--r--libs/binder/RpcState.cpp2
1 files changed, 0 insertions, 2 deletions
diff --git a/libs/binder/RpcState.cpp b/libs/binder/RpcState.cpp
index 976351c6c4..20fdbfe954 100644
--- a/libs/binder/RpcState.cpp
+++ b/libs/binder/RpcState.cpp
@@ -511,7 +511,6 @@ status_t RpcState::processTransactInternal(const base::unique_fd& fd, const sp<R
auto it = mNodeForAddress.find(addr);
if (it == mNodeForAddress.end()) {
ALOGE("Unknown binder address %s.", addr.toString().c_str());
- dump();
replyStatus = BAD_VALUE;
} else {
target = it->second.binder.promote();
@@ -707,7 +706,6 @@ status_t RpcState::processDecStrong(const base::unique_fd& fd, const RpcWireHead
auto it = mNodeForAddress.find(addr);
if (it == mNodeForAddress.end()) {
ALOGE("Unknown binder address %s for dec strong.", addr.toString().c_str());
- dump();
return OK;
}