avformat/utils: reorder operations in update_wrap_reference()
authorMichael Niedermayer <michaelni@gmx.at>
Wed, 4 Dec 2013 13:29:35 +0000 (14:29 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Wed, 4 Dec 2013 13:36:35 +0000 (14:36 +0100)
prevents a integer overflow

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libavformat/utils.c

index f33704d..dcb94e8 100644 (file)
@@ -887,12 +887,11 @@ static int update_wrap_reference(AVFormatContext *s, AVStream *st, int stream_in
 
     if (ref == AV_NOPTS_VALUE)
         ref = pkt->pts;
-    if (ref == AV_NOPTS_VALUE)
+    if (st->pts_wrap_reference != AV_NOPTS_VALUE || st->pts_wrap_bits >= 63 || ref == AV_NOPTS_VALUE || !s->correct_ts_overflow)
         return 0;
     ref &= (1LL<<st->pts_wrap_bits)-1;
 
-    if (s->correct_ts_overflow && st->pts_wrap_bits < 63 &&
-        st->pts_wrap_reference == AV_NOPTS_VALUE) {
+    {
         int i;
 
         // reference time stamp should be 60 s before first time stamp
@@ -945,7 +944,6 @@ static int update_wrap_reference(AVFormatContext *s, AVStream *st, int stream_in
         }
         return 1;
     }
-    return 0;
 }
 
 static void update_initial_timestamps(AVFormatContext *s, int stream_index,