commit bddad07ef1c387954281d20550b7b16ad84c9032
author Yichi Chen <yichichen@google.com> Mon Mar 23 12:50:12 2020 +0800
committer Gerrit - the friendly Code Review server <code-review@localhost> Tue Apr 14 08:21:22 2020 -0700
tree 2f1a5298b28a0a5a56f02dc7bd971c43c9d32c91
parent 8b2c335b17c39bee036a9f6b964dd95e5f37036e

gralloc: Refactor gralloc4.0 for some concerns in code review

There were some concerns raised in the intern code review process. Since
Qualcomm has merged their Gralloc4.0 changes into their codebase, the
patch created another standlane change to address the concerns within
the code review.

Bug: 141596968
Test: VtsHalGraphicsMapperV4_0TargetTest
Change-Id: I33905f5f1198089a1e3fe16e1fd46a8f614e8679
(cherry picked from commit 3dfff2a5af39b14422ede072e3e9d95596e5fc3f)

libqdmetadata/qdMetaData.cpp

diff --git a/libqdmetadata/qdMetaData.cpp b/libqdmetadata/qdMetaData.cpp
index 3577586..a3141f0 100644
--- a/libqdmetadata/qdMetaData.cpp
+++ b/libqdmetadata/qdMetaData.cpp
@@ -55,7 +55,10 @@
   } else if (in.colorPrimaries == ColorPrimaries_BT709_5) {
     *out = ITU_R_709;
   } else {
-    ALOGE("Cannot convert ColorMetaData to ColorSpace_t");
+    ALOGE(
+        "Cannot convert ColorMetaData to ColorSpace_t. "
+        "Primaries = %d, Range = %d",
+        in.colorPrimaries, in.range);
     return -1;
   }
 
@@ -86,7 +89,7 @@
       out->range = Range_Full;
       break;
     default:
-      ALOGE("Cannot convert ColorSpace_t to ColorMetaData");
+      ALOGE("Cannot convert ColorSpace_t %d to ColorMetaData", in);
       return -1;
       break;
   }
@@ -180,8 +183,9 @@
           ColorMetaData color = {};
           if (!colorSpaceToColorMetadata(*((ColorSpace_t *)param), &color)) {
             data->color = color;
+            break;
           }
-          break;
+          return -EINVAL;
         }
         case MAP_SECURE_BUFFER:
             data->mapSecureBuffer = *((int32_t *)param);
@@ -335,8 +339,8 @@
               ColorSpace_t color_space;
               if (!colorMetaDataToColorSpace(data->color, &color_space)) {
                 *((ColorSpace_t *)param) = color_space;
-              }
                 ret = 0;
+              }
             }
             break;
         case GET_MAP_SECURE_BUFFER: