From 648b8f3362453fb39ac8f1e87b83d5ef46f90189 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com>
Date: Fri, 2 Sep 2022 12:20:58 +0300
Subject: [PATCH] rtpjitterbuffer: Make it more explicit that
 update_rtx_timers() takes ownership of the passed in timer

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>
---
 .../gst-plugins-good/gst/rtpmanager/gstrtpjitterbuffer.c        | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/subprojects/gst-plugins-good/gst/rtpmanager/gstrtpjitterbuffer.c b/subprojects/gst-plugins-good/gst/rtpmanager/gstrtpjitterbuffer.c
index 177078112c..0f0c4c9b8e 100644
--- a/subprojects/gst-plugins-good/gst/rtpmanager/gstrtpjitterbuffer.c
+++ b/subprojects/gst-plugins-good/gst/rtpmanager/gstrtpjitterbuffer.c
@@ -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)