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

Michael Niedermayer michael at niedermayer.cc
Mon Feb 1 15:30:40 CET 2016


On Mon, Feb 01, 2016 at 10:53:04AM +0100, wm4 wrote:
> 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.

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Never trust a computer, one day, it may think you are the virus. -- Compn
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160201/b61094cd/attachment.sig>


More information about the ffmpeg-devel mailing list