commit 14ec0809e70202dfd9bab6fa662bb4f73c2089d2
author Christopher Ferris <cferris@google.com> Fri Apr 22 13:23:43 2022 -0700
committer Christopher Ferris <cferris@google.com> Fri Apr 22 20:27:56 2022 +0000
tree 38c24540cbe49e3e1e0ff13c47833039fc816e39
parent 5514be66be912d83307405a3bb7f26e80eb84f9b

Close native handle properly.

Bug: 229900304

Test: No leaks from this location.
Change-Id: Ib2618b0a5b55be648d0f12536e84a3a1b355b660

libs/ui/Gralloc4.cpp

diff --git a/libs/ui/Gralloc4.cpp b/libs/ui/Gralloc4.cpp
index 4f950b8..f6ab7b2 100644
--- a/libs/ui/Gralloc4.cpp
+++ b/libs/ui/Gralloc4.cpp
@@ -1245,8 +1245,9 @@
         } else {
             if (importBuffers) {
                 for (uint32_t i = 0; i < bufferCount; i++) {
-                    error = mMapper.importBuffer(makeFromAidl(result.buffers[i]),
-                                                 &outBufferHandles[i]);
+                    auto handle = makeFromAidl(result.buffers[i]);
+                    error = mMapper.importBuffer(handle, &outBufferHandles[i]);
+                    native_handle_delete(handle);
                     if (error != NO_ERROR) {
                         for (uint32_t j = 0; j < i; j++) {
                             mMapper.freeBuffer(outBufferHandles[j]);