Re: [FFmpeg-devel] [PATCH] hls: fix leaking avio_opts on hls_read_header error

2016-11-07 Thread Andreas Cadhalpun
On 07.11.2016 16:20, Michael Niedermayer wrote: > On Mon, Nov 07, 2016 at 12:11:25AM +0100, Andreas Cadhalpun wrote: >> Use the hls_close function to reduce code duplication. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/hls.c | 30 ++ >> 1 file changed,

Re: [FFmpeg-devel] [PATCH] hls: fix leaking avio_opts on hls_read_header error

2016-11-07 Thread Michael Niedermayer
On Mon, Nov 07, 2016 at 12:11:25AM +0100, Andreas Cadhalpun wrote: > Use the hls_close function to reduce code duplication. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/hls.c | 30 ++ > 1 file changed, 14 insertions(+), 16 deletions(-) LGTM but please spl

[FFmpeg-devel] [PATCH] hls: fix leaking avio_opts on hls_read_header error

2016-11-06 Thread Andreas Cadhalpun
Use the hls_close function to reduce code duplication. Signed-off-by: Andreas Cadhalpun --- libavformat/hls.c | 30 ++ 1 file changed, 14 insertions(+), 16 deletions(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 2bf86fa..3ae3c7c 100644 --- a/libavformat