diff options
| author | Wim Taymans <wim.taymans@collabora.co.uk> | 2009-01-22 13:33:14 +0100 |
|---|---|---|
| committer | Tim-Philipp Müller <tim.muller@collabora.co.uk> | 2009-08-11 02:30:40 +0100 |
| commit | 7f0b100db55ec912fb7d7c28530d402130055dbe (patch) | |
| tree | dc5f0a78f77ed3e724673079735dc622253fdfaa /gst/rtpmanager | |
| parent | dfdc9b66629f913571d181d4dc93ac35268e27b4 (diff) | |
Unlock the jitterbuffer before pushing out the packet-lost events.
Move some code before we do the unlock to make the jitterbuffer state
consistent while we are unlocked.
Diffstat (limited to 'gst/rtpmanager')
| -rw-r--r-- | gst/rtpmanager/gstrtpjitterbuffer.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/gst/rtpmanager/gstrtpjitterbuffer.c b/gst/rtpmanager/gstrtpjitterbuffer.c index 95cde211..2d3d4458 100644 --- a/gst/rtpmanager/gstrtpjitterbuffer.c +++ b/gst/rtpmanager/gstrtpjitterbuffer.c @@ -1442,6 +1442,11 @@ again: priv->num_late++; discont = TRUE; + /* update our expected next packet */ + priv->last_popped_seqnum = next_seqnum; + priv->last_out_time = out_time; + priv->next_seqnum = (next_seqnum + 1) & 0xffff; + if (priv->do_lost) { /* create paket lost event */ event = gst_event_new_custom (GST_EVENT_CUSTOM_DOWNSTREAM, @@ -1449,13 +1454,11 @@ again: "seqnum", G_TYPE_UINT, (guint) next_seqnum, "timestamp", G_TYPE_UINT64, out_time, "duration", G_TYPE_UINT64, duration, NULL)); + + JBUF_UNLOCK (priv); gst_pad_push_event (priv->srcpad, event); + JBUF_LOCK_CHECK (priv, flushing); } - - /* update our expected next packet */ - priv->last_popped_seqnum = next_seqnum; - priv->last_out_time = out_time; - priv->next_seqnum = (next_seqnum + 1) & 0xffff; /* look for next packet */ goto again; } |