commit ce5d6f73e866dafc0f2798c48c0043bf975e6431
author huans <huans@google.com> Fri Jun 01 14:19:14 2018 -0700
committer huans <huans@google.com> Fri Jun 01 16:45:14 2018 -0700
tree 71a695a1a57bb7cb0cf6e699cbff49f9d2fd9bea
parent 9a19e8ff16cbda78bb5389efe3ffb9f42d6be938

goldfish hwcomposer should return error if attribute is not provided

Impact emulator only

Bug: 90631175
Test: Manual
Change-Id: I0553892fb20db9d8bd6e25cbe5babfeb476042eb

hwcomposer/hwcomposer.cpp

diff --git a/hwcomposer/hwcomposer.cpp b/hwcomposer/hwcomposer.cpp
index bcd7c30..702527f 100644
--- a/hwcomposer/hwcomposer.cpp
+++ b/hwcomposer/hwcomposer.cpp
@@ -208,6 +208,9 @@
     for (int i = 0; attributes[i] != HWC_DISPLAY_NO_ATTRIBUTE; i++) {
         if (disp == HWC_DISPLAY_PRIMARY) {
             values[i] = hwc_attribute(pdev, attributes[i]);
+            if (values[i] == -EINVAL) {
+                return -EINVAL;
+            }
         } else {
             ALOGE("unknown display type %u", disp);
             return -EINVAL;