diff --git a/gst/gsturi.c b/gst/gsturi.c index 10214359b3..cd736fbcc4 100644 --- a/gst/gsturi.c +++ b/gst/gsturi.c @@ -405,8 +405,8 @@ gst_uri_get_protocol (const gchar * uri) { gchar *colon; - g_return_val_if_fail (uri != NULL, NULL); - g_return_val_if_fail (gst_uri_is_valid (uri), NULL); + if (!gst_uri_is_valid (uri)) + return NULL; colon = strstr (uri, ":"); @@ -427,9 +427,11 @@ gst_uri_has_protocol (const gchar * uri, const gchar * protocol) { gchar *colon; - g_return_val_if_fail (uri != NULL, FALSE); g_return_val_if_fail (protocol != NULL, FALSE); - g_return_val_if_fail (gst_uri_is_valid (uri), FALSE); + + if (!gst_uri_is_valid (uri)) { + return FALSE; + } colon = strstr (uri, ":"); @@ -460,8 +462,9 @@ gst_uri_get_location (const gchar * uri) const gchar *colon; gchar *unescaped = NULL; - g_return_val_if_fail (uri != NULL, NULL); - g_return_val_if_fail (gst_uri_is_valid (uri), NULL); + if (!gst_uri_is_valid (uri)) { + return NULL; + } colon = strstr (uri, "://"); if (!colon) @@ -632,9 +635,12 @@ gst_element_make_from_uri (const GstURIType type, const gchar * uri, g_return_val_if_fail (gst_is_initialized (), NULL); g_return_val_if_fail (GST_URI_TYPE_IS_VALID (type), NULL); - g_return_val_if_fail (gst_uri_is_valid (uri), NULL); g_return_val_if_fail (error == NULL || *error == NULL, NULL); + if (!gst_uri_is_valid (uri)) { + return NULL; + } + GST_DEBUG ("type:%d, uri:%s, elementname:%s", type, uri, elementname); protocol = gst_uri_get_protocol (uri); @@ -796,13 +802,16 @@ gst_uri_handler_set_uri (GstURIHandler * handler, const gchar * uri, gchar *protocol; g_return_val_if_fail (GST_IS_URI_HANDLER (handler), FALSE); - g_return_val_if_fail (gst_uri_is_valid (uri), FALSE); g_return_val_if_fail (error == NULL || *error == NULL, FALSE); iface = GST_URI_HANDLER_GET_INTERFACE (handler); g_return_val_if_fail (iface != NULL, FALSE); g_return_val_if_fail (iface->set_uri != NULL, FALSE); + if (!gst_uri_is_valid (uri)) { + return FALSE; + } + protocol = gst_uri_get_protocol (uri); if (iface->get_protocols) {