[FFmpeg-devel] [PATCH 1/4] avformat/concat: Check protocol prefix

[Andreas Cadhalpun]

On 20.01.2016 11:10, Michael Niedermayer wrote:
> From: Michael Niedermayer <michael at niedermayer.cc>
> 
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/concat.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/concat.c b/libavformat/concat.c
> index 81fe970..7bcc279 100644
> --- a/libavformat/concat.c
> +++ b/libavformat/concat.c
> @@ -65,7 +65,10 @@ static av_cold int concat_open(URLContext *h, const char *uri, int flags)
>      struct concat_data  *data = h->priv_data;
>      struct concat_nodes *nodes;
>  
> -    av_strstart(uri, "concat:", &uri);
> +    if (!av_strstart(uri, "concat:", &uri)) {
> +        av_log(h, AV_LOG_ERROR, "URL %s lacks prefix\n", uri);
> +        return AVERROR(EINVAL);
> +    }
>  
>      for (i = 0, len = 1; uri[i]; i++) {
>          if (uri[i] == *AV_CAT_SEPARATOR) {
> 

Looks good.

Best regards,
Andreas