projects / platform / upstream / gst-plugins-good.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: b3e0b1d) | patch
rtpjitterbuffer: fix crash when do-retransmission=true and a lot of buffers are lost
authorGeorge Kiagiadakis <george.kiagiadakis@collabora.com>
Tue, 5 Nov 2013 08:48:33 +0000 (10:48 +0200)
committerWim Taymans <wim.taymans@gmail.com>
Mon, 11 Nov 2013 10:51:45 +0000 (11:51 +0100)
commitb81b2efa3e4905f11136930da5191b1ac033825e
treee057e6f0d51c55a4fc3e61d9b7f21a1e0663dcb6tree | snapshot
parentb3e0b1dbcaa452c5419b3ad254a0f8cca6167f4acommit | diff
rtpjitterbuffer: fix crash when do-retransmission=true and a lot of buffers are lost

The problem here was that the jitterbuffer lock was unlocked to push
the event, but that caused another thread to remove the timer currently
being processed, probably because the amount of rtx events
(and therefore timers) was getting too high. The solution is to
unlock and push the event only after timer processing has finished.

fixes https://bugzilla.gnome.org/show_bug.cgi?id=711131
gst/rtpmanager/gstrtpjitterbuffer.c diff | blob | history
Domain: Multimedia / Media Common;