commit 92dc4546793dad4ce5aa71df17f5ca99286ee037
author Greg Kaiser <gkaiser@google.com> Fri Oct 08 06:58:19 2021 -0700
committer Greg Kaiser <gkaiser@google.com> Fri Oct 08 12:16:05 2021 -0700
tree a1435ea0e4361b1cd00b37e6ca61ff0f1f7067fb
parent 92da43cc21656232d89cd886e504e69bafd39128

CCodec: Fix null check

'dst' is always non-null, by assumption of the function.  We need
to check if "*dst" is null or not.

We also update the interface definition.

Bug: 197719179
Test: TreeHugger
Change-Id: Ic86891f936f4e84241c55674c1aedcb0d7e15483

media/codec2/sfplugin/CCodecBuffers.cpp

diff --git a/media/codec2/sfplugin/CCodecBuffers.cpp b/media/codec2/sfplugin/CCodecBuffers.cpp
index e12b209..d57eb0c 100644
--- a/media/codec2/sfplugin/CCodecBuffers.cpp
+++ b/media/codec2/sfplugin/CCodecBuffers.cpp
@@ -233,7 +233,7 @@
     if (!srcBuffer) {
         return false;
     }
-    if (!dst) {
+    if (!*dst) {
         *dst = new Codec2Buffer(
                 mFormat,
                 new ABuffer(mDataConverter->targetSize(srcBuffer->size())));