commit ff930b400c447b047ac12ab8066207a58a54f188
author Ady Abraham <adyabr@google.com> Fri Aug 05 19:04:16 2022 +0000
committer Ady Abraham <adyabr@google.com> Fri Aug 05 19:04:19 2022 +0000
tree 357229dbda0eae629c83c433c1744cbde2e7b886
parent 063f82e8f95ad6f1f4e0bbc07ce3fd22c9e1eb15

SurfaceComposerClient: add a range check in setAlpha

Change-Id: I172321a4d3533e0506bad75a0a91e33fe921c2b7
Test: presubmit
Bug: 238983569

libs/gui/SurfaceComposerClient.cpp

diff --git a/libs/gui/SurfaceComposerClient.cpp b/libs/gui/SurfaceComposerClient.cpp
index 6e8c886..2ecd0c1 100644
--- a/libs/gui/SurfaceComposerClient.cpp
+++ b/libs/gui/SurfaceComposerClient.cpp
@@ -1279,6 +1279,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;