Skip to content
Snippets Groups Projects
Commit 8c71de76 authored by Michael Niedermayer's avatar Michael Niedermayer
Browse files

Merge commit 'c5560e72'


* commit 'c5560e72':
  apetag: Fix APE tag size check

Merged-by: default avatarMichael Niedermayer <michaelni@gmx.at>
parents 43633c53 c5560e72
No related branches found
No related tags found
No related merge requests found
...@@ -55,8 +55,10 @@ static int ape_tag_read_field(AVFormatContext *s) ...@@ -55,8 +55,10 @@ static int ape_tag_read_field(AVFormatContext *s)
av_log(s, AV_LOG_WARNING, "Invalid APE tag key '%s'.\n", key); av_log(s, AV_LOG_WARNING, "Invalid APE tag key '%s'.\n", key);
return -1; return -1;
} }
if (size >= UINT_MAX) if (size > INT32_MAX - FF_INPUT_BUFFER_PADDING_SIZE) {
return -1; av_log(s, AV_LOG_ERROR, "APE tag size too large.\n");
return AVERROR_INVALIDDATA;
}
if (flags & APE_TAG_FLAG_IS_BINARY) { if (flags & APE_TAG_FLAG_IS_BINARY) {
uint8_t filename[1024]; uint8_t filename[1024];
enum AVCodecID id; enum AVCodecID id;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment