[FFmpeg-devel] [PATCH] hls: fix baseurl missing last char

[Steven Liu]

2017-12-02 8:59 GMT+08:00 Michael Niedermayer <michael at niedermayer.cc>:
> On Fri, Dec 01, 2017 at 11:06:07AM +0100, Robert Nagy wrote:
>> ---
>>  libavformat/hlsenc.c | 5 +----
>>  1 file changed, 1 insertion(+), 4 deletions(-)
>>
>> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
>> index cdfbf45823..dc8bf48791 100644
>> --- a/libavformat/hlsenc.c
>> +++ b/libavformat/hlsenc.c
>> @@ -1921,14 +1921,11 @@ static int hls_write_header(AVFormatContext *s)
>>      }
>>
>>      if (hls->baseurl) {
>> -        int baseurl_len;
>> -        baseurl_len = strlen(hls->baseurl);
>> -        vs->baseurl = av_malloc(baseurl_len);
>> +        vs->baseurl = av_strdup(hls->baseurl);
>>          if (!vs->baseurl) {
>>              ret = AVERROR(ENOMEM);
>>              goto fail;
>>          }
>> -        av_strlcpy(vs->baseurl, hls->baseurl, baseurl_len);
>>      }
>>
>>      if ((hls->flags & HLS_SINGLE_FILE) && (hls->segment_type ==
>> SEGMENT_TYPE_FMP4)) {
>
> this looks like a stray linebreak
> this wont apply automatically

Ok, let me apply it manually.


Thanks
>
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> In a rich man's house there is no place to spit but his face.
> -- Diogenes of Sinope
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>