[FFmpeg-devel] [PATCH 2/4] avformat/flvenc: Fix leak of oversized packets

[Andreas Rheinhardt]

Might happen for annex B H.264.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
---
This patchset was actually meant as a reply to [1], but I made an error
with the in-reply-to headers.

[1]: http://ffmpeg.org/pipermail/ffmpeg-devel/2019-October/252057.html

 libavformat/flvenc.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c
index e327545dcf..0e6c66a5ff 100644
--- a/libavformat/flvenc.c
+++ b/libavformat/flvenc.c
@@ -990,7 +990,8 @@ static int flv_write_packet(AVFormatContext *s, AVPacket *pkt)
     if (size + flags_size >= 1<<24) {
         av_log(s, AV_LOG_ERROR, "Too large packet with size %u >= %u\n",
                size + flags_size, 1<<24);
-        return AVERROR(EINVAL);
+        ret = AVERROR(EINVAL);
+        goto fail;
     }
 
     avio_wb24(pb, size + flags_size);
-- 
2.20.1