[Intel-gfx] [bug report] drm/i915: Replace some gamma_mode ifs with switches
Dan Carpenter
dan.carpenter at oracle.com
Mon Nov 2 13:07:36 UTC 2020
Hello Ville Syrjälä,
This is a semi-automatic email about new static checker warnings.
The patch 7852ddd5d60a: "drm/i915: Replace some gamma_mode ifs with
switches" from Sep 25, 2020, leads to the following Smatch complaint:
drivers/gpu/drm/i915/display/intel_color.c:765 ivb_load_luts()
error: we previously assumed 'gamma_lut' could be null (see line 755)
drivers/gpu/drm/i915/display/intel_color.c
754 const struct drm_property_blob *degamma_lut = crtc_state->hw.degamma_lut;
755 const struct drm_property_blob *blob = gamma_lut ?: degamma_lut;
^^^^^^^^^^^^
Check for NULL
756
757 switch (crtc_state->gamma_mode) {
758 case GAMMA_MODE_MODE_8BIT:
759 ilk_load_lut_8(crtc, blob);
760 break;
761 case GAMMA_MODE_MODE_SPLIT:
762 ivb_load_lut_10(crtc, degamma_lut, PAL_PREC_SPLIT_MODE |
763 PAL_PREC_INDEX_VALUE(0));
764 ivb_load_lut_ext_max(crtc_state);
765 ivb_load_lut_10(crtc, gamma_lut, PAL_PREC_SPLIT_MODE |
^^^^^^^^^
Unchecked dereference. Possibly pass "blob" here?
766 PAL_PREC_INDEX_VALUE(512));
767 break;
regards,
dan carpenter
More information about the Intel-gfx
mailing list