Merge "libbinder: fix trusty cpp fd transport modes" into main am: 33da0c2 am: 3ea3bd3

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3547982

Change-Id: Ic37d6529d7060d0bc4982c7b88e5948d8eb34241
Signed-off-by: Automerger Merge Worker <[email protected]>

Author: Treehugger Robot

libs/binder/trusty/include/binder/RpcServerTrusty.h

@@ -97,6 +97,15 @@ class RpcServerTrusty final : public virtual RefBase {
         // By default we use the latest stable version.
         LOG_ALWAYS_FATAL_IF(!rpcServer->setProtocolVersion(RPC_WIRE_PROTOCOL_VERSION));
 
+        // The default behavior in trusty is to allow handles to be passed with tipc IPC.
+        // We add mode NONE so that servers do not reject connections from clients who do
+        // not change their default transport mode.
+        static const std::vector<RpcSession::FileDescriptorTransportMode>
+                TRUSTY_SERVER_SUPPORTED_FD_MODES = {RpcSession::FileDescriptorTransportMode::TRUSTY,
+                                                    RpcSession::FileDescriptorTransportMode::NONE};
+
+        rpcServer->setSupportedFileDescriptorTransportModes(TRUSTY_SERVER_SUPPORTED_FD_MODES);
+
         return rpcServer;
     }
 

libs/binder/trusty/rust/binder_rpc_server_bindgen/cpp/ARpcServerTrusty.cpp

@@ -27,13 +27,6 @@ using android::RpcTransportCtxFactoryTipcTrusty;
 using android::sp;
 using android::wp;
 
-// The default behavior in trusty is to allow handles to be passed with tipc IPC.
-// We add mode NONE so that servers do not reject connections from clients who do
-// not change their default transport mode.
-static const std::vector<RpcSession::FileDescriptorTransportMode> TRUSTY_SERVER_SUPPORTED_FD_MODES =
-        {RpcSession::FileDescriptorTransportMode::TRUSTY,
-         RpcSession::FileDescriptorTransportMode::NONE};
-
 struct ARpcServerTrusty {
     sp<RpcServer> mRpcServer;
 
@@ -60,8 +53,6 @@ ARpcServerTrusty* ARpcServerTrusty_newPerSession(AIBinder* (*cb)(const void*, si
         return nullptr;
     }
 
-    rpcServer->setSupportedFileDescriptorTransportModes(TRUSTY_SERVER_SUPPORTED_FD_MODES);
-
     rpcServer->setPerSessionRootObject(
             [cb, cbArgSp](wp<RpcSession> /*session*/, const void* addrPtr, size_t len) {
                 auto* aib = (*cb)(addrPtr, len, cbArgSp.get());