aboutsummaryrefslogtreecommitdiffstats
path: root/libavcodec/mpeg2_metadata_bsf.c
diff options
context:
space:
mode:
authorAndriy Gelman <andriy.gelman@gmail.com>2019-12-06 14:22:14 -0500
committerJames Almer <jamrial@gmail.com>2019-12-31 16:57:37 -0300
commit662accb728639131638cd5be5f227b25a4559211 (patch)
tree97b15eb068857d7cc79ad0f84b338e7644893633 /libavcodec/mpeg2_metadata_bsf.c
parent4667920455c0d52c25835ff81098254213f2d018 (diff)
downloadffmpeg-662accb728639131638cd5be5f227b25a4559211.tar.gz
lavc/cbs_h2645_syntax_template: Fix memleak
payload_count is used to track the number of SEI payloads. It is also used to free the SEIs in cbs_h264_free_sei()/cbs_h265_free_sei(). Currently, payload_count is set after for loop is completed. Hence if there is an error and the function exits, the payload remains zero causing a memleak. This commit keeps track of payload_count inside the for loop to fix the issue. Note that that the contents of current are initialized with av_mallocz() so there is no need to zero initialize payload_count. Found-by: libFuzzer Reviewed-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: Andriy Gelman <andriy.gelman@gmail.com> (cherry picked from commit c07a77247363eb666a49536af505e7317225ee81)
Diffstat (limited to 'libavcodec/mpeg2_metadata_bsf.c')
0 files changed, 0 insertions, 0 deletions