mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2024-11-21 10:55:51 +02:00
dds: validate compressed source buffer size
A too small buffer will cause segfaults somewhere below decompress_texture_thread. Reviewed-by: Vittorio Giovara <vittorio.giovara@gmail.com> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
This commit is contained in:
parent
1675809d2d
commit
9a37d47644
@ -642,9 +642,18 @@ static int dds_decode(AVCodecContext *avctx, void *data,
|
||||
return ret;
|
||||
|
||||
if (ctx->compressed) {
|
||||
int size = (avctx->coded_height / TEXTURE_BLOCK_H) *
|
||||
(avctx->coded_width / TEXTURE_BLOCK_W) * ctx->tex_ratio;
|
||||
ctx->slice_count = av_clip(avctx->thread_count, 1,
|
||||
avctx->coded_height / TEXTURE_BLOCK_H);
|
||||
|
||||
if (bytestream2_get_bytes_left(gbc) < size) {
|
||||
av_log(avctx, AV_LOG_ERROR,
|
||||
"Compressed Buffer is too small (%d < %d).\n",
|
||||
bytestream2_get_bytes_left(gbc), size);
|
||||
return AVERROR_INVALIDDATA;
|
||||
}
|
||||
|
||||
/* Use the decompress function on the texture, one block per thread. */
|
||||
ctx->tex_data = gbc->buffer;
|
||||
avctx->execute2(avctx, decompress_texture_thread, frame, NULL, ctx->slice_count);
|
||||
|
Loading…
Reference in New Issue
Block a user