commit 280bc1fcc653357a4fbfcc8ea98acdd8acfa3d69
author Treehugger Robot <treehugger-gerrit@google.com> Tue Mar 23 21:51:23 2021 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Tue Mar 23 21:51:23 2021 +0000
tree 9c1ef4854c5d1f97b93136d75ead3625b7a5188d
parent 754a3f7d356bf057456469b46ca747e06869a81a
parent 2bb81b0c10589549eb88b29ee851975d0dd59ccf

Merge "nativewindow: report errors on error to allocate a handle" am: 04a1d0ddfe am: 2bb81b0c10

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1555466

Change-Id: I2af80fbe004bce834aa6b3f4bf802c52b70a9f69

libs/nativewindow/AHardwareBuffer.cpp

diff --git a/libs/nativewindow/AHardwareBuffer.cpp b/libs/nativewindow/AHardwareBuffer.cpp
index 1ec73ce..3030068 100644
--- a/libs/nativewindow/AHardwareBuffer.cpp
+++ b/libs/nativewindow/AHardwareBuffer.cpp
@@ -51,13 +51,13 @@
             std::string("AHardwareBuffer pid [") + std::to_string(getpid()) + "]"));
 
     status_t err = gbuffer->initCheck();
-    if (err != 0 || gbuffer->handle == 0) {
+    if (err != 0 || gbuffer->handle == nullptr) {
         if (err == NO_MEMORY) {
             GraphicBuffer::dumpAllocationsToSystemLog();
         }
         ALOGE("GraphicBuffer(w=%u, h=%u, lc=%u) failed (%s), handle=%p",
                 desc->width, desc->height, desc->layers, strerror(-err), gbuffer->handle);
-        return err;
+        return err == 0 ? UNKNOWN_ERROR : err;
     }
 
     *outBuffer = AHardwareBuffer_from_GraphicBuffer(gbuffer.get());