Michael Niedermayer: > On Wed, Apr 15, 2020 at 11:06:13PM +0200, Andreas Rheinhardt wrote: >> The data of an attachment file is put into an AVCodecParameter's >> extradata. The corresponding size field has type int, yet there was no >> check for the size to fit into an int. As a consequence, it was possible >> to create extradata with negative size (by using a big enough max_alloc). >> >> Other errors were also possible: If SIZE_MAX < INT64_MAX (e.g. on 32bit >> systems) then the file size might be truncated before the allocation; >> and avio_read() takes an int, too, so one would not have read as much >> as one desired. >> >> Furthermore, the extradata is now padded as is required. >> >> Signed-off-by: Andreas Rheinhardt <[email protected]> >> --- >> fftools/ffmpeg_opt.c | 6 ++++-- >> 1 file changed, 4 insertions(+), 2 deletions(-) > > LGTM > > thx > > [...] > Applied, thanks.
- Andreas _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
