Commit a7b57372 authored by Dan Carpenter's avatar Dan Carpenter Committed by Borislav Petkov (AMD)

RAS/AMD/ATL: Fix array overflow in get_logical_coh_st_fabric_id_mi300()

Check against ARRAY_SIZE() which is the number of elements instead of
sizeof() which is the number of bytes.

Fixes: 453f0ae7 ("RAS/AMD/ATL: Add MI300 support")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: default avatarBorislav Petkov (AMD) <bp@alien8.de>
Link: https://lore.kernel.org/r/279c8b5e-6c00-467a-9071-9c67926abea4@moroto.mountain
parent 453f0ae7
...@@ -405,7 +405,7 @@ static const u16 phy_to_log_coh_st_map_mi300[] = { ...@@ -405,7 +405,7 @@ static const u16 phy_to_log_coh_st_map_mi300[] = {
static u16 get_logical_coh_st_fabric_id_mi300(struct addr_ctx *ctx) static u16 get_logical_coh_st_fabric_id_mi300(struct addr_ctx *ctx)
{ {
if (ctx->inst_id >= sizeof(phy_to_log_coh_st_map_mi300)) { if (ctx->inst_id >= ARRAY_SIZE(phy_to_log_coh_st_map_mi300)) {
atl_debug(ctx, "Instance ID out of range"); atl_debug(ctx, "Instance ID out of range");
return ~0; return ~0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment