From dd5f665b4010f8a0142ce3cba3305b173eb37dfe Mon Sep 17 00:00:00 2001 From: James Almer Date: Thu, 11 Apr 2024 12:32:11 -0300 Subject: [PATCH] avutil/frame: free the old side data buffer before replacing it Fixes memleaks reported by ASAN/Valgrind. Reported-by: Sean McGovern Signed-off-by: James Almer --- libavutil/frame.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/frame.c b/libavutil/frame.c index 10b9f7fa94..0775e2abd9 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c @@ -811,6 +811,7 @@ static AVFrameSideData *replace_side_data_from_buf(AVFrameSideData *dst, return NULL; av_dict_free(&dst->metadata); + av_buffer_unref(&dst->buf); dst->buf = buf; dst->data = buf->data; dst->size = buf->size;