On Thu, 03 Aug 2023, Juha-Pekka Heikkila <[email protected]> wrote:
> If drm_edid_raw returned NULL on error don't try accessing anything behind
> that NULL
>
> Signed-off-by: Juha-Pekka Heikkila <[email protected]>

Reviewed-by: Jani Nikula <[email protected]>

> ---
>  drivers/gpu/drm/i915/display/intel_sdvo.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_sdvo.c 
> b/drivers/gpu/drm/i915/display/intel_sdvo.c
> index 8298a86d1334..065e619f934f 100644
> --- a/drivers/gpu/drm/i915/display/intel_sdvo.c
> +++ b/drivers/gpu/drm/i915/display/intel_sdvo.c
> @@ -2097,7 +2097,7 @@ intel_sdvo_tmds_sink_detect(struct drm_connector 
> *connector)
>               const struct edid *edid = drm_edid_raw(drm_edid);
>  
>               /* DDC bus is shared, match EDID to connector type */
> -             if (edid->input & DRM_EDID_INPUT_DIGITAL)
> +             if (edid && edid->input & DRM_EDID_INPUT_DIGITAL)
>                       status = connector_status_connected;
>               else
>                       status = connector_status_disconnected;

-- 
Jani Nikula, Intel Open Source Graphics Center

Reply via email to