diff options
author | 2021-05-05 23:36:46 +0000 | |
---|---|---|
committer | 2021-05-05 23:41:04 +0000 | |
commit | b6a1e7eb34980400bdbabcab756c5a244b1f525b (patch) | |
tree | 4f681c34a0ecc1747f15d979c449c462c5686638 /libs/binder/RpcState.cpp | |
parent | e8393349fe438e65babd434f0a37c976ef0ff3ed (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.cpp | 2 |
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; } |