commit d146d6da82c6eb95e61942f341eb0d7a42f55327
author Roman Stratiienko <r.stratiienko@gmail.com> Sun Oct 04 23:56:46 2020 +0300
committer Roman Stratiienko <r.stratiienko@gmail.com> Mon Oct 05 00:01:15 2020 +0300
tree 45b1aa1662d153c2c04a5a928f44a847f7515ae8
parent 27d2ed670bb343669e1d790710fb82e9ec900e85

drm_hwcomposer: Check parameter range in SetColorMode()

Fix GraphicsComposerHidlTest#SetColorModeBadParameter VTS test.

Fixes: 27d2ed670bb34 ("drm_hwcomposer: Fix SetColorMode() and SetColorModeWithIntent()")
Signed-off-by: Roman Stratiienko <r.stratiienko@gmail.com>

DrmHwcTwo.cpp

diff --git a/DrmHwcTwo.cpp b/DrmHwcTwo.cpp
index 38ca663..5b9b5df 100644
--- a/DrmHwcTwo.cpp
+++ b/DrmHwcTwo.cpp
@@ -782,6 +782,9 @@
 HWC2::Error DrmHwcTwo::HwcDisplay::SetColorMode(int32_t mode) {
   supported(__func__);
 
+  if (mode < HAL_COLOR_MODE_NATIVE || mode > HAL_COLOR_MODE_BT2100_HLG)
+    return HWC2::Error::BadParameter;
+
   if (mode != HAL_COLOR_MODE_NATIVE)
     return HWC2::Error::Unsupported;