commit f0fd0c8d293c2608395ee240345c5b8ff1a9f2d3
author TreeHugger Robot <treehugger-gerrit@google.com> Fri Jun 12 22:55:06 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Jun 12 22:55:06 2020 +0000
tree 801de92e7a8d39db760c39191d9eab582cad2886
parent 456513b73e70469a896d3ae4c94787ef8d8d25b3
parent 19d51164324653a7312c6d7bc0fd0ac43d581503

Merge "CCodec: Guard against null output->buffers" into rvc-dev am: 19d5116432

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/11850304

Change-Id: I2e2f618e4ac2c7e550b3f5520d855838a23cb636

media/codec2/sfplugin/CCodecBufferChannel.cpp

diff --git a/media/codec2/sfplugin/CCodecBufferChannel.cpp b/media/codec2/sfplugin/CCodecBufferChannel.cpp
index c07f3dc..2cd357b 100644
--- a/media/codec2/sfplugin/CCodecBufferChannel.cpp
+++ b/media/codec2/sfplugin/CCodecBufferChannel.cpp
@@ -618,13 +618,14 @@
 }
 
 void CCodecBufferChannel::feedInputBufferIfAvailableInternal() {
-    if (mInputMetEos ||
-           mOutput.lock()->buffers->hasPending() ||
-           mPipelineWatcher.lock()->pipelineFull()) {
+    if (mInputMetEos || mPipelineWatcher.lock()->pipelineFull()) {
         return;
-    } else {
+    }
+    {
         Mutexed<Output>::Locked output(mOutput);
-        if (!output->buffers || output->buffers->numClientBuffers() >= output->numSlots) {
+        if (!output->buffers ||
+                output->buffers->hasPending() ||
+                output->buffers->numClientBuffers() >= output->numSlots) {
             return;
         }
     }