mirror of
https://github.com/tuxbox-fork-migrations/recycled-ni-libstb-hal.git
synced 2025-08-27 07:22:44 +02:00
fix memleak , free unused track.aacbuf
Origin commit data
------------------
Branch: master
Commit: 77011582f7
Author: Jacek Jendrzej <overx300@gmail.com>
Date: 2019-02-22 (Fri, 22 Feb 2019)
------------------
No further description and justification available within origin commit message!
------------------
This commit was generated by Migit
This commit is contained in:
@@ -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);
|
ffmpeg_printf(1, "cAVIdx[%d]: MANAGER_ADD track AUDIO\n", cAVIdx);
|
||||||
if (context->manager->audio->Command(context, MANAGER_ADD, &track) < 0)
|
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? */
|
/* konfetti: fixme: is this a reason to return with error? */
|
||||||
ffmpeg_err("failed to add track %d\n", n);
|
ffmpeg_err("failed to add track %d\n", n);
|
||||||
}
|
}
|
||||||
|
@@ -85,6 +85,10 @@ static int ManagerAdd(Context_t *context, Track_t track)
|
|||||||
if (Tracks[i].Id == track.Id)
|
if (Tracks[i].Id == track.Id)
|
||||||
{
|
{
|
||||||
Tracks[i].pending = 0;
|
Tracks[i].pending = 0;
|
||||||
|
if(track.aacbuf){
|
||||||
|
free(track.aacbuf);
|
||||||
|
track.aacbuf = NULL;
|
||||||
|
}
|
||||||
return cERR_AUDIO_MGR_NO_ERROR;
|
return cERR_AUDIO_MGR_NO_ERROR;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user