commit 7aff26123e69d9e86da9d5c7c6f9430abab71d8f
author Naseer Ahmed <naseer@codeaurora.org> Thu Oct 05 20:39:05 2017 -0400
committer Naseer Ahmed <naseer@codeaurora.org> Thu Oct 12 18:27:08 2017 -0400
tree 2ea03392c067c5bbabc208b70e898e4d62b9d61f
parent 71f5daa4ea77f07ed7807b44f3a8a9a737d2edbe

display: metadata fixes

* Do not log error on invalid metadata fds, such usages are
possible in some scenarios
* Correct usage of metadata API for some cases.

CRs-Fixed: 2122143
Change-Id: Id9cc7f4d27ef1890668f414d92b96cd419b6a066

libqdutils/qdMetaData.cpp

diff --git a/libqdutils/qdMetaData.cpp b/libqdutils/qdMetaData.cpp
index 0837c99..af2adf7 100644
--- a/libqdutils/qdMetaData.cpp
+++ b/libqdutils/qdMetaData.cpp
@@ -44,9 +44,8 @@
         ALOGE("%s: Private handle is invalid - handle:%p", __func__, handle);
         return -1;
     }
-    if (handle->fd_metadata == -1) {
-        ALOGE("%s: Invalid metadata fd - handle:%p fd: %d",
-                __func__, handle, handle->fd_metadata);
+    if (handle->fd_metadata < 0) {
+        // Silently return, metadata cannot be used
         return -1;
     }