mxfdec: Ignore the last entry in Avid's index table segments
authorTomas Härdin <tomas.hardin@codemill.se>
Mon, 19 Dec 2011 10:00:14 +0000 (11:00 +0100)
committerTomas Härdin <tomas.hardin@codemill.se>
Tue, 20 Dec 2011 08:18:44 +0000 (09:18 +0100)
The last entry is the total size of the essence container.
Previously a TemporalOffset error would be logged, even though segments like
these are expected.

libavformat/mxfdec.c

index 3a09624..8a5c3c9 100644 (file)
@@ -1072,11 +1072,16 @@ static int mxf_compute_ptses_fake_index(MXFContext *mxf, MXFIndexTable *index_ta
     for (i = x = 0; i < index_table->nb_segments; i++) {
         MXFIndexTableSegment *s = index_table->segments[i];
         int index_delta = 1;
+        int n = s->nb_index_entries;
 
-        if (s->nb_index_entries == 2 * s->index_duration + 1)
+        if (s->nb_index_entries == 2 * s->index_duration + 1) {
             index_delta = 2;    /* Avid index */
 
-        for (j = 0; j < s->nb_index_entries; j += index_delta, x++) {
+            /* ignore the last entry - it's the size of the essence container */
+            n--;
+        }
+
+        for (j = 0; j < n; j += index_delta, x++) {
             int offset = s->temporal_offset_entries[j] / index_delta;
             int index  = x + offset;