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