[FFmpeg-devel] [PATCH] Revert "avformat/hls: Require the file extension to be m3u / m3u8 for probing to succeed"

wm4 nfxjfg at googlemail.com
Mon Feb 1 10:53:04 CET 2016


On Mon,  1 Feb 2016 00:05:33 +0100
Michael Niedermayer <michael at niedermayer.cc> wrote:

> This can cause problems with urls that have arguments after the filename
> 
> This reverts commit b0c57206d583517a5ea35dd7f365f8260d9106f2.
> ---
>  libavformat/hls.c |    3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/libavformat/hls.c b/libavformat/hls.c
> index dce8a5c..2d8e6c0 100644
> --- a/libavformat/hls.c
> +++ b/libavformat/hls.c
> @@ -1984,9 +1984,6 @@ static int hls_probe(AVProbeData *p)
>      if (strncmp(p->buf, "#EXTM3U", 7))
>          return 0;
>  
> -    if (p->filename && *p->filename && !av_match_ext(p->filename, "m3u8,m3u"))
> -        return 0;
> -
>      if (strstr(p->buf, "#EXT-X-STREAM-INF:")     ||
>          strstr(p->buf, "#EXT-X-TARGETDURATION:") ||
>          strstr(p->buf, "#EXT-X-MEDIA-SEQUENCE:"))

I'd say this didn't increase security anyway, so it can go.


More information about the ffmpeg-devel mailing list