commit f7249f811f094cc3472b5739350c22fb6010cfc3
author Greg Kaiser <gkaiser@google.com> Tue Sep 21 07:10:12 2021 -0700
committer Greg Kaiser <gkaiser@google.com> Tue Sep 21 14:11:55 2021 +0000
tree c25e8cff84c0f87cb99943edb14225bf690f7240
parent 05a9b71cf506aed8911b44aa65b1cdafac75b38e

Spatializer: Avoid null pointer dereference

Bug: 188502620
Test: TreeHugger
Change-Id: Ib8f0a239e2271981febd9a11da20d7631a9daec3

services/audiopolicy/service/Spatializer.cpp

diff --git a/services/audiopolicy/service/Spatializer.cpp b/services/audiopolicy/service/Spatializer.cpp
index 0ecfad4..0773f27 100644
--- a/services/audiopolicy/service/Spatializer.cpp
+++ b/services/audiopolicy/service/Spatializer.cpp
@@ -536,7 +536,8 @@
 }
 
 Status Spatializer::getParameter(int key, std::vector<unsigned char> *value) {
-    ALOGV("%s key %d value size %d", __func__, key, (int)value->size());
+    ALOGV("%s key %d value size %d", __func__, key,
+          (value != nullptr ? (int)value->size() : -1));
     if (value == nullptr) {
         binderStatusFromStatusT(BAD_VALUE);
     }