From ef34ee52694811700de0da9875f828b473a2111f Mon Sep 17 00:00:00 2001 From: Martijn van Beurden Date: Tue, 16 May 2023 20:53:11 +0200 Subject: Small correction on commit 7ed6f4f The 4 byte application id was not considered Credit: Oss-Fuzz Issue: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=59015 --- src/flac/foreign_metadata.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/flac/foreign_metadata.c b/src/flac/foreign_metadata.c index 1a638bc9..1cef901b 100644 --- a/src/flac/foreign_metadata.c +++ b/src/flac/foreign_metadata.c @@ -100,7 +100,7 @@ static FLAC__bool compare_data_(FILE *fin, FILE *fout, size_t size, const char * static FLAC__bool append_block_(foreign_metadata_t *fm, FLAC__off_t offset, FLAC__uint32 size, const char **error) { foreign_block_t *fb; - if(size >= (1u << FLAC__STREAM_METADATA_LENGTH_LEN)) { + if(size >= ((1u << FLAC__STREAM_METADATA_LENGTH_LEN) - FLAC__STREAM_METADATA_APPLICATION_ID_LEN/8)) { if(error) *error = "found foreign metadata chunk is too large (max is 16MiB per chunk)"; return false; } -- cgit v1.2.3