rtsp-client: fix leak adding headers
authorDoug Nazar <nazard@nazar.ca>
Wed, 5 May 2021 00:45:19 +0000 (20:45 -0400)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Wed, 5 May 2021 10:02:48 +0000 (10:02 +0000)
gst_rtsp_message_add_header() makes a copy of the header, instead
of taking ownership.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-rtsp-server/-/merge_requests/204>

gst/rtsp-server/rtsp-client.c

index 8312e3c..e5a62c0 100644 (file)
@@ -3018,7 +3018,8 @@ handle_setup_request (GstRTSPClient * client, GstRTSPContext * ctx)
           (gdouble) seekable / GST_SECOND);
 
     gst_rtsp_message_add_header (ctx->response, GST_RTSP_HDR_MEDIA_PROPERTIES,
-        g_string_free (media_properties, FALSE));
+        media_properties->str);
+    g_string_free (media_properties, TRUE);
     /* TODO Check how Accept-Ranges should be filled */
     gst_rtsp_message_add_header (ctx->request, GST_RTSP_HDR_ACCEPT_RANGES,
         "npt, clock, smpte, clock");