aboutsummaryrefslogtreecommitdiffstats
path: root/libswresample/x86/audio_convert_init.c
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2021-03-20 07:43:09 +0100
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2021-03-22 08:17:10 +0100
commit2a5c577ef347c9d947f3421c51e01cb88d7aa452 (patch)
treebd289b50c398f5a1814d2f679c2ab111a350ac99 /libswresample/x86/audio_convert_init.c
parent8f099e3a67655fc498e44bb3a33961120449f866 (diff)
downloadffmpeg-2a5c577ef347c9d947f3421c51e01cb88d7aa452.tar.gz
avformat/pp_bnk: Fix memleaks when reading non-stereo tracks
Commit 6973df112275c8ea4af0bf3cb1338baecc1d06b3 added support for music tracks by outputting its two containing tracks together in one packet. But the actual data is not contiguous in the file and therefore one can't simply use av_get_packet() (which has been used before) for it. Therefore the packet was now allocated via av_new_packet() and read via avio_read(); and this is also for non-music files. This causes problems because one can now longer rely on things done automatically by av_get_packet(): It automatically freed the packet in case of errors; this lead to memleaks in several FATE-tests covering this demuxer. Furthermore, in case the data read is less than the data desired, the returned packet was not zero-allocated (the packet's padding was uninitialized); for music files the actual data could even be uninitialized. The former problems are fixed by using av_get_packet() for non-music files; the latter problem is handled by erroring out unless both tracks could be fully read. Reviewed-by: Zane van Iperen <zane@zanevaniperen.com> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> (cherry picked from commit 8a73313412eeafcfa5afa45f39f65f2581ba3bbc)
Diffstat (limited to 'libswresample/x86/audio_convert_init.c')
0 files changed, 0 insertions, 0 deletions