summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEmilian Peev <epeev@google.com>2023-01-20 13:50:46 -0800
committerEmilian Peev <epeev@google.com>2023-01-20 13:57:17 -0800
commit7195938fe2a8ed801249d1cf00e8f66539b96ecf (patch)
tree4f39d14355bf84b3814362c26d89539e579aa1a9
parent2f27d9e80c29cea447450be03fcda29d0aa09f27 (diff)
Camera VTS: Fix AIDL status typo
"physicalCameraDeviceStatusChange" returns a default ScopedAStatus that will not include a valid AStatus instance. This will result in binder instabilities once the status is being parceled. Return "ScopedAStatus::ok" instead. Bug: 253993253 Test: adb shell VtsAidlHalCameraProvider_TargetTest Change-Id: I219f608a04aaec0631e3de58a7e0d47a30294e52
-rw-r--r--camera/provider/aidl/vts/camera_aidl_test.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/camera/provider/aidl/vts/camera_aidl_test.cpp b/camera/provider/aidl/vts/camera_aidl_test.cpp
index 2c2f1b2c66..48816ad93c 100644
--- a/camera/provider/aidl/vts/camera_aidl_test.cpp
+++ b/camera/provider/aidl/vts/camera_aidl_test.cpp
@@ -167,7 +167,7 @@ std::vector<std::string> CameraAidlTest::getCameraDeviceNames(
ScopedAStatus physicalCameraDeviceStatusChange(
const std::string&, const std::string&,
::aidl::android::hardware::camera::common::CameraDeviceStatus) override {
- return ndk::ScopedAStatus();
+ return ScopedAStatus::ok();
}
std::vector<std::string> externalCameraDeviceNames;