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

Commit 3b3c0bd4 authored by Steven Moreland's avatar Steven Moreland Committed by Automerger Merge Worker
Browse files

Merge "libbinder: scope down Parcel data in RPC transact" am: 659917d3 am: e4954587 

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

Change-Id: Iad162b9e5a26e59424cbbaa5f7638a912884ba07
parents e9920739 e4954587

libs/binder/RpcState.cpp

+10 −9
Original line number Diff line number Diff line
@@ -498,19 +498,20 @@ status_t RpcState::processTransactInternal(const base::unique_fd& fd, 
        }

    }



    Parcel reply;

    reply.markForRpc(connection);



    if (replyStatus == OK) {

        Parcel data;

        // transaction->data is owned by this function. Parcel borrows this data and

        // only holds onto it for the duration of this function call. Parcel will be

        // deleted before the 'transactionData' object.

        data.ipcSetDataReference(transaction->data,

                                 transactionData.size() - offsetof(RpcWireTransaction, data),

                             nullptr /*object*/, 0 /*objectCount*/, do_nothing_to_transact_data);

                                 nullptr /*object*/, 0 /*objectCount*/,

                                 do_nothing_to_transact_data);

        data.markForRpc(connection);



    Parcel reply;

    reply.markForRpc(connection);



    if (replyStatus == OK) {

        if (target) {

            replyStatus = target->transact(transaction->code, data, &reply, transaction->flags);

        } else {