[PATCH 3/3] drm/amd/display: Add hpd_source index check for dcn401 link encoder setup

[Li, Roman]

[Public]

The series
Reviewed-by: Roman Li <roman.li at amd.com>

> -----Original Message-----
> From: SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM at amd.com>
> Sent: Thursday, October 17, 2024 2:56 AM
> To: Siqueira, Rodrigo <Rodrigo.Siqueira at amd.com>; Pillai, Aurabindo
> <Aurabindo.Pillai at amd.com>
> Cc: amd-gfx at lists.freedesktop.org; SHANMUGAM, SRINIVASAN
> <SRINIVASAN.SHANMUGAM at amd.com>; Chung, ChiaHsuan (Tom)
> <ChiaHsuan.Chung at amd.com>; Li, Roman <Roman.Li at amd.com>; Hung, Alex
> <Alex.Hung at amd.com>; Wentland, Harry <Harry.Wentland at amd.com>; Mahfooz,
> Hamza <Hamza.Mahfooz at amd.com>
> Subject: [PATCH 3/3] drm/amd/display: Add hpd_source index check for dcn401
> link encoder setup
>
> This patch adds a boundary check for the hpd_source index during the link encoder
> creation process for all dcn401 ip. The check ensures that the index is within the
> valid range of the link_enc_hpd_regs array to prevent out-of-bounds access.
>
> Cc: Tom Chung <chiahsuan.chung at amd.com>
> Cc: Rodrigo Siqueira <Rodrigo.Siqueira at amd.com>
> Cc: Roman Li <roman.li at amd.com>
> Cc: Alex Hung <alex.hung at amd.com>
> Cc: Aurabindo Pillai <aurabindo.pillai at amd.com>
> Cc: Harry Wentland <harry.wentland at amd.com>
> Cc: Hamza Mahfooz <hamza.mahfooz at amd.com>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
> ---
>  .../gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c    | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c
> b/drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c
> index 46feff9348d9..b425b34dc7f5 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c
> @@ -1032,7 +1032,7 @@ static struct link_encoder
> *dcn401_link_encoder_create(
>       struct dcn20_link_encoder *enc20 =
>               kzalloc(sizeof(struct dcn20_link_encoder), GFP_KERNEL);
>
> -     if (!enc20)
> +     if (!enc20 || enc_init_data->hpd_source >=
> +ARRAY_SIZE(link_enc_hpd_regs))
>               return NULL;
>
>  #undef REG_STRUCT
> --
> 2.34.1