[Public] The series Reviewed-by: Roman Li <[email protected]>
> -----Original Message----- > From: SHANMUGAM, SRINIVASAN <[email protected]> > Sent: Thursday, October 17, 2024 2:56 AM > To: Siqueira, Rodrigo <[email protected]>; Pillai, Aurabindo > <[email protected]> > Cc: [email protected]; SHANMUGAM, SRINIVASAN > <[email protected]>; Chung, ChiaHsuan (Tom) > <[email protected]>; Li, Roman <[email protected]>; Hung, Alex > <[email protected]>; Wentland, Harry <[email protected]>; Mahfooz, > Hamza <[email protected]> > 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 <[email protected]> > Cc: Rodrigo Siqueira <[email protected]> > Cc: Roman Li <[email protected]> > Cc: Alex Hung <[email protected]> > Cc: Aurabindo Pillai <[email protected]> > Cc: Harry Wentland <[email protected]> > Cc: Hamza Mahfooz <[email protected]> > Signed-off-by: Srinivasan Shanmugam <[email protected]> > --- > .../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
