Skip to content

Commit 35fbd0a

Browse files
Treehugger RobotGerrit Code Review
authored andcommitted
Merge "Remove onAsBinder() override for BackendUnifiedServiceManager" into main
2 parents 382fc4d + bee6be1 commit 35fbd0a

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

libs/binder/BackendUnifiedServiceManager.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,6 @@ class BackendUnifiedServiceManager : public android::os::BnServiceManager {
5757
return mTheRealServiceManager->getInterfaceDescriptor();
5858
}
5959

60-
IBinder* onAsBinder() override { return IInterface::asBinder(mTheRealServiceManager).get(); }
61-
6260
private:
6361
sp<os::IServiceManager> mTheRealServiceManager;
6462
void toBinderService(const os::Service& in, os::Service* _out);

libs/binder/tests/binderRpcUniversalTests.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,8 @@ TEST_P(BinderRpc, CannotSendRegularBinderOverSocketBinder) {
301301

302302
auto proc = createRpcTestSocketServerProcess({});
303303

304-
sp<IBinder> someRealBinder = IInterface::asBinder(defaultServiceManager());
304+
sp<IBinder> someRealBinder = defaultServiceManager()->getService(String16("activity"));
305+
ASSERT_NE(someRealBinder, nullptr);
305306
sp<IBinder> outBinder;
306307
EXPECT_EQ(INVALID_OPERATION,
307308
proc.rootIface->repeatBinder(someRealBinder, &outBinder).transactionError());

0 commit comments

Comments
 (0)