commit bbdb1f6f5d7be1378395299c77d088b735e47edb
author Emilian Peev <epeev@google.com> Mon Feb 13 15:26:00 2017 +0000
committer Emilian Peev <epeev@google.com> Tue Feb 14 09:30:35 2017 +0000
tree df85322fec738aa3a5a974f89982d36c8a065ad3
parent 60536d8e9f63f070e2d1269374ddb8f61cd78b23

Camera: Fix ION buffer leaks

Native buffer handles should always get closed
and deleted during buffer release.

BUG: 35338606
Test: Manual using application
Change-Id: I5d7a4b02a28b29b872b66843d623166f04207c2e

camera/device/3.2/default/CameraDeviceSession.cpp

diff --git a/camera/device/3.2/default/CameraDeviceSession.cpp b/camera/device/3.2/default/CameraDeviceSession.cpp
index b47a220..76310ef 100644
--- a/camera/device/3.2/default/CameraDeviceSession.cpp
+++ b/camera/device/3.2/default/CameraDeviceSession.cpp
@@ -194,9 +194,9 @@
             mRelease(mDevice, handle);
         } else {
             mModule->unregisterBuffer(mModule, handle);
-            native_handle_close(handle);
-            native_handle_delete(const_cast<native_handle_t*>(handle));
         }
+        native_handle_close(handle);
+        native_handle_delete(const_cast<native_handle_t*>(handle));
     }
 
     // gralloc1