[AMD Official Use Only - General]

Reviewed-by: Hawking Zhang <[email protected]>

Regards,
Hawking
-----Original Message-----
From: amd-gfx <[email protected]> On Behalf Of Lijo Lazar
Sent: Tuesday, May 23, 2023 16:00
To: [email protected]
Cc: Deucher, Alexander <[email protected]>; Dan Carpenter 
<[email protected]>; kernel test robot <[email protected]>; Zhang, Hawking 
<[email protected]>
Subject: [PATCH] drm/amdgpu: Fix warnings

Fix warnings reported by kernel test bot/smatch

smatch warnings:
drivers/gpu/drm/amd/amdgpu/amdgpu_xcp.c:65 amdgpu_xcp_run_transition()
error: buffer overflow 'xcp_mgr->xcp' 8 <= 8

Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>
Closes: https://lore.kernel.org/r/[email protected]/

Signed-off-by: Lijo Lazar <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_xcp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_xcp.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_xcp.c
index e9586a0dc335..fcdc0862d258 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_xcp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_xcp.c
@@ -65,7 +65,7 @@ static int amdgpu_xcp_run_transition(struct amdgpu_xcp_mgr 
*xcp_mgr, int xcp_id,
        struct amdgpu_xcp *xcp;
        int i, ret;
 
-       if (xcp_id > MAX_XCP || !xcp_mgr->xcp[xcp_id].valid)
+       if (xcp_id >= MAX_XCP || !xcp_mgr->xcp[xcp_id].valid)
                return -EINVAL;
 
        xcp = &xcp_mgr->xcp[xcp_id];
-- 
2.25.1

Reply via email to