From 39f2c932ee8ef2f61e6980db15097053660a5aa4 Mon Sep 17 00:00:00 2001 From: "U. Artie Eoff" Date: Fri, 21 Feb 2020 06:54:47 -0800 Subject: [PATCH] vaapipostproc: only set VPP colorimetry when VPP is available If we don't have functional vpp then we should not call gst_vaapi_filter_set_colorimetry. --- gst/vaapi/gstvaapipostproc.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/gst/vaapi/gstvaapipostproc.c b/gst/vaapi/gstvaapipostproc.c index 31c35f0246..0fb75b8206 100644 --- a/gst/vaapi/gstvaapipostproc.c +++ b/gst/vaapi/gstvaapipostproc.c @@ -1654,11 +1654,14 @@ gst_vaapipostproc_set_caps (GstBaseTransform * trans, GstCaps * caps, gst_vaapipostproc_set_passthrough (trans); } - ret = gst_vaapi_filter_set_colorimetry (postproc->filter, - &GST_VIDEO_INFO_COLORIMETRY (GST_VAAPI_PLUGIN_BASE_SINK_PAD_INFO - (postproc)), - &GST_VIDEO_INFO_COLORIMETRY (GST_VAAPI_PLUGIN_BASE_SRC_PAD_INFO - (postproc))); + if (postproc->has_vpp && !gst_vaapi_filter_set_colorimetry (postproc->filter, + &GST_VIDEO_INFO_COLORIMETRY (GST_VAAPI_PLUGIN_BASE_SINK_PAD_INFO + (postproc)), + &GST_VIDEO_INFO_COLORIMETRY (GST_VAAPI_PLUGIN_BASE_SRC_PAD_INFO + (postproc)))) + goto done; + + ret = TRUE; done: g_mutex_unlock (&postproc->postproc_lock);