FaSTTUBe/GST-Tensordecoder-ov_ep
61
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

rtpjitterbuffer: Lock clock_rate variable

Browse Source 
The priv->clock_rate variable could become -1 between when its checked to not
be -1 and when its used, causing an assertion. Fixed by taking the mutex
earlier in the chain() function.

Fixes #593955
This commit is contained in:
Olivier Crête 2009-09-02 15:21:02 -04:00 committed by Wim Taymans
parent 3fcde4486d
commit f542f710cf
1 changed files with 31 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
gst/rtpmanager/gstrtpjitterbuffer.c
View File

@ -609,7 +609,10 @@ gst_rtp_jitter_buffer_clear_pt_map (GstRtpJitterBuffer * jitterbuffer)
priv = jitterbuffer->priv; priv = jitterbuffer->priv;
/* this will trigger a new pt-map request signal, FIXME, do something better. */ /* this will trigger a new pt-map request signal, FIXME, do something better. */
JBUF_LOCK (priv);
priv->clock_rate = -1; priv->clock_rate = -1;
JBUF_UNLOCK (priv);
} }
static GstCaps * static GstCaps *
@ -647,6 +650,10 @@ gst_rtp_jitter_buffer_getcaps (GstPad * pad)
return caps; return caps;
} }
/*
* Must be called with JBUF_LOCK held
*/
static gboolean static gboolean
gst_jitter_buffer_sink_parse_caps (GstRtpJitterBuffer * jitterbuffer, gst_jitter_buffer_sink_parse_caps (GstRtpJitterBuffer * jitterbuffer,
GstCaps * caps) GstCaps * caps)
@ -737,7 +744,9 @@ gst_jitter_buffer_sink_setcaps (GstPad * pad, GstCaps * caps)
jitterbuffer = GST_RTP_JITTER_BUFFER (gst_pad_get_parent (pad)); jitterbuffer = GST_RTP_JITTER_BUFFER (gst_pad_get_parent (pad));
priv = jitterbuffer->priv; priv = jitterbuffer->priv;
JBUF_LOCK (priv);
res = gst_jitter_buffer_sink_parse_caps (jitterbuffer, caps); res = gst_jitter_buffer_sink_parse_caps (jitterbuffer, caps);
JBUF_UNLOCK (priv);
/* set same caps on srcpad on success */ /* set same caps on srcpad on success */
if (res) if (res)
@ -1032,6 +1041,10 @@ gst_rtp_jitter_buffer_sink_rtcp_event (GstPad * pad, GstEvent * event)
return TRUE; return TRUE;
} }
/*
* Must be called with JBUF_LOCK held
*/
static gboolean static gboolean
gst_rtp_jitter_buffer_get_clock_rate (GstRtpJitterBuffer * jitterbuffer, gst_rtp_jitter_buffer_get_clock_rate (GstRtpJitterBuffer * jitterbuffer,
guint8 pt) guint8 pt)
@ -1094,6 +1107,23 @@ gst_rtp_jitter_buffer_chain (GstPad * pad, GstBuffer * buffer)
pt = gst_rtp_buffer_get_payload_type (buffer); pt = gst_rtp_buffer_get_payload_type (buffer);
/* take the timestamp of the buffer. This is the time when the packet was
* received and is used to calculate jitter and clock skew. We will adjust
* this timestamp with the smoothed value after processing it in the
* jitterbuffer. */
timestamp = GST_BUFFER_TIMESTAMP (buffer);
/* bring to running time */
timestamp = gst_segment_to_running_time (&priv->segment, GST_FORMAT_TIME,
timestamp);
seqnum = gst_rtp_buffer_get_seq (buffer);
GST_DEBUG_OBJECT (jitterbuffer,
"Received packet #%d at time %" GST_TIME_FORMAT, seqnum,
GST_TIME_ARGS (timestamp));
JBUF_LOCK_CHECK (priv, out_flushing);
if (G_UNLIKELY (priv->last_pt != pt)) { if (G_UNLIKELY (priv->last_pt != pt)) {
GstCaps *caps; GstCaps *caps;
@ -1117,22 +1147,6 @@ gst_rtp_jitter_buffer_chain (GstPad * pad, GstBuffer * buffer)
goto no_clock_rate; goto no_clock_rate;
} }
/* take the timestamp of the buffer. This is the time when the packet was
* received and is used to calculate jitter and clock skew. We will adjust
* this timestamp with the smoothed value after processing it in the
* jitterbuffer. */
timestamp = GST_BUFFER_TIMESTAMP (buffer);
/* bring to running time */
timestamp = gst_segment_to_running_time (&priv->segment, GST_FORMAT_TIME,
timestamp);
seqnum = gst_rtp_buffer_get_seq (buffer);
GST_DEBUG_OBJECT (jitterbuffer,
"Received packet #%d at time %" GST_TIME_FORMAT, seqnum,
GST_TIME_ARGS (timestamp));
JBUF_LOCK_CHECK (priv, out_flushing);
/* don't accept more data on EOS */ /* don't accept more data on EOS */
if (G_UNLIKELY (priv->eos)) if (G_UNLIKELY (priv->eos))
goto have_eos; goto have_eos;
@ -1252,8 +1266,7 @@ no_clock_rate:
GST_WARNING_OBJECT (jitterbuffer, GST_WARNING_OBJECT (jitterbuffer,
"No clock-rate in caps!, dropping buffer"); "No clock-rate in caps!, dropping buffer");
gst_buffer_unref (buffer); gst_buffer_unref (buffer);
gst_object_unref (jitterbuffer); goto finished;
return GST_FLOW_OK;
} }
out_flushing: out_flushing:
{ {