[Cogl] [PATCH 2/3] cogl-gst: Fix get_caps implementation when no context is set
Lionel Landwerlin
llandwerlin at gmail.com
Wed Jan 29 02:12:46 PST 2014
This patch looks good to me.
-
Lionel
On 21/01/14 17:33, Neil Roberts wrote:
> If no context is set on the CoglGstVideoSink then it would previously
> call gst_caps_ref with a NULL pointer. This patch makes it just return
> NULL instead. I think that is a valid thing to do because that is what
> gst_base_sink_default_get_caps does. If we don't do this then it's not
> possible to use CoglGstVideoSink with GstParse because that tries to
> link the pipeline after parsing the string. That was previously
> causing a critical error because the freshly parsed sink doesn't have
> a CoglContext.
> ---
> cogl-gst/cogl-gst-video-sink.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/cogl-gst/cogl-gst-video-sink.c b/cogl-gst/cogl-gst-video-sink.c
> index 7708f59..9b217d4 100644
> --- a/cogl-gst/cogl-gst-video-sink.c
> +++ b/cogl-gst/cogl-gst-video-sink.c
> @@ -1035,7 +1035,11 @@ cogl_gst_video_sink_get_caps (GstBaseSink *bsink,
> {
> CoglGstVideoSink *sink;
> sink = COGL_GST_VIDEO_SINK (bsink);
> - return gst_caps_ref (sink->priv->caps);
> +
> + if (sink->priv->caps == NULL)
> + return NULL;
> + else
> + return gst_caps_ref (sink->priv->caps);
> }
>
> static CoglBool
More information about the Cogl
mailing list