mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-04-02 14:20:06 +00:00
Code Issues 1 Projects Releases Wiki Activity

rtpjitterbuffer: Handle seqnum gaps in TCP streams without erroring out or overflowing calculations

Browse source 
That is, handle DTS==GST_CLOCK_TIME_NONE correctly.

https://bugzilla.gnome.org/show_bug.cgi?id=749536
This commit is contained in:
Sebastian Dröge 2015-07-08 15:13:17 +03:00
parent 5439fc9a0c
commit 243730ced4
1 changed files with 6 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
gst/rtpmanager/gstrtpjitterbuffer.c
View file

@ -2030,7 +2030,7 @@ update_timers (GstRtpJitterBuffer * jitterbuffer, guint16 seqnum,
}
}
if (do_next_seqnum) {
if (do_next_seqnum && dts != GST_CLOCK_TIME_NONE) {
GstClockTime expected, delay;
/* calculate expected arrival time of the next seqnum */
@ -2106,6 +2106,10 @@ calculate_expected (GstRtpJitterBuffer * jitterbuffer, guint32 expected,
"dts %" GST_TIME_FORMAT ", last %" GST_TIME_FORMAT,
GST_TIME_ARGS (dts), GST_TIME_ARGS (priv->last_in_dts));
/* Nothing to be done here if we don't get packet receive times */
if (dts == GST_CLOCK_TIME_NONE)
return;
/* the total duration spanned by the missing packets */
if (dts >= priv->last_in_dts)
total_duration = dts - priv->last_in_dts;
@ -2449,12 +2453,7 @@ gst_rtp_jitter_buffer_chain (GstPad * pad, GstObject * parent,
} else {
gboolean reset = FALSE;
if (!GST_CLOCK_TIME_IS_VALID (dts)) {
/* We would run into calculations with GST_CLOCK_TIME_NONE below
* and can't compensate for anything without DTS on RTP packets
*/
goto gap_but_no_dts;
} else if (gap < 0) {
if (gap < 0) {
/* we received an old packet */
if (G_UNLIKELY (gap != -1 && gap < -RTP_MAX_MISORDER)) {
reset =
@ -2707,15 +2706,6 @@ duplicate:
free_item (item);
goto finished;
}
gap_but_no_dts:
{
/* this is fatal as we can't compensate for gaps without DTS */
GST_ELEMENT_ERROR (jitterbuffer, STREAM, DECODE, (NULL),
("Received packet without DTS after a gap"));
gst_buffer_unref (buffer);
ret = GST_FLOW_ERROR;
goto finished;
}
}
static GstClockTime