diff options
author | Emilian Peev <epeev@google.com> | 2023-01-20 13:50:46 -0800 |
---|---|---|
committer | Emilian Peev <epeev@google.com> | 2023-01-20 13:57:17 -0800 |
commit | 7195938fe2a8ed801249d1cf00e8f66539b96ecf (patch) | |
tree | 4f39d14355bf84b3814362c26d89539e579aa1a9 | |
parent | 2f27d9e80c29cea447450be03fcda29d0aa09f27 (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.cpp | 2 |
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; |