diff options
| author | James Almer <jamrial@gmail.com> | 2024-01-23 10:13:25 -0300 |
|---|---|---|
| committer | James Almer <jamrial@gmail.com> | 2024-01-23 10:14:10 -0300 |
| commit | 9813df77d68a43c1dbf4e5b0dc4a7e5eb525b5f9 (patch) | |
| tree | 76711cd2edfa3b8ab51995eba8e16f0b55786951 | |
| parent | b6af126c72e6f2988ca29c189926b20f12ff19de (diff) | |
| download | ffmpeg-9813df77d68a43c1dbf4e5b0dc4a7e5eb525b5f9.tar.gz | |
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 <jamrial@gmail.com>
| -rw-r--r-- | libavformat/iamf_writer.c | 5 |
1 files changed, 0 insertions, 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)) |