commit 5e30146515bf3c6fa9632bf2ee0aef288cfe180f
author Andrei Homescu <ahomescu@google.com> Thu May 12 00:52:34 2022 +0000
committer Andrei Homescu <ahomescu@google.com> Fri Jun 10 00:23:51 2022 +0000
tree 5db109b3e239b87144c8ca0a11d0055b2039b14c
parent 86124ca8d2983bd3eefb2e48d1fcfd6d7f3df70f

libbinder: clear RpcTransportCtx for new sessions in RpcServer

RpcServer previously created new RpcSession instances
using the default transport. There is currently no code
in RpcSession that uses that transport for server sessions,
so this patch initializes the transport to nullptr instead.

Bug: 224644083
Test: atest binderRpcTest
Change-Id: Ie9fa8c9c758821614fb161b360a9935955e7e62b

libs/binder/RpcServer.cpp

diff --git a/libs/binder/RpcServer.cpp b/libs/binder/RpcServer.cpp
index 05e0f37..528341e 100644
--- a/libs/binder/RpcServer.cpp
+++ b/libs/binder/RpcServer.cpp
@@ -392,7 +392,7 @@
                 }
             } while (server->mSessions.end() != server->mSessions.find(sessionId));
 
-            session = RpcSession::make();
+            session = sp<RpcSession>::make(nullptr);
             session->setMaxIncomingThreads(server->mMaxThreads);
             if (!session->setProtocolVersion(protocolVersion)) return;