commit 84df28c28a88f2c16dce70c4bd35cae9238044b6
author Jayant Chowdhary <jchowdhary@google.com> Wed May 26 22:32:21 2021 -0700
committer Jayant Chowdhary <jchowdhary@google.com> Wed May 26 22:32:22 2021 -0700
tree f353c54e2d6e7f9cf30f8e105532a911660114cb
parent 2a8c18b4576fda11a68fe03d189f02b3f61ccac5

camera: Fix sensor pixel modes unparcelling in OutputConfiguration.

Also fix a corner case in checkAndOverrideSensorPixelModes

Bug: 184396641

Test: atest RobustnessTest.java#testConfigureInvalidSensorPixelModes

Change-Id: I990a8e4c16a6442a146b1d6c867d4e8132954a70
Signed-off-by: Jayant Chowdhary <jchowdhary@google.com>

camera/camera2/OutputConfiguration.cpp

diff --git a/camera/camera2/OutputConfiguration.cpp b/camera/camera2/OutputConfiguration.cpp
index d6642f3..2bccd87 100644
--- a/camera/camera2/OutputConfiguration.cpp
+++ b/camera/camera2/OutputConfiguration.cpp
@@ -161,7 +161,7 @@
     }
 
     std::vector<int32_t> sensorPixelModesUsed;
-    if ((err = parcel->readParcelableVector(&sensorPixelModesUsed)) != OK) {
+    if ((err = parcel->readInt32Vector(&sensorPixelModesUsed)) != OK) {
         ALOGE("%s: Failed to read sensor pixel mode(s) from parcel", __FUNCTION__);
         return err;
     }