Re: [FFmpeg-devel] [PATCH] avformat/hls: fail on probing non hls/m3u8 file extensions

[Steven Liu <lingjiujianke@gmail.com>]

Michael Niedermayer <michael@niedermayer.cc> 于2023年5月3日周三 20:30写道：
>
> Its unexpected that a .avi or other "standard" file turns into a playlist.
> The goal of this patch is to avoid this unexpected behavior and possible
> privacy or security differences.
>
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavformat/hls.c | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/libavformat/hls.c b/libavformat/hls.c
> index 8a96a37ff9..826135016d 100644
> --- a/libavformat/hls.c
> +++ b/libavformat/hls.c
> @@ -2530,10 +2530,18 @@ static int hls_probe(const AVProbeData *p)
>      if (strncmp(p->buf, "#EXTM3U", 7))
>          return 0;
>
> +
Empty line,
>      if (strstr(p->buf, "#EXT-X-STREAM-INF:")     ||
>          strstr(p->buf, "#EXT-X-TARGETDURATION:") ||
> -        strstr(p->buf, "#EXT-X-MEDIA-SEQUENCE:"))
> +        strstr(p->buf, "#EXT-X-MEDIA-SEQUENCE:")) {
> +
> +        if (!av_match_ext(p->filename, "m3u8,hls,m3u")) {
> +            av_log(NULL, AV_LOG_ERROR, "Not detecting m3u8/hls with non standard extension\n");
> +            return 0;
> +        }
> +
>          return AVPROBE_SCORE_MAX;
> +    }
>      return 0;
>  }
>
> --
> 2.17.1
>
> _______________________________________________

LGTM

Thanks
Steven
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".