mxf: Do not use int to check the seek position
authorLuca Barbato <lu_zero@gentoo.org>
Tue, 7 Jan 2014 20:26:14 +0000 (21:26 +0100)
committerLuca Barbato <lu_zero@gentoo.org>
Tue, 14 Jan 2014 16:29:03 +0000 (17:29 +0100)
Overly large files are to be expected.

Reported-by: Jean-Baptiste Kempf <jb@videolan.org>
libavformat/mxfdec.c

index 7126bed..37e1cfa 100644 (file)
@@ -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 {