diff options
author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2019-12-16 01:04:07 +0100 |
---|---|---|
committer | Steven Liu <lq@chinaffmpeg.org> | 2019-12-23 14:05:40 +0800 |
commit | bd131b64bc308ab036d0bbe9da0a49f482ef94f9 (patch) | |
tree | 384267110e44af8a946225b4aa200e19c403dbf3 /libavformat/udp.c | |
parent | 9e4b3ccbb62c29eb1f95af485ec6f0d1e0e4109a (diff) | |
download | ffmpeg-bd131b64bc308ab036d0bbe9da0a49f482ef94f9.tar.gz |
avformat/hlsenc: Fix leak of options when writing packets
Under certain circumstances hls_write_packet() would add options to an
AVDictionary. Said dictionary was never explicitly freed, instead it was
presumed that these options would be consumed when opening a new
IO-context. This left several possibilities for memleaks:
a) When no new IO-context would be opened at all. This is possible when
using both the flags temp_file and single_file together with a file
output.
b) When an error happens before one actually tries to open the new
IO-context.
c) When the new IO-context does not consume all options.
All three have been fixed; furthermore, the AVDictionary has been put
into a smaller scope (namely the only part of hls_write_packet() where
it is actually used).
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Reviewed-by: Steven Liu <lq@onvideo.cn>
Diffstat (limited to 'libavformat/udp.c')
0 files changed, 0 insertions, 0 deletions