Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix a memory leak when http_persistent is 1

2017-12-20 Thread Karthick Jeyapal 



On 12/21/17 9:11 AM, 刘歧 wrote:



On 19 Dec 2017, at 16:49, 刘歧  wrote:


On 19 Dec 2017, at 14:58, Karthick J  wrote:

From: Karthick Jeyapal 

---
libavformat/hlsenc.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
index e3442c3..5ee28ea 100644
--- a/libavformat/hlsenc.c
+++ b/libavformat/hlsenc.c
@@ -1918,6 +1918,8 @@ static int hls_write_trailer(struct AVFormatContext *s)
av_freep(>baseurl);
}

+ff_format_io_close(s, >m3u8_out);
+ff_format_io_close(s, >sub_m3u8_out);
av_freep(>key_basename);
av_freep(>var_streams);
av_freep(>master_m3u8_url);
--
1.9.1




LGTM



Pushed

Thanks!


Thanks





___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix a memory leak when http_persistent is 1

2017-12-20 Thread 刘歧 


> On 19 Dec 2017, at 16:49, 刘歧  wrote:
> 
>> 
>> On 19 Dec 2017, at 14:58, Karthick J  wrote:
>> 
>> From: Karthick Jeyapal 
>> 
>> ---
>> libavformat/hlsenc.c | 2 ++
>> 1 file changed, 2 insertions(+)
>> 
>> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
>> index e3442c3..5ee28ea 100644
>> --- a/libavformat/hlsenc.c
>> +++ b/libavformat/hlsenc.c
>> @@ -1918,6 +1918,8 @@ static int hls_write_trailer(struct AVFormatContext *s)
>>av_freep(>baseurl);
>>}
>> 
>> +ff_format_io_close(s, >m3u8_out);
>> +ff_format_io_close(s, >sub_m3u8_out);
>>av_freep(>key_basename);
>>av_freep(>var_streams);
>>av_freep(>master_m3u8_url);
>> -- 
>> 1.9.1
>> 
>> 
> 
> 
> LGTM
> 
> 

Pushed

Thanks



___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: Fix a memory leak when http_persistent is 1

2017-12-19 Thread 刘歧 

> On 19 Dec 2017, at 14:58, Karthick J  wrote:
> 
> From: Karthick Jeyapal 
> 
> ---
> libavformat/hlsenc.c | 2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
> index e3442c3..5ee28ea 100644
> --- a/libavformat/hlsenc.c
> +++ b/libavformat/hlsenc.c
> @@ -1918,6 +1918,8 @@ static int hls_write_trailer(struct AVFormatContext *s)
> av_freep(>baseurl);
> }
> 
> +ff_format_io_close(s, >m3u8_out);
> +ff_format_io_close(s, >sub_m3u8_out);
> av_freep(>key_basename);
> av_freep(>var_streams);
> av_freep(>master_m3u8_url);
> -- 
> 1.9.1
> 
> 


LGTM

Thanks

Steven

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avformat/hlsenc: Fix a memory leak when http_persistent is 1

2017-12-18 Thread Karthick J 
From: Karthick Jeyapal 

---
 libavformat/hlsenc.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
index e3442c3..5ee28ea 100644
--- a/libavformat/hlsenc.c
+++ b/libavformat/hlsenc.c
@@ -1918,6 +1918,8 @@ static int hls_write_trailer(struct AVFormatContext *s)
 av_freep(>baseurl);
 }
 
+ff_format_io_close(s, >m3u8_out);
+ff_format_io_close(s, >sub_m3u8_out);
 av_freep(>key_basename);
 av_freep(>var_streams);
 av_freep(>master_m3u8_url);
-- 
1.9.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel