Commit c2852e4e authored by Michael Niedermayer's avatar Michael Niedermayer

avcodec/xwddec: Use ff_set_dimensions()

Fixes: OOM
Fixes: 8178/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_XWD_fuzzer-4844793342459904

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpegReviewed-by: 's avatarPaul B Mahol <onemda@gmail.com>
Signed-off-by: 's avatarMichael Niedermayer <michael@niedermayer.cc>
parent d13379fb
...@@ -39,6 +39,7 @@ static int xwd_decode_frame(AVCodecContext *avctx, void *data, ...@@ -39,6 +39,7 @@ static int xwd_decode_frame(AVCodecContext *avctx, void *data,
uint32_t pixformat, pixdepth, bunit, bitorder, bpad; uint32_t pixformat, pixdepth, bunit, bitorder, bpad;
uint32_t rgb[3]; uint32_t rgb[3];
uint8_t *ptr; uint8_t *ptr;
int width, height;
GetByteContext gb; GetByteContext gb;
if (buf_size < XWD_HEADER_SIZE) if (buf_size < XWD_HEADER_SIZE)
...@@ -60,8 +61,8 @@ static int xwd_decode_frame(AVCodecContext *avctx, void *data, ...@@ -60,8 +61,8 @@ static int xwd_decode_frame(AVCodecContext *avctx, void *data,
pixformat = bytestream2_get_be32u(&gb); pixformat = bytestream2_get_be32u(&gb);
pixdepth = bytestream2_get_be32u(&gb); pixdepth = bytestream2_get_be32u(&gb);
avctx->width = bytestream2_get_be32u(&gb); width = bytestream2_get_be32u(&gb);
avctx->height = bytestream2_get_be32u(&gb); height = bytestream2_get_be32u(&gb);
xoffset = bytestream2_get_be32u(&gb); xoffset = bytestream2_get_be32u(&gb);
be = bytestream2_get_be32u(&gb); be = bytestream2_get_be32u(&gb);
bunit = bytestream2_get_be32u(&gb); bunit = bytestream2_get_be32u(&gb);
...@@ -77,6 +78,9 @@ static int xwd_decode_frame(AVCodecContext *avctx, void *data, ...@@ -77,6 +78,9 @@ static int xwd_decode_frame(AVCodecContext *avctx, void *data,
ncolors = bytestream2_get_be32u(&gb); ncolors = bytestream2_get_be32u(&gb);
bytestream2_skipu(&gb, header_size - (XWD_HEADER_SIZE - 20)); bytestream2_skipu(&gb, header_size - (XWD_HEADER_SIZE - 20));
if ((ret = ff_set_dimensions(avctx, width, height)) < 0)
return ret;
av_log(avctx, AV_LOG_DEBUG, av_log(avctx, AV_LOG_DEBUG,
"pixformat %"PRIu32", pixdepth %"PRIu32", bunit %"PRIu32", bitorder %"PRIu32", bpad %"PRIu32"\n", "pixformat %"PRIu32", pixdepth %"PRIu32", bunit %"PRIu32", bitorder %"PRIu32", bpad %"PRIu32"\n",
pixformat, pixdepth, bunit, bitorder, bpad); pixformat, pixdepth, bunit, bitorder, bpad);
......
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