| commit | 80ca14e28c9dcb9800cbddba675e4e5ad678e0c8 | [log] [tgz] |
|---|---|---|
| author | TreeHugger Robot <treehugger-gerrit@google.com> | Fri Jun 18 13:56:20 2021 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Fri Jun 18 13:56:20 2021 +0000 |
| tree | 1d41ea4e4f5725e9c0767b77b1ded6b95a84af84 | |
| parent | 241295940e500f5570ad6b368eb0feb76cc1bc58 [diff] | |
| parent | cca7e93111f4c160b7ee99e9a28c624b9e1fd773 [diff] |
Merge "Fix default vibrator HAL to check support on getPrimitiveDuration" into sc-dev am: cca7e93111 Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/14985772 Change-Id: Iae9dd5718297aed34396938794f7e5055ef4460e
diff --git a/vibrator/aidl/default/Vibrator.cpp b/vibrator/aidl/default/Vibrator.cpp index c6682b3..5755ce5 100644 --- a/vibrator/aidl/default/Vibrator.cpp +++ b/vibrator/aidl/default/Vibrator.cpp
@@ -138,6 +138,11 @@ ndk::ScopedAStatus Vibrator::getPrimitiveDuration(CompositePrimitive primitive, int32_t* durationMs) { + std::vector<CompositePrimitive> supported; + getSupportedPrimitives(&supported); + if (std::find(supported.begin(), supported.end(), primitive) == supported.end()) { + return ndk::ScopedAStatus::fromExceptionCode(EX_UNSUPPORTED_OPERATION); + } if (primitive != CompositePrimitive::NOOP) { *durationMs = 100; } else {