commit 49600b10aa5675d4e7e985203d69f252ead13e45
author Devin Moore <devinmoore@google.com> Tue Apr 25 00:17:13 2023 +0000
committer Devin Moore <devinmoore@google.com> Wed Apr 26 16:18:41 2023 +0000
tree 5683e9ea75b0f86474a8e1875fe0ab446f30a31a
parent 8bf29931eabb065027fda7ffa7db294b04772a36

Allow sensors list to be empty

Test: atest VtsHalSensorManagerV1_0TargetTest
Bug: 278013275
Change-Id: I091f57de9570b0ace3a8da76f16fe0e83f0aa624

libs/sensor/SensorManager.cpp

diff --git a/libs/sensor/SensorManager.cpp b/libs/sensor/SensorManager.cpp
index ba190e0..980f8d1 100644
--- a/libs/sensor/SensorManager.cpp
+++ b/libs/sensor/SensorManager.cpp
@@ -176,11 +176,8 @@
 
         mSensors = mSensorServer->getSensorList(mOpPackageName);
         size_t count = mSensors.size();
-        if (count == 0) {
-            ALOGE("Failed to get Sensor list");
-            mSensorServer.clear();
-            return UNKNOWN_ERROR;
-        }
+        // If count is 0, mSensorList will be non-null. This is old
+        // existing behavior and callers expect this.
         mSensorList =
                 static_cast<Sensor const**>(malloc(count * sizeof(Sensor*)));
         LOG_ALWAYS_FATAL_IF(mSensorList == nullptr, "mSensorList NULL");