commit 764662167fb7330ee2b8118db4ea7772534d3f9e
author Sushil Chauhan <sushilchauhan@codeaurora.org> Tue Jan 05 18:09:22 2016 -0800
committer Sushil Chauhan <sushilchauhan@codeaurora.org> Wed Jan 06 16:17:09 2016 -0800
tree 3fd35077db17fee8dd906a8a9a0376f67b8e5e08
parent 57f2732c793b3f0184958ea7afbb4f08a10c10f2

gralloc: Fix the result of SET_SINGLE_BUFFER_MODE operation

Fix the result value of successful SET_SINGLE_BUFFER_MODE perform
operation.

CRs-Fixed: 958764
Change-Id: Ie84522776008b1f77213be406ec1b866353b265a

libgralloc/mapper.cpp

diff --git a/libgralloc/mapper.cpp b/libgralloc/mapper.cpp
index dcf2ee8..421069f 100644
--- a/libgralloc/mapper.cpp
+++ b/libgralloc/mapper.cpp
@@ -484,10 +484,10 @@
             {
                 private_handle_t* hnd =  va_arg(args, private_handle_t*);
                 uint32_t *enable = va_arg(args, uint32_t*);
-                if (private_handle_t::validate(hnd)) {
-                    return res;
+                if (!private_handle_t::validate(hnd)) {
+                    setMetaData(hnd, SET_SINGLE_BUFFER_MODE, enable);
+                    res = 0;
                 }
-                setMetaData(hnd, SET_SINGLE_BUFFER_MODE, enable);
             } break;
         default:
             break;