On Tuesday 12 January 2016 02:16:52 pm Michael Niedermayer wrote:
> On Tue, Jan 12, 2016 at 09:58:53AM +0100, Carl Eugen Hoyos wrote:

> > -    if (memcmp("AVI1", buf + 6, 4)) {
> > -        av_log(avctx, AV_LOG_ERROR, "input is not MJPEG/AVI1\n");
> > -        return AVERROR_INVALIDDATA;
> > +    if (buf[2] == 0xff && buf[3] == APP0) {
> > +        input_skip = (buf[4] << 8) + buf[5] + 4;
> > +    } else {
> > +        input_skip = 2;
>
> shouldnt the first 2 bytes that are being skiped be checked ?

I don't know (possibly) but it seems unrelated to this patch: 
They are not checked now.

Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to