Commit 08c0b7a4 authored by Alex Hung's avatar Alex Hung Committed by Xiaomeng Zhang
Browse files

drm/amd/display: Check null pointer before dereferencing se 

mainline inclusion
from mainline-v6.12-rc1
commit ff599ef6970ee000fa5bc38d02fa5ff5f3fc7575
category: bugfix
bugzilla: https://gitee.com/src-openeuler/kernel/issues/IAZM9W

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=65b2d49e55fe13ae56da3a7685bdccadca31134a



--------------------------------

[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: default avatarAlex Hung <alex.hung@amd.com>
Reviewed-by: default avatarRodrigo Siqueira <rodrigo.siqueira@amd.com>
Signed-off-by: default avatarAlex Hung <alex.hung@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarXiaomeng Zhang <zhangxiaomeng13@huawei.com>
parent 27e5d43e

drivers/gpu/drm/amd/display/dc/core/dc.c

+1 −1
Original line number Diff line number Diff line
@@ -1691,7 +1691,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(