commit ee56a7cb99641804c573665ac551490c96e79f90
author David Schleef <ds@schleef.org> Wed Mar 13 17:08:03 2013 -0700
committer David Schleef <ds@schleef.org> Fri Mar 22 14:14:04 2013 -0700
tree 45d038e71924523cbfd6b25e4e50997890f06d66
parent 816e18602929bca9bb371e35be9c5a0289b1ae00

qtmux: Fix problems causing bad durations in file

- Fix up out-of-order incoming DTS values.
- Fix duration of initial sample.

gst/isomp4/gstqtmux.c

diff --git a/gst/isomp4/gstqtmux.c b/gst/isomp4/gstqtmux.c
index 2b4a4e2..9875853 100644
--- a/gst/isomp4/gstqtmux.c
+++ b/gst/isomp4/gstqtmux.c
@@ -2154,6 +2154,7 @@
   if (G_UNLIKELY (pad->first_ts == GST_CLOCK_TIME_NONE) && last_buf) {
     if (GST_CLOCK_TIME_IS_VALID (GST_BUFFER_PTS (last_buf))) {
       pad->first_ts = GST_BUFFER_PTS (last_buf);
+      GST_DEBUG ("setting first_ts to %" G_GUINT64_FORMAT, pad->first_ts);
       check_and_subtract_ts (qtmux, &GST_BUFFER_DTS (last_buf), pad->first_ts);
       check_and_subtract_ts (qtmux, &GST_BUFFER_PTS (last_buf), pad->first_ts);
       check_and_subtract_ts (qtmux, &GST_BUFFER_DTS (buf), pad->first_ts);
@@ -2169,6 +2170,15 @@
 
   }
 
+  if (last_buf && buf && GST_CLOCK_TIME_IS_VALID (GST_BUFFER_DTS (buf)) &&
+      GST_CLOCK_TIME_IS_VALID (GST_BUFFER_DTS (last_buf)) &&
+      GST_BUFFER_DTS (buf) < GST_BUFFER_DTS (last_buf)) {
+    GST_ERROR ("decreasing DTS value %" GST_TIME_FORMAT " < %" GST_TIME_FORMAT,
+        GST_TIME_ARGS (GST_BUFFER_DTS (buf)),
+        GST_TIME_ARGS (GST_BUFFER_DTS (last_buf)));
+    GST_BUFFER_DTS (buf) = GST_BUFFER_DTS (last_buf);
+  }
+
   duration = GST_BUFFER_DURATION (last_buf);
 
   /* for computing the avg bitrate */
@@ -2208,7 +2218,12 @@
     sample_size = gst_buffer_get_size (last_buf);
     if (pad->have_dts) {
       gint64 scaled_dts;
-      pad->last_dts = GST_BUFFER_DTS (last_buf);
+      if (pad->last_buf) {
+        pad->last_dts = GST_BUFFER_DTS (pad->last_buf);
+      } else {
+        pad->last_dts = GST_BUFFER_DTS (last_buf) +
+            GST_BUFFER_DURATION (last_buf);
+      }
       if ((gint64) (pad->last_dts) < 0) {
         scaled_dts = -gst_util_uint64_scale_round (-pad->last_dts,
             atom_trak_get_timescale (pad->trak), GST_SECOND);