Commit aa180169 authored by Andreas Cadhalpun's avatar Andreas Cadhalpun

asfdec_o: reject size > INT64_MAX in asf_read_unknown

Both avio_skip and detect_unknown_subobject use int64_t for the size
parameter.

This fixes a segmentation fault due to infinite recursion.
Reviewed-by: 's avatarAlexandra Hájková <alexandra.khirnova@gmail.com>
Signed-off-by: 's avatarAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
parent 763c5728
...@@ -178,6 +178,9 @@ static int asf_read_unknown(AVFormatContext *s, const GUIDParseTable *g) ...@@ -178,6 +178,9 @@ static int asf_read_unknown(AVFormatContext *s, const GUIDParseTable *g)
uint64_t size = avio_rl64(pb); uint64_t size = avio_rl64(pb);
int ret; int ret;
if (size > INT64_MAX)
return AVERROR_INVALIDDATA;
if (asf->is_header) if (asf->is_header)
asf->unknown_size = size; asf->unknown_size = size;
asf->is_header = 0; asf->is_header = 0;
......
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