aboutsummaryrefslogtreecommitdiffstats
path: root/libavcodec/xpmdec.c
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-03-27 08:31:29 +0100
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-07-02 23:56:13 +0200
commitc4c3d3bc1de75a112f84cfae319c3d30aa6d8a90 (patch)
treed258654a6fec74c476b1a9a26d537a759f7fe906 /libavcodec/xpmdec.c
parentf34d5ab876a952fcb878c3380371336f80daf45a (diff)
downloadffmpeg-c4c3d3bc1de75a112f84cfae319c3d30aa6d8a90.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> (cherry picked from commit da44bbefaabeb2fdb58a03fe533a44aa150486fc) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Diffstat (limited to 'libavcodec/xpmdec.c')
0 files changed, 0 insertions, 0 deletions