From: Jonathan Cavitt Date: Mon, 2 Feb 2026 21:47:10 +0000 (+0000) Subject: drm/colorop: Check if getting curve_1d_type default succeeds X-Git-Tag: v7.2-rc1~141^2~26^2~62 X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=51c0256409eb218b499166068f863c754a672a11;p=thirdparty%2Fkernel%2Flinux.git drm/colorop: Check if getting curve_1d_type default succeeds Static analysis issue: In all other uses of the function drm_object_property_get_default_value, the return value of the function is checked before the output is saved to the relevant object parameter. Though likely unnecessary given the execution path involved, keep the behavior consistent across uses and only set colorop_state->curve_1d_type in __drm_colorop_state_reset if drm_object_property_get_default_value succeeds. Signed-off-by: Jonathan Cavitt Reviewed-by: Chaitanya Kumar Borah Link: https://patch.msgid.link/20260202214709.8037-2-jonathan.cavitt@intel.com --- diff --git a/drivers/gpu/drm/drm_colorop.c b/drivers/gpu/drm/drm_colorop.c index 566816e3c6f0c..764d120606665 100644 --- a/drivers/gpu/drm/drm_colorop.c +++ b/drivers/gpu/drm/drm_colorop.c @@ -516,10 +516,10 @@ static void __drm_colorop_state_reset(struct drm_colorop_state *colorop_state, colorop_state->bypass = true; if (colorop->curve_1d_type_property) { - drm_object_property_get_default_value(&colorop->base, - colorop->curve_1d_type_property, - &val); - colorop_state->curve_1d_type = val; + if (!drm_object_property_get_default_value(&colorop->base, + colorop->curve_1d_type_property, + &val)) + colorop_state->curve_1d_type = val; } }