From 9813df77d68a43c1dbf4e5b0dc4a7e5eb525b5f9 Mon Sep 17 00:00:00 2001 From: James Almer Date: Tue, 23 Jan 2024 10:13:25 -0300 Subject: [PATCH] avformat/iamf_writer: remove bogus check Probably an artifact of a rebase, as this check is done below. Fixes "Conditional jump or move depends on uninitialised value(s)" errors as reported by Valgrind. Signed-off-by: James Almer --- libavformat/iamf_writer.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/libavformat/iamf_writer.c b/libavformat/iamf_writer.c index 9962845049..99602ae204 100644 --- a/libavformat/iamf_writer.c +++ b/libavformat/iamf_writer.c @@ -750,11 +750,6 @@ static int iamf_write_mixing_presentation(const IAMFContext *iamf, int album = submix_layout->album_anchored_loudness.num && submix_layout->album_anchored_loudness.den; - if (layout == FF_ARRAY_ELEMS(ff_iamf_sound_system_map)) { - av_log(log_ctx, AV_LOG_ERROR, "Invalid Sound System value in a submix\n"); - return AVERROR(EINVAL); - } - if (submix_layout->layout_type == AV_IAMF_SUBMIX_LAYOUT_TYPE_LOUDSPEAKERS) { for (layout = 0; layout < FF_ARRAY_ELEMS(ff_iamf_sound_system_map); layout++) { if (!av_channel_layout_compare(&submix_layout->sound_system, &ff_iamf_sound_system_map[layout].layout))