commit dbfb32e902ab99aa9bc4004bfcac476589e1f430
author Jamie Gennis <jgennis@google.com> Wed Oct 20 15:53:59 2010 -0700
committer Jamie Gennis <jgennis@google.com> Wed Oct 20 15:53:59 2010 -0700
tree 4554daa6e1dfafb6562b039037c72511a7b6fe59
parent e6aaa3da3359cb7c48a9d54376e0880497c22cc8

Fix the non-ANW video decode path.

The BufferInfo::mOwnedByNativeWindow field was not being initialized in
the non-ANativeWindow video decode path.

Change-Id: If2f6249902a08690f6c62ef30791b9979050c0e5

media/libstagefright/OMXCodec.cpp

diff --git a/media/libstagefright/OMXCodec.cpp b/media/libstagefright/OMXCodec.cpp
index 560d0f1..2e368b6 100644
--- a/media/libstagefright/OMXCodec.cpp
+++ b/media/libstagefright/OMXCodec.cpp
@@ -744,7 +744,8 @@
         mQuirks &= ~kOutputBuffersAreUnreadable;
     }
 
-    if (!mIsEncoder
+    if (mNativeWindow != NULL
+        && !mIsEncoder
         && !strncasecmp(mMIME, "video/", 6)
         && !strncmp(mComponentName, "OMX.", 4)) {
         status_t err = initNativeWindow();
@@ -1606,7 +1607,7 @@
 }
 
 status_t OMXCodec::allocateBuffersOnPort(OMX_U32 portIndex) {
-    if (mNativeWindow != 0 && portIndex == kPortIndexOutput) {
+    if (mNativeWindow != NULL && portIndex == kPortIndexOutput) {
         return allocateOutputBuffersFromNativeWindow();
     }
 
@@ -1676,6 +1677,7 @@
 
         info.mBuffer = buffer;
         info.mOwnedByComponent = false;
+        info.mOwnedByNativeWindow = false;
         info.mMem = mem;
         info.mMediaBuffer = NULL;