diff options
author | Andy McFadden <fadden@android.com> | 2013-11-08 07:30:05 -0800 |
---|---|---|
committer | Android Git Automerger <android-git-automerger@android.com> | 2013-11-08 07:30:05 -0800 |
commit | 8c8c9fb742f678ca34f78fff05274690ba6b00a5 (patch) | |
tree | 98c1b103729a9bce397f35fb8b05efe8ea570ea9 | |
parent | 7bf17159f233111a21b293cde8e227231353cd75 (diff) | |
parent | 2e496677be678f5201d0e6c3712c664dd60502d9 (diff) | |
download | frameworks_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
-rw-r--r-- | media/libstagefright/omx/GraphicBufferSource.cpp | 7 |
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; |