[FFmpeg-cvslog] mxf: Do not use int to check the seek position

Luca Barbato git at videolan.org
Tue Jan 14 21:43:38 CET 2014


ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Tue Jan  7 21:26:14 2014 +0100| [42f9132218ca11a8e9a3c82a175b46bca092113e] | committer: Luca Barbato

mxf: Do not use int to check the seek position

Overly large files are to be expected.

Reported-by: Jean-Baptiste Kempf <jb at videolan.org>

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

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

diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
index 7126bed..37e1cfa 100644
--- a/libavformat/mxfdec.c
+++ b/libavformat/mxfdec.c
@@ -2238,8 +2238,10 @@ static int mxf_read_seek(AVFormatContext *s, int stream_index, int64_t sample_ti
         sample_time = 0;
     seconds = av_rescale(sample_time, st->time_base.num, st->time_base.den);
 
-    if ((ret = avio_seek(s->pb, (s->bit_rate * seconds) >> 3, SEEK_SET)) < 0)
-        return ret;
+    seekpos = avio_seek(s->pb, (s->bit_rate * seconds) >> 3, SEEK_SET);
+    if (seekpos < 0)
+        return seekpos;
+
     ff_update_cur_dts(s, st, sample_time);
     mxf->current_edit_unit = sample_time;
     } else {



More information about the ffmpeg-cvslog mailing list