mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-01-24 13:56:33 +02:00
sanm: Check dimensions before use
Fixes integer overflow and out of array accesses Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
This commit is contained in:
parent
ad002e1a13
commit
9dd04f6d8c
@ -732,6 +732,11 @@ static int process_frame_obj(SANMVideoContext *ctx)
|
|||||||
w = bytestream2_get_le16u(&ctx->gb);
|
w = bytestream2_get_le16u(&ctx->gb);
|
||||||
h = bytestream2_get_le16u(&ctx->gb);
|
h = bytestream2_get_le16u(&ctx->gb);
|
||||||
|
|
||||||
|
if (!w || !h) {
|
||||||
|
av_log(ctx->avctx, AV_LOG_ERROR, "dimensions are invalid\n");
|
||||||
|
return AVERROR_INVALIDDATA;
|
||||||
|
}
|
||||||
|
|
||||||
if (ctx->width < left + w || ctx->height < top + h) {
|
if (ctx->width < left + w || ctx->height < top + h) {
|
||||||
if (av_image_check_size(FFMAX(left + w, ctx->width),
|
if (av_image_check_size(FFMAX(left + w, ctx->width),
|
||||||
FFMAX(top + h, ctx->height), 0, ctx->avctx) < 0)
|
FFMAX(top + h, ctx->height), 0, ctx->avctx) < 0)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user