[bug report] drm/amd/display: switch DC over to the new DRM logging macros

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Hamza Mahfooz,

The patch 5d72e247e58c: "drm/amd/display: switch DC over to the new
DRM logging macros" from Sep 20, 2023 (linux-next), leads to the
following Smatch static checker warning:

	drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_helpers.c:542 dm_helpers_dp_read_dpcd()
	error: we previously assumed 'aconnector' could be null (see line 541)

drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_helpers.c
    531 bool dm_helpers_dp_read_dpcd(
    532                 struct dc_context *ctx,
    533                 const struct dc_link *link,
    534                 uint32_t address,
    535                 uint8_t *data,
    536                 uint32_t size)
    537 {
    538 
    539         struct amdgpu_dm_connector *aconnector = link->priv;
    540 
    541         if (!aconnector) {
                     ^^^^^^^^^^
Check for NULL

--> 542                 drm_dbg_dp(aconnector->base.dev,
                                   ^^^^^^^^^^^^^^^^^^^^
NULL dereference.

    543                            "Failed to find connector for link!\n");
    544                 return false;
    545         }
    546 
    547         return drm_dp_dpcd_read(&aconnector->dm_dp_aux.aux, address, data,
    548                                 size) == size;
    549 }

regards,
dan carpenter



[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux