| commit | 16724c2c8a4a8425f160f45053bc646d28d80a64 | [log] [tgz] |
|---|---|---|
| author | Ady Abraham <adyabr@google.com> | Wed Apr 12 02:15:50 2023 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Wed Apr 12 02:15:50 2023 +0000 |
| tree | ee2967602f4da69a11ffc2e186f406059d240c5c | |
| parent | 748cecb7efcee3e7c0d0929444abe1abe0095cfb [diff] | |
| parent | f4bd36b72dfde8d67cf62d43d56b8ff27e15e2cc [diff] |
SurfaceComposerClient: add a range check in setAlpha am: f4bd36b72d Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/22434167 Change-Id: I6ab336bed33a26b1bb9fc855ffe43ed1b0ceadc5 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/libs/gui/SurfaceComposerClient.cpp b/libs/gui/SurfaceComposerClient.cpp index 0f5192d..7001339 100644 --- a/libs/gui/SurfaceComposerClient.cpp +++ b/libs/gui/SurfaceComposerClient.cpp
@@ -1274,6 +1274,12 @@ mStatus = BAD_INDEX; return *this; } + if (alpha < 0.0f || alpha > 1.0f) { + ALOGE("SurfaceComposerClient::Transaction::setAlpha: invalid alpha %f", alpha); + mStatus = BAD_VALUE; + return *this; + + } s->what |= layer_state_t::eAlphaChanged; s->alpha = alpha;