commit 4832438c3a31a5316f8b33ccfbe961accd946e2e
author liulvping <liulvping@xiaomi.com> Thu Dec 20 20:34:56 2018 +0800
committer lvping liu <liulvping@xiaomi.com> Wed Jan 02 10:54:57 2019 +0000
tree 2eb0fb5baa39fd9e1a2d0a6b1d65a448c50358c0
parent 99f6957f2e42caeea209d2069174cab24c347a95

fix mmap leak of graphicsstats service

Test: repeat dumpsys graphicsstats and check process maps

Change-Id: Iae2baa7eb7761aefb867644edfaaf8dbe2132c7b
Signed-off-by: liulvping <liulvping@xiaomi.com>

libs/hwui/service/GraphicsStatsService.cpp

diff --git a/libs/hwui/service/GraphicsStatsService.cpp b/libs/hwui/service/GraphicsStatsService.cpp
index 3d50d2d..29e4256 100644
--- a/libs/hwui/service/GraphicsStatsService.cpp
+++ b/libs/hwui/service/GraphicsStatsService.cpp
@@ -139,6 +139,7 @@
     uint32_t file_version = *reinterpret_cast<uint32_t*>(addr);
     if (file_version != sCurrentFileVersion) {
         ALOGW("file_version mismatch! expected %d got %d", sCurrentFileVersion, file_version);
+        munmap(addr, sb.st_size);
         return false;
     }
 
@@ -150,6 +151,7 @@
         ALOGW("Parse failed on '%s' error='%s'", path.c_str(),
               output->InitializationErrorString().c_str());
     }
+    munmap(addr, sb.st_size);
     return success;
 }