mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2024-12-28 20:53:54 +02:00
avcodec/tiff: Check frame parameters before blit for DNG
Fixes: out of array access Fixes: 23888/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TIFF_fuzzer-6021365974171648.fuzz Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
parent
865a34970e
commit
4091f4f780
@ -911,12 +911,23 @@ static int dng_decode_jpeg(AVCodecContext *avctx, AVFrame *frame,
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
is_u16 = (s->bpp > 8);
|
||||||
|
|
||||||
/* Copy the outputted tile's pixels from 'jpgframe' to 'frame' (final buffer) */
|
/* Copy the outputted tile's pixels from 'jpgframe' to 'frame' (final buffer) */
|
||||||
|
|
||||||
/* See dng_blit for explanation */
|
/* See dng_blit for explanation */
|
||||||
is_single_comp = (s->avctx_mjpeg->width == w * 2 && s->avctx_mjpeg->height == h / 2);
|
if (s->avctx_mjpeg->width == w * 2 &&
|
||||||
|
s->avctx_mjpeg->height == h / 2 &&
|
||||||
|
s->avctx_mjpeg->pix_fmt == AV_PIX_FMT_GRAY16LE) {
|
||||||
|
is_single_comp = 1;
|
||||||
|
} else if (s->avctx_mjpeg->width == w &&
|
||||||
|
s->avctx_mjpeg->height == h &&
|
||||||
|
s->avctx_mjpeg->pix_fmt == (is_u16 ? AV_PIX_FMT_GRAY16 : AV_PIX_FMT_GRAY8)
|
||||||
|
) {
|
||||||
|
is_single_comp = 0;
|
||||||
|
} else
|
||||||
|
return AVERROR_INVALIDDATA;
|
||||||
|
|
||||||
is_u16 = (s->bpp > 8);
|
|
||||||
pixel_size = (is_u16 ? sizeof(uint16_t) : sizeof(uint8_t));
|
pixel_size = (is_u16 ? sizeof(uint16_t) : sizeof(uint8_t));
|
||||||
|
|
||||||
if (is_single_comp && !is_u16) {
|
if (is_single_comp && !is_u16) {
|
||||||
|
Loading…
Reference in New Issue
Block a user