summaryrefslogtreecommitdiffstats
path: root/media
diff options
context:
space:
mode:
authorAndy McFadden <fadden@android.com>2013-11-08 07:30:05 -0800
committerAndroid Git Automerger <android-git-automerger@android.com>2013-11-08 07:30:05 -0800
commit8c8c9fb742f678ca34f78fff05274690ba6b00a5 (patch)
tree98c1b103729a9bce397f35fb8b05efe8ea570ea9 /media
parent7bf17159f233111a21b293cde8e227231353cd75 (diff)
parent2e496677be678f5201d0e6c3712c664dd60502d9 (diff)
downloadframeworks_av-8c8c9fb742f678ca34f78fff05274690ba6b00a5.zip
frameworks_av-8c8c9fb742f678ca34f78fff05274690ba6b00a5.tar.gz
frameworks_av-8c8c9fb742f678ca34f78fff05274690ba6b00a5.tar.bz2
am 2e496677: am 5572b3af: Replace assert with error message
* commit '2e496677be678f5201d0e6c3712c664dd60502d9': Replace assert with error message
Diffstat (limited to 'media')
-rw-r--r--media/libstagefright/omx/GraphicBufferSource.cpp7
1 files changed, 6 insertions, 1 deletions
diff --git a/media/libstagefright/omx/GraphicBufferSource.cpp b/media/libstagefright/omx/GraphicBufferSource.cpp
index f295e5a..7d45386 100644
--- a/media/libstagefright/omx/GraphicBufferSource.cpp
+++ b/media/libstagefright/omx/GraphicBufferSource.cpp
@@ -213,7 +213,12 @@ void GraphicBufferSource::codecBufferEmptied(OMX_BUFFERHEADERTYPE* header) {
// see if the GraphicBuffer reference was null, which should only ever
// happen for EOS.
if (codecBuffer.mGraphicBuffer == NULL) {
- CHECK(mEndOfStream && mEndOfStreamSent);
+ if (!(mEndOfStream && mEndOfStreamSent)) {
+ // This can happen when broken code sends us the same buffer
+ // twice in a row.
+ ALOGE("ERROR: codecBufferEmptied on non-EOS null buffer "
+ "(buffer emptied twice?)");
+ }
// No GraphicBuffer to deal with, no additional input or output is
// expected, so just return.
return;