diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2021-03-18 23:27:57 +0100 |
|---|---|---|
| committer | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2021-03-24 00:30:45 +0100 |
| commit | 3903c139a9cb13cf66178d7f262b8bcc9fa4be5a (patch) | |
| tree | ff29cd25405311284b7065df61994fb512364b09 /libavformat/webpenc.c | |
| parent | df9d969954133c02eabe308f67b819401092d30b (diff) | |
| download | ffmpeg-3903c139a9cb13cf66178d7f262b8bcc9fa4be5a.tar.gz | |
avformat/webpenc: Fix memleak when trailer is never written
When the trailer is never written (or when a stream switches from
non-animation mode to animation mode mid-stream), a cached packet
(if existing) would leak. Fix this by adding a deinit function.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Diffstat (limited to 'libavformat/webpenc.c')
| -rw-r--r-- | libavformat/webpenc.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/libavformat/webpenc.c b/libavformat/webpenc.c index d5edf89289..ed8325c02d 100644 --- a/libavformat/webpenc.c +++ b/libavformat/webpenc.c @@ -191,6 +191,13 @@ static int webp_write_trailer(AVFormatContext *s) return 0; } +static void webp_deinit(AVFormatContext *s) +{ + WebpContext *w = s->priv_data; + + av_packet_unref(&w->last_pkt); +} + #define OFFSET(x) offsetof(WebpContext, x) #define ENC AV_OPT_FLAG_ENCODING_PARAM static const AVOption options[] = { @@ -214,6 +221,7 @@ AVOutputFormat ff_webp_muxer = { .init = webp_init, .write_packet = webp_write_packet, .write_trailer = webp_write_trailer, + .deinit = webp_deinit, .priv_class = &webp_muxer_class, .flags = AVFMT_VARIABLE_FPS, }; |