Fix T68091: Adding a corrupt video crashes/confuses Blender
authorSybren A. Stüvel <sybren@blender.org>
Thu, 19 Sep 2019 11:55:44 +0000 (13:55 +0200)
committerSybren A. Stüvel <sybren@blender.org>
Thu, 19 Sep 2019 13:12:53 +0000 (15:12 +0200)
The problematic video from T68091 clearly has an invalid stream duration
(it would be 55 centuries long if interpreted at 30 FPS, and given that
it was recorded with an Android 9 device, it's unlikely that recording
started that long ago). I've added a heuristic to check the stream
duration against the container duration; if the stream is more than 4x
longer than the container, Blender now falls back to the container
duration.

We could use MIN(stream duration, container duration), but there might
be video files out there where the container duration is less precise
than the stream duration; they are measured in different units of time
(microseconds for the container vs. frames for the stream).

Includes a unit test for the above heuristic.

Reviewed by: jbakker

Differential revision: https://developer.blender.org/D5853

source/blender/imbuf/intern/anim_movie.c
tests/python/ffmpeg_tests.py

index 5bfcca22faee66696347795aa87eb3370610af11..a5fa55479dd11dc8f08ee271547a97531563cc85 100644 (file)
@@ -570,10 +570,26 @@ static int startffmpeg(struct anim *anim)
   }
 
   frame_rate = av_guess_frame_rate(pFormatCtx, video_stream, NULL);
+  anim->duration = 0;
+
+  /* Take from the stream if we can. */
   if (video_stream->nb_frames != 0) {
     anim->duration = video_stream->nb_frames;
+
+    /* Sanity check on the detected duration. This is to work around corruption like reported in
+     * T68091. */
+    if (frame_rate.den != 0 && pFormatCtx->duration > 0) {
+      double stream_sec = anim->duration * av_q2d(frame_rate);
+      double container_sec = pFormatCtx->duration / (double)AV_TIME_BASE;
+      if (stream_sec > 4.0 * container_sec) {
+        /* The stream is significantly longer than the container duration, which is
+         * suspicious. */
+        anim->duration = 0;
+      }
+    }
   }
-  else {
+  /* Fall back to the container. */
+  if (anim->duration == 0) {
     anim->duration = (int)(pFormatCtx->duration * av_q2d(frame_rate) / AV_TIME_BASE + 0.5f);
   }
 
index d6e7127c35add6095feb00199971bdad80dada55..92734b5bc7ddd69ed96270b98ba027113753b360 100755 (executable)
@@ -86,6 +86,15 @@ class FPSDetectionTest(AbstractFFmpegSequencerTest):
             1.0,
             places=2)
 
+    def test_T68091(self):
+        self.assertAlmostEqual(
+            self.get_movie_file_fps('T68091-invalid-nb_frames-at-10fps.mp4'),
+            10.0,
+            places=2)
+        self.assertEqual(
+            self.get_movie_file_duration('T68091-invalid-nb_frames-at-10fps.mp4'),
+            10)
+
     def test_T54834(self):
         self.assertEqual(
             self.get_movie_file_duration('T54834.ogg'),