diff options
| author | Alex Hung <alex.hung@amd.com> | 2024-08-29 17:30:26 -0600 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2024-09-18 16:15:08 -0400 |
| commit | ff599ef6970ee000fa5bc38d02fa5ff5f3fc7575 (patch) | |
| tree | 3b2e8c0cbaa86bacd0c173d6f7c787fe8f95427f /drivers/gpu | |
| parent | ce83ae29f93772d604b4ea73459fb17822d6a6b0 (diff) | |
| download | linux-ff599ef6970ee000fa5bc38d02fa5ff5f3fc7575.tar.gz linux-ff599ef6970ee000fa5bc38d02fa5ff5f3fc7575.tar.bz2 linux-ff599ef6970ee000fa5bc38d02fa5ff5f3fc7575.zip | |
drm/amd/display: Check null pointer before dereferencing se
[WHAT & HOW]
se is null checked previously in the same function, indicating
it might be null; therefore, it must be checked when used again.
This fixes 1 FORWARD_NULL issue reported by Coverity.
Acked-by: Alex Hung <alex.hung@amd.com>
Reviewed-by: Rodrigo Siqueira <rodrigo.siqueira@amd.com>
Signed-off-by: Alex Hung <alex.hung@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/amd/display/dc/core/dc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c index eb7c7681bdd9..67812fbbb006 100644 --- a/drivers/gpu/drm/amd/display/dc/core/dc.c +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c @@ -1767,7 +1767,7 @@ bool dc_validate_boot_timing(const struct dc *dc, if (crtc_timing->pix_clk_100hz != pix_clk_100hz) return false; - if (!se->funcs->dp_get_pixel_format) + if (!se || !se->funcs->dp_get_pixel_format) return false; if (!se->funcs->dp_get_pixel_format( |