rtpsrc: Fix wrong/NULL URI handling

We can reset the URI to NULL and this fix a deadlock in that case or
when the URI was invalid.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/2512>

1 files changed, 16 insertions, 3 deletions

diff --git a/gst/rtp/gstrtpsrc.c b/gst/rtp/gstrtpsrc.c
index 69a9caf8d..89ba91c04 100644
--- a/gst/rtp/gstrtpsrc.c
+++ b/gst/rtp/gstrtpsrc.c
@@ -165,16 +165,29 @@ gst_rtp_src_set_property (GObject * object, guint prop_id,
   switch (prop_id) {
     case PROP_URI:{
       GstUri *uri = NULL;
+      const gchar *str_uri = g_value_get_string (value);
 
       GST_RTP_SRC_LOCK (object);
-      uri = gst_uri_from_string (g_value_get_string (value));
-      if (uri == NULL)
-        break;
+      uri = gst_uri_from_string (str_uri);
+      if (uri == NULL) {
+        if (str_uri) {
+          GST_RTP_SRC_UNLOCK (object);
+          GST_ERROR_OBJECT (object, "Invalid uri: %s", str_uri);
+
+          break;
+        }
+      }
 
       if (self->uri)
         gst_uri_unref (self->uri);
       self->uri = uri;
 
+      if (!uri) {
+        GST_RTP_SRC_UNLOCK (object);
+
+        break;
+      }
+
       /* Recursive set to self, do not use the same lock in all property
        * setters. */
       g_object_set (self, "address", gst_uri_get_host (self->uri), NULL);