diff options
author | Andriy Gelman <andriy.gelman@gmail.com> | 2019-12-06 14:22:14 -0500 |
---|---|---|
committer | James Almer <jamrial@gmail.com> | 2019-12-31 16:57:37 -0300 |
commit | 662accb728639131638cd5be5f227b25a4559211 (patch) | |
tree | 97b15eb068857d7cc79ad0f84b338e7644893633 /libavformat/jacosubdec.c | |
parent | 4667920455c0d52c25835ff81098254213f2d018 (diff) | |
download | ffmpeg-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 'libavformat/jacosubdec.c')
0 files changed, 0 insertions, 0 deletions