summaryrefslogtreecommitdiff
path: root/libs/binder/Parcel.cpp
diff options
context:
space:
mode:
author Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> 2024-07-09 02:14:21 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2024-07-09 02:14:21 +0000
commit02705d68a8ceae0ee21ab6629eb154ce6f7aa95f (patch)
treede3d0acb0574ec9ae509a8d244528d41fc5ba415 /libs/binder/Parcel.cpp
parent20f122f9ad974cf0bf6e322bae47cfa93a7d95b0 (diff)
parent923c710a6d62684a017a62a091363b645beaaf82 (diff)
Merge "libbinder: Add log when FDs aren't supported in RpcSession" into main am: f5883db738 am: 923c710a6d
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3156117 Change-Id: I6d42f7acabd898cf6f3dd5fc655d2f0649b0704a Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
Diffstat (limited to 'libs/binder/Parcel.cpp')
-rw-r--r--libs/binder/Parcel.cpp5
1 files changed, 5 insertions, 0 deletions
diff --git a/libs/binder/Parcel.cpp b/libs/binder/Parcel.cpp
index 3f70e8c104..e8fe555ab3 100644
--- a/libs/binder/Parcel.cpp
+++ b/libs/binder/Parcel.cpp
@@ -1585,10 +1585,15 @@ status_t Parcel::writeFileDescriptor(int fd, bool takeOwnership) {
fdVariant = borrowed_fd(fd);
}
if (!mAllowFds) {
+ ALOGE("FDs are not allowed in this parcel. Both the service and the client must set "
+ "the FileDescriptorTransportMode and agree on the support.");
return FDS_NOT_ALLOWED;
}
switch (rpcFields->mSession->getFileDescriptorTransportMode()) {
case RpcSession::FileDescriptorTransportMode::NONE: {
+ ALOGE("FDs are not allowed in this RpcSession. Both the service and the client "
+ "must set "
+ "the FileDescriptorTransportMode and agree on the support.");
return FDS_NOT_ALLOWED;
}
case RpcSession::FileDescriptorTransportMode::UNIX: