]> git.sesse.net Git - ffmpeg/commit
avformat/matroskadec: Fix buffer overflow when demuxing RealAudio 28.8
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Tue, 21 Apr 2020 22:15:54 +0000 (00:15 +0200)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Fri, 1 May 2020 05:27:36 +0000 (07:27 +0200)
commit4f5c6c1b0ec2407dfd42fcfa3441067de1962a54
treedaab26d918d23996f3fd839d137170cb0bed9c06
parentc91e3690d9c6667123b116c9fd3becf5f4f4530e
avformat/matroskadec: Fix buffer overflow when demuxing RealAudio 28.8

RealAudio 28.8 (like other RealAudio codecs) uses a special demuxing
mode in which the data of the existing Matroska Blocks is not simply
forwarded as-is. Instead data from several Blocks is recombined
together to output several packets. The parameters governing this
process are parsed from the CodecPrivate: Coded framesize (cfs), frame
size (w) and sub_packet_h (h).

During demuxing, h/2 pieces of data of size cfs each are read from every
Matroska (Simple)Block and put at offset m * 2 * w + n * cfs of a buffer
of size h * w, where m ranges from 0 to h/2 - 1 for each Block while n
is initially zero and incremented after a Block has been parsed until it
is h, at which poin the assembled packets are output and n reset.

The highest offset is given by (h/2 - 1) * 2 * w + (h - 1) * cfs + cfs
while the destination buffer's size is given by h * w. For even h, this
leads to a buffer overflow (and potential segfault) if h * cfs > 2 * w;
for odd h, the condition is h * cfs > 3 * w.

This commit adds a check to rule this out.

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