From 1ef18d022373b3cd478ad50e7017591667f0d15c Mon Sep 17 00:00:00 2001 From: James Almer Date: Fri, 19 Jul 2024 20:50:52 -0300 Subject: [PATCH] avformat/iamf_writer: disallow Opus extradata with mapping family other than 0 Clause 3.11.1 of IAMF[1] states the Opus ID Header should conform to ChannelMappingFamily == 0. [1]https://aomediacodec.github.io/iamf/#opus-specific Signed-off-by: James Almer (cherry picked from commit 2aab4e4cc0b4a666c7e5a752b25a337710b20bdb) --- libavformat/iamf_writer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/iamf_writer.c b/libavformat/iamf_writer.c index afe7a149cb..5e8d8f768b 100644 --- a/libavformat/iamf_writer.c +++ b/libavformat/iamf_writer.c @@ -39,7 +39,7 @@ static int update_extradata(IAMFCodecConfig *codec_config) switch(codec_config->codec_id) { case AV_CODEC_ID_OPUS: - if (codec_config->extradata_size < 19) + if (codec_config->extradata_size != 19) return AVERROR_INVALIDDATA; codec_config->extradata_size -= 8; AV_WB8(codec_config->extradata + 0, AV_RL8(codec_config->extradata + 8)); // version