[FFmpeg-devel] [PATCH 3/3] avformat/mov: Expose encryption info to the app.

[Carl Eugen Hoyos]

2018-01-09 0:22 GMT+01:00 Jacob Trimble <modmaker-at-google.com at ffmpeg.org>:

> Updated with the new design for the side data and applied the realloc
> fix to avoid large allocations.

> +        kid_count = avio_rb32(pb);

Missing check here ...

> +        for (; i < kid_count && !pb->eof_reached; i++) {
> +            unsigned int min_kid_count = FFMIN(FFMAX(i, 1024), kid_count);
> +            key_ids = av_fast_realloc(info->key_ids, &alloc_size,

> +                                      min_kid_count * sizeof(*key_ids));

... for an overflow here.

Thank you, Carl Eugen