diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2025-09-27 15:26:34 +0300 |
|---|---|---|
| committer | Borislav Petkov (AMD) <bp@alien8.de> | 2025-10-13 17:14:47 +0200 |
| commit | 79c0a2b7abc906c7cf3c793256c6b638d7dc477f (patch) | |
| tree | d0e994821c6687424244c52522d876f5d2d7a37e | |
| parent | 3a8660878839faadb4f1a6dd72c3179c1df56787 (diff) | |
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: d5fe2fec6c40 ("EDAC: Add a driver for the AMD Versal NET DDR controller")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Reviewed-by: Yazen Ghannam <yazen.ghannam@amd.com>
| -rw-r--r-- | drivers/edac/versalnet_edac.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/edac/versalnet_edac.c b/drivers/edac/versalnet_edac.c index 7c5db8bf0595..1ded4c3f0213 100644 --- a/drivers/edac/versalnet_edac.c +++ b/drivers/edac/versalnet_edac.c @@ -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]; |