[Public] Reviewed-by: Tim Huang <tim.hu...@amd.com>
Best Regards, Tim Huang > -----Original Message----- > From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of Ma Jun > Sent: Tuesday, May 7, 2024 11:19 AM > To: amd-gfx@lists.freedesktop.org; Koenig, Christian > <christian.koe...@amd.com>; Deucher, Alexander > <alexander.deuc...@amd.com> > Cc: Ma, Jun <jun....@amd.com> > Subject: [PATCH] drm/amdgpu: Fix out-of-bounds read of > df_v1_7_channel_number > > Check the fb_channel_number range to avoid the array out-of-bounds read error > > Signed-off-by: Ma Jun <jun....@amd.com> > --- > drivers/gpu/drm/amd/amdgpu/df_v1_7.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/df_v1_7.c > b/drivers/gpu/drm/amd/amdgpu/df_v1_7.c > index 5dfab80ffff2..cd298556f7a6 100644 > --- a/drivers/gpu/drm/amd/amdgpu/df_v1_7.c > +++ b/drivers/gpu/drm/amd/amdgpu/df_v1_7.c > @@ -70,6 +70,8 @@ static u32 df_v1_7_get_hbm_channel_number(struct > amdgpu_device *adev) > int fb_channel_number; > > fb_channel_number = adev->df.funcs->get_fb_channel_number(adev); > + if (fb_channel_number >= ARRAY_SIZE(df_v1_7_channel_number)) > + fb_channel_number = 0; > > return df_v1_7_channel_number[fb_channel_number]; > } > -- > 2.34.1
