Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make sure the OMX callback thread is properly shutdown after the node goes away.

Browse Source 
Change-Id: Ib0b25855b0dfc191e5529193b4cb519f644a8412
This commit is contained in:
Andreas Huber
2010-07-07 08:20:49 -07:00
parent 9406f626b2
commit 6615defd4b
1 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
media/libstagefright/omx/OMX.cpp
View File

@@ -164,6 +164,10 @@ void OMX::binderDied(const wp<IBinder> &the_late_who) {
instance = mLiveNodes.editValueAt(index); instance = mLiveNodes.editValueAt(index);
mLiveNodes.removeItemsAt(index); mLiveNodes.removeItemsAt(index);
index = mDispatchers.indexOfKey(instance->nodeID());
CHECK(index >= 0);
mDispatchers.removeItemsAt(index);
invalidateNodeID_l(instance->nodeID()); invalidateNodeID_l(instance->nodeID());
} }
@@ -240,6 +244,11 @@ status_t OMX::freeNode(node_id node) {
ssize_t index = mLiveNodes.indexOfKey(instance->observer()->asBinder()); ssize_t index = mLiveNodes.indexOfKey(instance->observer()->asBinder());
CHECK(index >= 0); CHECK(index >= 0);
mLiveNodes.removeItemsAt(index); mLiveNodes.removeItemsAt(index);
index = mDispatchers.indexOfKey(node);
CHECK(index >= 0);
mDispatchers.removeItemsAt(index);
instance->observer()->asBinder()->unlinkToDeath(this); instance->observer()->asBinder()->unlinkToDeath(this);
return instance->freeNode(mMaster); return instance->freeNode(mMaster);