Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b782b89d authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "libbinder_ndk: avoid BpRefBase" am: be9d2d41 am: 0fb2ccb4 

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



Change-Id: I1c5fa08aeeb2151a7bd4d19471de718f24c63f8e
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents d07469c8 0fb2ccb4

libs/binder/ndk/ibinder.cpp

+4 −16
Original line number Diff line number Diff line
@@ -64,6 +64,9 @@ struct Value { 
    wp<ABpBinder> binder;

};

void clean(const void* id, void* obj, void* cookie) {

    // be weary of leaks!

    // LOG(INFO) << "Deleting an ABpBinder";



    CHECK(id == kId) << id << " " << obj << " " << cookie;



    delete static_cast<Value*>(obj);
@@ -239,26 +242,11 @@ status_t ABBinder::onTransact(transaction_code_t code, const Parcel& data, Parce 
}



ABpBinder::ABpBinder(const ::android::sp<::android::IBinder>& binder)

    : AIBinder(nullptr /*clazz*/), BpRefBase(binder) {

    : AIBinder(nullptr /*clazz*/), mRemote(binder) {

    CHECK(binder != nullptr);

}

ABpBinder::~ABpBinder() {}



void ABpBinder::onLastStrongRef(const void* id) {

    // Since ABpBinder is OBJECT_LIFETIME_WEAK, we must remove this weak reference in order for

    // the ABpBinder to be deleted. Even though we have no more references on the ABpBinder

    // (BpRefBase), the remote object may still exist (for instance, if we

    // receive it from another process, before the ABpBinder is attached).



    ABpBinderTag::Value* value =

            static_cast<ABpBinderTag::Value*>(remote()->findObject(ABpBinderTag::kId));

    CHECK_NE(nullptr, value) << "ABpBinder must always be attached";



    remote()->withLock([&]() { value->binder = nullptr; });



    BpRefBase::onLastStrongRef(id);

}



sp<AIBinder> ABpBinder::lookupOrCreateFromBinder(const ::android::sp<::android::IBinder>& binder) {

    if (binder == nullptr) {

        return nullptr;

libs/binder/ndk/ibinder_internal.h

+3 −4
Original line number Diff line number Diff line
@@ -91,7 +91,7 @@ struct ABBinder : public AIBinder, public ::android::BBinder { 


// This binder object may be remote or local (even though it is 'Bp'). The implication if it is

// local is that it is an IBinder object created outside of the domain of libbinder_ndk.

struct ABpBinder : public AIBinder, public ::android::BpRefBase {

struct ABpBinder : public AIBinder {

    // Looks up to see if this object has or is an existing ABBinder or ABpBinder object, otherwise

    // it creates an ABpBinder object.

    static ::android::sp<AIBinder> lookupOrCreateFromBinder(
@@ -99,14 +99,13 @@ struct ABpBinder : public AIBinder, public ::android::BpRefBase { 


    virtual ~ABpBinder();



    void onLastStrongRef(const void* id) override;



    ::android::sp<::android::IBinder> getBinder() override { return remote(); }

    ::android::sp<::android::IBinder> getBinder() override { return mRemote; }

    ABpBinder* asABpBinder() override { return this; }



   private:

    friend android::sp<ABpBinder>;

    explicit ABpBinder(const ::android::sp<::android::IBinder>& binder);

    ::android::sp<::android::IBinder> mRemote;

};



struct AIBinder_Class {