[FFmpeg-cvslog] avformat/webpenc: Check seeks

Andreas Rheinhardt git at videolan.org
Fri Nov 3 16:15:15 EET 2023


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Fri Oct  6 03:55:31 2023 +0200| [155f0c8ef7430e1f963c811e6bd4e2d85ee9f81b] | committer: Andreas Rheinhardt

avformat/webpenc: Check seeks

Addresses the issue reported in ticket #4609.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=155f0c8ef7430e1f963c811e6bd4e2d85ee9f81b
---

 libavformat/webpenc.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/libavformat/webpenc.c b/libavformat/webpenc.c
index b3f5bdc9b5..fb0cca3922 100644
--- a/libavformat/webpenc.c
+++ b/libavformat/webpenc.c
@@ -177,8 +177,8 @@ static int webp_write_trailer(AVFormatContext *s)
 
     if (w->using_webp_anim_encoder) {
         if (w->loop) {  // Write loop count.
-            avio_seek(s->pb, 42, SEEK_SET);
-            avio_wl16(s->pb, w->loop);
+            if (avio_seek(s->pb, 42, SEEK_SET) == 42)
+                avio_wl16(s->pb, w->loop);
         }
     } else {
         int ret;
@@ -186,10 +186,11 @@ static int webp_write_trailer(AVFormatContext *s)
             return ret;
 
         filesize = avio_tell(s->pb);
-        avio_seek(s->pb, 4, SEEK_SET);
-        avio_wl32(s->pb, filesize - 8);
-        // Note: without the following, avio only writes 8 bytes to the file.
-        avio_seek(s->pb, filesize, SEEK_SET);
+        if (avio_seek(s->pb, 4, SEEK_SET) == 4) {
+            avio_wl32(s->pb, filesize - 8);
+            // Note: without the following, avio only writes 8 bytes to the file.
+            avio_seek(s->pb, filesize, SEEK_SET);
+        }
     }
 
     return 0;



More information about the ffmpeg-cvslog mailing list