From 1801d92c2f75a240d3dd590a0a69bba5088973d7 Mon Sep 17 00:00:00 2001 From: Jacek Jendrzej Date: Fri, 22 Feb 2019 15:23:26 +0100 Subject: [PATCH] fix memleak , free unused track.aacbuf Origin commit data ------------------ Branch: master Commit: https://github.com/neutrino-images/ni-libstb-hal/commit/77011582f7e6de29431197eb376e8f9d91583ed6 Author: Jacek Jendrzej Date: 2019-02-22 (Fri, 22 Feb 2019) ------------------ No further description and justification available within origin commit message! ------------------ This commit was generated by Migit --- libeplayer3-arm/container/container_ffmpeg.c | 4 ++++ libeplayer3-arm/manager/audio.c | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/libeplayer3-arm/container/container_ffmpeg.c b/libeplayer3-arm/container/container_ffmpeg.c index c40e243..074155c 100644 --- a/libeplayer3-arm/container/container_ffmpeg.c +++ b/libeplayer3-arm/container/container_ffmpeg.c @@ -2508,6 +2508,10 @@ int32_t container_ffmpeg_update_tracks(Context_t *context, char *filename, int32 ffmpeg_printf(1, "cAVIdx[%d]: MANAGER_ADD track AUDIO\n", cAVIdx); if (context->manager->audio->Command(context, MANAGER_ADD, &track) < 0) { + if(track.aacbuf){ + free(track.aacbuf); + track.aacbuf = NULL; + } /* konfetti: fixme: is this a reason to return with error? */ ffmpeg_err("failed to add track %d\n", n); } diff --git a/libeplayer3-arm/manager/audio.c b/libeplayer3-arm/manager/audio.c index 0bbb3ed..d2240a0 100644 --- a/libeplayer3-arm/manager/audio.c +++ b/libeplayer3-arm/manager/audio.c @@ -85,6 +85,10 @@ static int ManagerAdd(Context_t *context, Track_t track) if (Tracks[i].Id == track.Id) { Tracks[i].pending = 0; + if(track.aacbuf){ + free(track.aacbuf); + track.aacbuf = NULL; + } return cERR_AUDIO_MGR_NO_ERROR; } }