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

Commit c6ad7d76 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "NuPlayer::GenericSource::onPrepareAsync():Change lock order mLock and... 

Merge "NuPlayer::GenericSource::onPrepareAsync():Change lock order mLock and mDisconnect lock order interleaving, which has potential risk of deadlock if other code locks/unlocks them in a different order." into qt-dev
parents 6ce715b6 932f63ef

media/libmediaplayerservice/nuplayer/GenericSource.cpp

+1 −1
Original line number Diff line number Diff line
@@ -404,8 +404,8 @@ void NuPlayer::GenericSource::onPrepareAsync() { 
            sp<DataSource> dataSource = DataSourceFactory::CreateFromURI(

                   mHTTPService, uri, &mUriHeaders, &contentType,

                   static_cast<HTTPBase *>(mHttpSource.get()));

            mLock.lock();

            mDisconnectLock.lock();

            mLock.lock();

            if (!mDisconnected) {

                mDataSource = dataSource;

            }