Re: [FFmpeg-devel] [PATCH 1/2] fftools/ffmpeg: fix memory leak issue.

2018-06-14 Thread myp...@gmail.com
On Thu, Jun 14, 2018 at 9:44 PM Nicolas George wrote: > > Jun Zhao (2018-06-13): > > need to call av_bprint_finalize to free the memory source to match > > av_bprint_init. > > As you may have seen, in this case too, the buffer is never allocated > dynamically. > > The av_bprint_finalize() already

Re: [FFmpeg-devel] [PATCH 1/2] fftools/ffmpeg: fix memory leak issue.

2018-06-14 Thread Nicolas George
Jun Zhao (2018-06-13): > need to call av_bprint_finalize to free the memory source to match > av_bprint_init. As you may have seen, in this case too, the buffer is never allocated dynamically. The av_bprint_finalize() already present is thus useless (I think the mistake is mine), but not harmful

[FFmpeg-devel] [PATCH 1/2] fftools/ffmpeg: fix memory leak issue.

2018-06-13 Thread Jun Zhao
need to call av_bprint_finalize to free the memory source to match av_bprint_init. Signed-off-by: Jun Zhao --- fftools/ffmpeg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index d4ac690..cbe89a9 100644 --- a/fftools/ffmpeg.c +++ b/fftoo