diff options
author | Santosh Madhava <smadhava@google.com> | 2011-02-27 20:27:20 -0800 |
---|---|---|
committer | Santosh Madhava <smadhava@google.com> | 2011-02-27 20:27:20 -0800 |
commit | 7c8f25f5b81ed2c6520174ead6d0fdf96a313945 (patch) | |
tree | dec9926db493f2ea9ba42af1db52176d5b8a09f4 | |
parent | f272ad210c6834d92b4b4ddaacd069baedf90dcf (diff) | |
download | frameworks_base-7c8f25f5b81ed2c6520174ead6d0fdf96a313945.zip frameworks_base-7c8f25f5b81ed2c6520174ead6d0fdf96a313945.tar.gz frameworks_base-7c8f25f5b81ed2c6520174ead6d0fdf96a313945.tar.bz2 |
Fix for issue 3431967: Memory leak from video editor engine
Change-Id: I610e82f5bdb8ac7b32541301a9fa41bbd8f1459d
-rwxr-xr-x | media/jni/mediaeditor/VideoEditorMain.cpp | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/media/jni/mediaeditor/VideoEditorMain.cpp b/media/jni/mediaeditor/VideoEditorMain.cpp index 23081f8..9fe0266 100755 --- a/media/jni/mediaeditor/VideoEditorMain.cpp +++ b/media/jni/mediaeditor/VideoEditorMain.cpp @@ -1707,12 +1707,19 @@ videoEditor_populateSettings( if (aFramingCtx->FramingYuv != M4OSA_NULL ) { - if (aFramingCtx->FramingYuv->pac_data != M4OSA_NULL) { - M4OSA_free((M4OSA_MemAddr32)aFramingCtx->FramingYuv->pac_data); - aFramingCtx->FramingYuv->pac_data = M4OSA_NULL; + if (aFramingCtx->FramingYuv[0].pac_data != M4OSA_NULL) { + M4OSA_free((M4OSA_MemAddr32)aFramingCtx->FramingYuv[0].pac_data); + aFramingCtx->FramingYuv[0].pac_data = M4OSA_NULL; } - } - if (aFramingCtx->FramingYuv != M4OSA_NULL) { + if (aFramingCtx->FramingYuv[1].pac_data != M4OSA_NULL) { + M4OSA_free((M4OSA_MemAddr32)aFramingCtx->FramingYuv[1].pac_data); + aFramingCtx->FramingYuv[1].pac_data = M4OSA_NULL; + } + if (aFramingCtx->FramingYuv[2].pac_data != M4OSA_NULL) { + M4OSA_free((M4OSA_MemAddr32)aFramingCtx->FramingYuv[2].pac_data); + aFramingCtx->FramingYuv[2].pac_data = M4OSA_NULL; + } + M4OSA_free((M4OSA_MemAddr32)aFramingCtx->FramingYuv); aFramingCtx->FramingYuv = M4OSA_NULL; } |