From 413c6ec57b6bad7c2e6ccf7c3933ea81b98ad506 Mon Sep 17 00:00:00 2001 From: Vivia Nikolaidou Date: Fri, 11 Oct 2019 17:25:04 +0300 Subject: [PATCH] tsdemux: Always issue a DTS even when it's equal to PTS Currently tsdemux timestamps only the PTS, and only issues the DTS if it's different. In that case, parsers tend to estimate the next DTS based on the previous DTS and the duration, which can accumulate rounding errors. --- gst/mpegtsdemux/tsdemux.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/gst/mpegtsdemux/tsdemux.c b/gst/mpegtsdemux/tsdemux.c index 53f92e3295..c50f811798 100644 --- a/gst/mpegtsdemux/tsdemux.c +++ b/gst/mpegtsdemux/tsdemux.c @@ -3103,7 +3103,8 @@ gst_ts_demux_push_pending_data (GstTSDemux * demux, TSDemuxStream * stream, buffer = gst_buffer_list_get (buffer_list, 0); if (GST_CLOCK_TIME_IS_VALID (stream->pts)) - GST_BUFFER_PTS (buffer) = stream->pts; + GST_BUFFER_PTS (buffer) = GST_BUFFER_DTS (buffer) = stream->pts; + /* DTS = PTS by default, we override it if there's a real DTS */ if (GST_CLOCK_TIME_IS_VALID (stream->dts)) GST_BUFFER_DTS (buffer) = stream->dts;