commit 2b4f604238a77d61cfc7f398f885974d08435b23
author Baldev Sahu <bsahu@codeaurora.org> Tue Sep 27 12:29:29 2016 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon Oct 24 03:51:25 2016 -0700
tree 4bcb60b00f3d3c1fd761f5c9efc2ec184e7922a0
parent 82253a477eaca34f19cb05dac0e2fdf7f52bb78b

qdMetadata: Fix potential memory leak in copyMetaData

- Fix potential memory leak which can happen due to
  change "Iba741215e524254b88f88058052fb6408e1f4a36"

Change-Id: I4e0fdee0a69c309b9fc06b1e7df83a0045159777

libqdutils/qdMetaData.cpp

diff --git a/libqdutils/qdMetaData.cpp b/libqdutils/qdMetaData.cpp
index 51dfa82..f62653a 100644
--- a/libqdutils/qdMetaData.cpp
+++ b/libqdutils/qdMetaData.cpp
@@ -233,6 +233,9 @@
         dst->fd_metadata, 0);
     if (base_dst == reinterpret_cast<void*>(MAP_FAILED)) {
         ALOGE("%s: dst mmap() failed: error is %s!", __func__, strerror(errno));
+        if(munmap(base_src, size))
+            ALOGE("%s: failed to unmap src ptr %p, err %d", __func__,
+                                             (void*)base_src, errno);
         return -1;
     }