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

[Michael Niedermayer]

On Thu, Jan 21, 2016 at 12:17:48AM +0100, Andreas Cadhalpun wrote:
> 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.

applied

thx

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

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- 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/20160122/cfa823ca/attachment.sig>