[FFmpeg-cvslog] avformat/mov: fix eof check in mov_read_iinf()

[Kacper Michajłow]

ffmpeg | branch: master | Kacper Michajłow <kasper93 at gmail.com> | Mon Feb  3 06:25:53 2025 +0100| [c17774a9ae8353373e868418fb8789394bb3f8d9] | committer: Michael Niedermayer

avformat/mov: fix eof check in mov_read_iinf()

This fix ensures that the loop stops early on EOF. The issue occurs
because mov_read_infe() performs a version check and skips unsupported
versions. The problem is that seeking within the stream clears the EOF
flag, causing avio_feof() to not function as expected. This is resolved
by moving the EOF check after reading the size and type, ensuring the
EOF flag is set when necessary.

Signed-off-by: Kacper Michajłow <kasper93 at gmail.com>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

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

 libavformat/mov.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 2c8be51063..9a388bf723 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -8868,12 +8868,12 @@ static int mov_read_iinf(MOVContext *c, AVIOContext *pb, MOVAtom atom)
     for (i = 0; i < entry_count; i++) {
         MOVAtom infe;
 
+        infe.size = avio_rb32(pb) - 8;
+        infe.type = avio_rl32(pb);
         if (avio_feof(pb)) {
             ret = AVERROR_INVALIDDATA;
             goto fail;
         }
-        infe.size = avio_rb32(pb) - 8;
-        infe.type = avio_rl32(pb);
         ret = mov_read_infe(c, pb, infe, i);
         if (ret < 0)
             goto fail;