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

Commit 4f5c2743 authored by Steven Moreland's avatar Steven Moreland Committed by Automerger Merge Worker
Browse files

Merge "ANDROID: binder: prevent double flushing when free buffer" into sc-dev am: 2870c451 

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

Change-Id: Ief000df5a4fc0b0481cb88e5e84cc8e3d6918e38
parents b5f57fc5 2870c451

libs/binder/IPCThreadState.cpp

+12 −10
Original line number Diff line number Diff line
@@ -489,14 +489,16 @@ void IPCThreadState::flushCommands() 


bool IPCThreadState::flushIfNeeded()

{

    if (mIsLooper || mServingStackPointer != nullptr) {

    if (mIsLooper || mServingStackPointer != nullptr || mIsFlushing) {

        return false;

    }

    mIsFlushing = true;

    // In case this thread is not a looper and is not currently serving a binder transaction,

    // there's no guarantee that this thread will call back into the kernel driver any time

    // soon. Therefore, flush pending commands such as BC_FREE_BUFFER, to prevent them from getting

    // stuck in this thread's out buffer.

    flushCommands();

    mIsFlushing = false;

    return true;

}
@@ -852,10 +854,10 @@ IPCThreadState::IPCThreadState() 
        mWorkSource(kUnsetWorkSource),

        mPropagateWorkSource(false),

        mIsLooper(false),

        mIsFlushing(false),

        mStrictModePolicy(0),

        mLastTransactionBinderFlags(0),

      mCallRestriction(mProcess->mCallRestriction)

{

        mCallRestriction(mProcess->mCallRestriction) {

    pthread_setspecific(gTLS, this);

    clearCaller();

    mIn.setDataCapacity(256);

libs/binder/include/binder/IPCThreadState.h

+1 −0
Original line number Diff line number Diff line
@@ -212,6 +212,7 @@ private: 
            // Whether the work source should be propagated.

            bool                mPropagateWorkSource;

            bool                mIsLooper;

            bool mIsFlushing;

            int32_t             mStrictModePolicy;

            int32_t             mLastTransactionBinderFlags;

            CallRestriction     mCallRestriction;