Commit a3f5ee29 authored by Michael Niedermayer's avatar Michael Niedermayer

mjpeg: Check for interlaced progressive frames

Fixes null pointer dereference.

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
parent 0af48e29
...@@ -312,8 +312,13 @@ int ff_mjpeg_decode_sof(MJpegDecodeContext *s) ...@@ -312,8 +312,13 @@ int ff_mjpeg_decode_sof(MJpegDecodeContext *s)
s->first_picture = 0; s->first_picture = 0;
} }
if (s->interlaced && (s->bottom_field == !s->interlace_polarity)) if (s->interlaced && (s->bottom_field == !s->interlace_polarity)) {
if (s->progressive) {
av_log_ask_for_sample(s->avctx, "progressively coded interlaced pictures not supported\n");
return AVERROR_INVALIDDATA;
}
return 0; return 0;
}
/* XXX: not complete test ! */ /* XXX: not complete test ! */
pix_fmt_id = (s->h_count[0] << 28) | (s->v_count[0] << 24) | pix_fmt_id = (s->h_count[0] << 28) | (s->v_count[0] << 24) |
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment