rtpjitterbuffer: Make it more explicit that update_rtx_timers() takes ownership of...
authorSebastian Dröge <sebastian@centricular.com>
Fri, 2 Sep 2022 09:20:58 +0000 (12:20 +0300)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Sat, 3 Sep 2022 09:26:24 +0000 (09:26 +0000)
It is not valid anymore afterwards and must not be used, otherwise an
already freed pointer might be used.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2973>

subprojects/gst-plugins-good/gst/rtpmanager/gstrtpjitterbuffer.c

index 1770781..0f0c4c9 100644 (file)
@@ -3534,7 +3534,7 @@ gst_rtp_jitter_buffer_chain (GstPad * pad, GstObject * parent,
   /* update rtx timers */
   if (priv->do_retransmission)
     update_rtx_timers (jitterbuffer, seqnum, dts, pts, do_next_seqnum, is_rtx,
-        timer);
+        g_steal_pointer (&timer));
 
   /* we had an unhandled SR, handle it now */
   if (priv->last_sr)