commit d0e30e1d0e834a6e7922b40dbad6ab53e2648229
author Dongwon Kang <dwkang@google.com> Thu Jul 20 14:40:25 2017 -0700
committer android-build-team Robot <android-build-team-robot@google.com> Wed Aug 23 18:44:01 2017 +0000
tree 9e887387478634725d8b9d717a6cbe512e739708
parent 5665aee83b5d710b2963d3cbeecdeef7222576c0

Fix memory leak in OggExtractor

Test: added a temporal log and run poc
Bug: 63581671
Change-Id: I436a08e54d5e831f9fbdb33c26d15397ce1fbeba
(cherry picked from commit 63079e7c8e12cda4eb124fbe565213d30b9ea34c)

media/libstagefright/OggExtractor.cpp

diff --git a/media/libstagefright/OggExtractor.cpp b/media/libstagefright/OggExtractor.cpp
index e31c37c..810b0d6 100644
--- a/media/libstagefright/OggExtractor.cpp
+++ b/media/libstagefright/OggExtractor.cpp
@@ -578,6 +578,10 @@
             }
             // First two pages are header pages.
             if (err == ERROR_END_OF_STREAM || mCurrentPage.mPageNo > 2) {
+                if (mBuf != NULL) {
+                    mBuf->release();
+                    mBuf = NULL;
+                }
                 break;
             }
             curGranulePosition = mCurrentPage.mGranulePosition;