Commit 79c0a2b7 authored by Dan Carpenter's avatar Dan Carpenter Committed by Borislav Petkov (AMD)
Browse files

EDAC/versalnet: Fix off by one in handle_error() 



The priv->mci[] array has NUM_CONTROLLERS so this > comparison needs to be >=
to prevent an out of bounds access.

Fixes: d5fe2fec ("EDAC: Add a driver for the AMD Versal NET DDR controller")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: default avatarBorislav Petkov (AMD) <bp@alien8.de>
Reviewed-by: default avatarYazen Ghannam <yazen.ghannam@amd.com>
parent 3a866087

drivers/edac/versalnet_edac.c

+1 −1
Original line number Diff line number Diff line
@@ -433,7 +433,7 @@ static void handle_error(struct mc_priv *priv, struct ecc_status *stat, 
	phys_addr_t pfn;

	int err;



	if (WARN_ON_ONCE(ctl_num > NUM_CONTROLLERS))

	if (WARN_ON_ONCE(ctl_num >= NUM_CONTROLLERS))

		return;



	mci = priv->mci[ctl_num];