[bug report] drm/amd/display: Fix allocate_mst_payload assert on resume

Dan Carpenter dan.carpenter at oracle.com
Tue Apr 5 08:18:07 UTC 2022


Hello Roman Li,

This is a semi-automatic email about new static checker warnings.

The patch c4af42b4eb5b: "drm/amd/display: Fix allocate_mst_payload
assert on resume" from Mar 17, 2022, leads to the following Smatch
complaint:

    drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:2722 dm_resume()
    error: we previously assumed 'aconnector->dc_link' could be null (see line 2717)

drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c
  2716			 */
  2717			if (aconnector->dc_link &&
                            ^^^^^^^^^^^^^^^^^^^
Patch adds a check for NULL

  2718			    aconnector->dc_link->type == dc_connection_mst_branch)
  2719				continue;
  2720	
  2721			mutex_lock(&aconnector->hpd_lock);
  2722			if (!dc_link_detect_sink(aconnector->dc_link, &new_connection_type))
                                                 ^^^^^^^^^^^^^^^^^^^
If aconnector->dc_link is NULL this will crash inside the function call.

  2723				DRM_ERROR("KMS: Failed to detect connector\n");
  2724	

regards,
dan carpenter


More information about the amd-gfx mailing list