From 7f118f5eb5d8dfd69600a8bb3c613056a234f85c Mon Sep 17 00:00:00 2001 From: Praveen Chavan Date: Fri, 13 Sep 2013 15:51:03 -0700 Subject: mm-video: Free frame_parser in component de-init frame_parser is alloc'ed in component init, it should be accordingly freed in de-init rather than when input buffers are freed. author: Leena Winterrowd Bug: 10668205 Change-Id: I2a8dd90b4831f9d9e7d50f5a1093d1a0055d3aad --- mm-video-v4l2/vidc/vdec/src/omx_vdec_msm8974.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/mm-video-v4l2/vidc/vdec/src/omx_vdec_msm8974.cpp b/mm-video-v4l2/vidc/vdec/src/omx_vdec_msm8974.cpp index bb2e6df9..2bc8b595 100755 --- a/mm-video-v4l2/vidc/vdec/src/omx_vdec_msm8974.cpp +++ b/mm-video-v4l2/vidc/vdec/src/omx_vdec_msm8974.cpp @@ -5664,6 +5664,12 @@ OMX_ERRORTYPE omx_vdec::component_deinit(OMX_IN OMX_HANDLETYPE hComp) h264_parser = NULL; } + if (m_frame_parser.mutils) { + DEBUG_PRINT_LOW("\n Free utils parser"); + delete (m_frame_parser.mutils); + m_frame_parser.mutils = NULL; + } + if (m_platform_list) { free(m_platform_list); m_platform_list = NULL; @@ -7187,12 +7193,6 @@ void omx_vdec::free_input_buffer_header() { input_use_buffer = false; if (arbitrary_bytes) { - if (m_frame_parser.mutils) { - DEBUG_PRINT_LOW("\n Free utils parser"); - delete (m_frame_parser.mutils); - m_frame_parser.mutils = NULL; - } - if (m_inp_heap_ptr) { DEBUG_PRINT_LOW("\n Free input Heap Pointer"); free (m_inp_heap_ptr); -- cgit v1.2.3