From: James Almer <jamrial@gmail.com>
Date: Sun, 11 Apr 2021 01:53:34 +0000 (-0300)
Subject: avformat/webpenc: don't assume animated webp streams will have more than one packet
X-Git-Url: https://git.sesse.net/?p=ffmpeg;a=commitdiff_plain;h=55d667d86af7d13fc5aa2b4071a5b97eb10e2da6

avformat/webpenc: don't assume animated webp streams will have more than one packet

The libwebp_animencoder returns a single packet with the entire animated
stream, as that's what the external library produces. As such, only ensure the
stream was produced by said encoder (or propagated by a demuxer, once support
is added) when attempting to write the requested loop value.

Fixes ticket #9179.

Signed-off-by: James Almer <jamrial@gmail.com>
---

diff --git a/libavformat/webpenc.c b/libavformat/webpenc.c
index ed8325c02d7..ca4ffc4e2dd 100644
--- a/libavformat/webpenc.c
+++ b/libavformat/webpenc.c
@@ -172,7 +172,7 @@ static int webp_write_trailer(AVFormatContext *s)
     WebpContext *w = s->priv_data;
 
     if (w->using_webp_anim_encoder) {
-        if ((w->frame_count > 1) && w->loop) {  // Write loop count.
+        if (w->loop) {  // Write loop count.
             avio_seek(s->pb, 42, SEEK_SET);
             avio_wl16(s->pb, w->loop);
         }