summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Steven Moreland <smoreland@google.com> 2021-05-11 17:11:40 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2021-05-11 17:11:40 +0000
commit5cf70afccd66c33f8d41a855370eb5e93fc1eef9 (patch)
tree37238286eb20409f9d537aac666025a1b121b75c
parent9c1769322e6ef65fe6a636879fd64080872c9246 (diff)
parentf9df1ab9f27d3206361943899a41531d4b5b8b2f (diff)
Merge "libbinder: RPC - add DOS TODO" am: a416e7b190 am: f9df1ab9f2
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1702013 Change-Id: I50207ddcbc75c29ce48f1607a9b55e3703d9074e
-rw-r--r--libs/binder/RpcServer.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/libs/binder/RpcServer.cpp b/libs/binder/RpcServer.cpp
index a6850ff6aa..51c770e85d 100644
--- a/libs/binder/RpcServer.cpp
+++ b/libs/binder/RpcServer.cpp
@@ -134,7 +134,8 @@ void RpcServer::join() {
}
LOG_RPC_DETAIL("accept4 on fd %d yields fd %d", mServer.get(), clientFd.get());
- // TODO(b/183988761): cannot trust this simple ID
+ // TODO(b/183988761): cannot trust this simple ID, should not block this
+ // thread
LOG_ALWAYS_FATAL_IF(!mAgreedExperimental, "no!");
int32_t id;
if (sizeof(id) != read(clientFd.get(), &id, sizeof(id))) {