On Thu, 2023-08-03 at 18:24 +0300, Jani Nikula wrote: > On Thu, 03 Aug 2023, Juha-Pekka Heikkila > <juhapekka.heikkila@xxxxxxxxx> wrote: > > If drm_edid_raw returned NULL on error don't try accessing anything > > behind > > that NULL > > > > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@xxxxxxxxx> > > Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> Thank you for the patch and the review. This is now merged. BR, Jouni Högander > > > --- > > 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; >
