diff options
author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-03-27 08:31:29 +0100 |
---|---|---|
committer | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-04-05 18:38:26 +0200 |
commit | da44bbefaabeb2fdb58a03fe533a44aa150486fc (patch) | |
tree | 42cab3bfc781d570f3f691ded20a9dc6dedd60d8 /libavformat/chromaprint.c | |
parent | de0b04551da4e1470ce1c99321a768887df8c650 (diff) | |
download | ffmpeg-da44bbefaabeb2fdb58a03fe533a44aa150486fc.tar.gz |
avformat/avidec: Fix memleak with embedded GAB2 subtitles
The code for GAB2 subtitles predates refcounting AVPackets. So in order
to transfer the ownership of a packet's data pkt->data was simply stored
and the packet zeroed; in the end (i.e. in the read_close-function) this
data was then simply freed with av_freep(). This of course leads to a leak
of an AVBufferRef and an AVBuffer. It has been fixed by keeping and
eventually unreferencing the packet's buf instead.
Additionally, the packet is now reset via av_packet_unref().
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Diffstat (limited to 'libavformat/chromaprint.c')
0 files changed, 0 insertions, 0 deletions