Re: [FFmpeg-devel] [PATCH] avformat/mov: fix memleak

2020-06-29 Thread Andreas Rheinhardt
Zhao Zhili: > Remove the check on dv_demux since dv_fctx will leak if allocate > dv_demux failed. > --- > libavformat/mov.c | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index adc52de947..f179b6efdd 100644 > --- a/libavform

Re: [FFmpeg-devel] [PATCH] avformat/mov: fix memleak

2020-06-29 Thread Andreas Rheinhardt
Dave Rice: > Hi Andreas, > >> On Jun 27, 2020, at 5:09 PM, Andreas Rheinhardt >> wrote: >> >> Zhao Zhili: >>> Remove the check on dv_demux since dv_fctx will leak if allocate >>> dv_demux failed. >>> --- >>> libavformat/mov.c | 7 +++ >>> 1 file changed, 3 insertions(+), 4 deletions(-) >>> >>

Re: [FFmpeg-devel] [PATCH] avformat/mov: fix memleak

2020-06-28 Thread Dave Rice
Hi Andreas, > On Jun 27, 2020, at 5:09 PM, Andreas Rheinhardt > wrote: > > Zhao Zhili: >> Remove the check on dv_demux since dv_fctx will leak if allocate >> dv_demux failed. >> --- >> libavformat/mov.c | 7 +++ >> 1 file changed, 3 insertions(+), 4 deletions(-) >> >> diff --git a/libavform

Re: [FFmpeg-devel] [PATCH] avformat/mov: fix memleak

2020-06-27 Thread zhilizhao
> On Jun 28, 2020, at 5:09 AM, Andreas Rheinhardt > wrote: > > Zhao Zhili: >> Remove the check on dv_demux since dv_fctx will leak if allocate >> dv_demux failed. >> --- >> libavformat/mov.c | 7 +++ >> 1 file changed, 3 insertions(+), 4 deletions(-) >> >> diff --git a/libavformat/mov.c b/

Re: [FFmpeg-devel] [PATCH] avformat/mov: fix memleak

2020-06-27 Thread Andreas Rheinhardt
Zhao Zhili: > Remove the check on dv_demux since dv_fctx will leak if allocate > dv_demux failed. > --- > libavformat/mov.c | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index adc52de947..f179b6efdd 100644 > --- a/libavform

[FFmpeg-devel] [PATCH] avformat/mov: fix memleak

2020-06-27 Thread Zhao Zhili
Remove the check on dv_demux since dv_fctx will leak if allocate dv_demux failed. --- libavformat/mov.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index adc52de947..f179b6efdd 100644 --- a/libavformat/mov.c +++ b/libavformat/mov