Reviewed-by: Tao Zhou <[email protected]>
> -----Original Message----- > From: amd-gfx <[email protected]> On Behalf Of > Guchun Chen > Sent: 2019年8月8日 14:59 > To: [email protected]; Zhang, Hawking > <[email protected]>; Li, Dennis <[email protected]>; Pan, Xinhui > <[email protected]>; Zhou1, Tao <[email protected]> > Cc: Chen, Guchun <[email protected]> > Subject: [PATCH] drm/amdgpu: add check to avoid array bound issue > > Sub_block_index can be passed from user level, so add one check before > accessing the array first to prevent array index out of bound problem. > > Change-Id: I556fc560b44215848ba3a95f757febdf3d0af422 > Signed-off-by: Guchun Chen <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > index 3e46cce60553..59c77b01c88a 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > @@ -5960,6 +5960,9 @@ static int gfx_v9_0_ras_error_inject(struct > amdgpu_device *adev, > if (adev->asic_type != CHIP_VEGA20) > return -EINVAL; > > + if (info->head.sub_block_index >= ARRAY_SIZE(ras_gfx_subblocks)) > + return -EINVAL; > + > if (!ras_gfx_subblocks[info->head.sub_block_index].name) > return -EPERM; > > -- > 2.17.1 > > _______________________________________________ > amd-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
