]> git.sesse.net Git - ffmpeg/commit
avformat/matroskaenc: Replace impossible condition with assert
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Tue, 28 Apr 2020 00:24:16 +0000 (02:24 +0200)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Sun, 3 May 2020 10:44:54 +0000 (12:44 +0200)
commitca0a38f2f7803ab8fa12913d0385d4c70f8d6345
tree2d69f2f082a7a90ad3974757ac812bf8745ad8a0
parentd13feae0f85bebdd35e44ccbadf974b10dc3cef1
avformat/matroskaenc: Replace impossible condition with assert

If a FLAC track uses an unconventional channel layout, the Matroska
muxer adds a WAVEFORMATEXTENSIBLE_CHANNEL_MASK VorbisComment to the
CodecPrivate to preserve this information. And given that FLAC uses
24bit length fields, the muxer checks if the length is more than this
and errors out if it is.

Yet this can never happen, because we create the AVDictionary that is
the source for the VorbisComment. It only contains exactly one entry
that can't grow infinitely large (in fact, the length of the
VorbisComment is <= 4 + 33 + 1 + 18 + strlen(LIBAVFORMAT_IDENT)).
So we can simply assert the size to be < (1 << 24) - 4.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavformat/matroskaenc.c