mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-06-06 23:48:53 +00:00
Code Issues 1 Projects Releases Wiki Activity

stream: Add some missing API safe guards

Browse source
This commit is contained in:
Thibault Saunier 2018-06-13 16:27:24 -04:00
parent b0777476ff
commit 8833ca942e
1 changed files with 20 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
gst/gststreams.c
View file

@ -232,6 +232,8 @@ gst_stream_new (const gchar * stream_id, GstCaps * caps, GstStreamType type,
static void static void
gst_stream_set_stream_id (GstStream * stream, const gchar * stream_id) gst_stream_set_stream_id (GstStream * stream, const gchar * stream_id)
{ {
g_return_if_fail (GST_IS_STREAM (stream));
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
g_assert (stream->stream_id == NULL); g_assert (stream->stream_id == NULL);
if (stream_id) if (stream_id)
@ -262,6 +264,8 @@ gst_stream_set_stream_id (GstStream * stream, const gchar * stream_id)
const gchar * const gchar *
gst_stream_get_stream_id (GstStream * stream) gst_stream_get_stream_id (GstStream * stream)
{ {
g_return_val_if_fail (GST_IS_STREAM (stream), NULL);
return stream->stream_id; return stream->stream_id;
} }
@ -277,6 +281,8 @@ gst_stream_get_stream_id (GstStream * stream)
void void
gst_stream_set_stream_flags (GstStream * stream, GstStreamFlags flags) gst_stream_set_stream_flags (GstStream * stream, GstStreamFlags flags)
{ {
g_return_if_fail (GST_IS_STREAM (stream));
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
stream->priv->flags = flags; stream->priv->flags = flags;
GST_OBJECT_UNLOCK (stream); GST_OBJECT_UNLOCK (stream);
@ -300,6 +306,8 @@ gst_stream_get_stream_flags (GstStream * stream)
{ {
GstStreamFlags res; GstStreamFlags res;
g_return_val_if_fail (GST_IS_STREAM (stream), GST_STREAM_FLAG_NONE);
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
res = stream->priv->flags; res = stream->priv->flags;
GST_OBJECT_UNLOCK (stream); GST_OBJECT_UNLOCK (stream);
@ -319,6 +327,8 @@ gst_stream_get_stream_flags (GstStream * stream)
void void
gst_stream_set_stream_type (GstStream * stream, GstStreamType stream_type) gst_stream_set_stream_type (GstStream * stream, GstStreamType stream_type)
{ {
g_return_if_fail (GST_IS_STREAM (stream));
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
stream->priv->type = stream_type; stream->priv->type = stream_type;
GST_OBJECT_UNLOCK (stream); GST_OBJECT_UNLOCK (stream);
@ -342,6 +352,8 @@ gst_stream_get_stream_type (GstStream * stream)
{ {
GstStreamType res; GstStreamType res;
g_return_val_if_fail (GST_IS_STREAM (stream), GST_STREAM_TYPE_UNKNOWN);
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
res = stream->priv->type; res = stream->priv->type;
GST_OBJECT_UNLOCK (stream); GST_OBJECT_UNLOCK (stream);
@ -363,6 +375,8 @@ gst_stream_set_tags (GstStream * stream, GstTagList * tags)
{ {
gboolean notify = FALSE; gboolean notify = FALSE;
g_return_if_fail (GST_IS_STREAM (stream));
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
if (stream->priv->tags == NULL || tags == NULL if (stream->priv->tags == NULL || tags == NULL
|| !gst_tag_list_is_equal (stream->priv->tags, tags)) { || !gst_tag_list_is_equal (stream->priv->tags, tags)) {
@ -391,6 +405,8 @@ gst_stream_get_tags (GstStream * stream)
{ {
GstTagList *res = NULL; GstTagList *res = NULL;
g_return_val_if_fail (GST_IS_STREAM (stream), NULL);
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
if (stream->priv->tags) if (stream->priv->tags)
res = gst_tag_list_ref (stream->priv->tags); res = gst_tag_list_ref (stream->priv->tags);
@ -413,6 +429,8 @@ gst_stream_set_caps (GstStream * stream, GstCaps * caps)
{ {
gboolean notify = FALSE; gboolean notify = FALSE;
g_return_if_fail (GST_IS_STREAM (stream));
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
if (stream->priv->caps == NULL || (caps if (stream->priv->caps == NULL || (caps
&& !gst_caps_is_equal (stream->priv->caps, caps))) { && !gst_caps_is_equal (stream->priv->caps, caps))) {
@ -441,6 +459,8 @@ gst_stream_get_caps (GstStream * stream)
{ {
GstCaps *res = NULL; GstCaps *res = NULL;
g_return_val_if_fail (GST_IS_STREAM (stream), NULL);
GST_OBJECT_LOCK (stream); GST_OBJECT_LOCK (stream);
if (stream->priv->caps) if (stream->priv->caps)
res = gst_caps_ref (stream->priv->caps); res = gst_caps_ref (stream->priv->caps);