ArmPkg/ArmGicLib: use correct loop variable

The ArmGicLib API function GicGetCpuRedistributorBase () declares
GicCpuRedistributorBase to iterate over the redistributors of all
CPUs, but then inadvertently advances GicRedistributorBase instead.

Reported-by: "Oliyil Kunnil, Vishal" <vishalo@qti.qualcomm.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

diff --git a/ArmPkg/Drivers/ArmGic/ArmGicLib.c b/ArmPkg/Drivers/ArmGic/ArmGicLib.c
index 73795ed4e56c41a67f7d3778b03bce20db908d10..e658e9bff5d8107b3914bdf1e9e1e51a4e4d4cd7 100644 (file)
--- a/ArmPkg/Drivers/ArmGic/ArmGicLib.c
+++ b/ArmPkg/Drivers/ArmGic/ArmGicLib.c
@@ -76,7 +76,7 @@ GicGetCpuRedistributorBase (
     }\r
 \r
     // Move to the next GIC Redistributor frame\r
-    GicRedistributorBase += GicRedistributorGranularity;\r
+    GicCpuRedistributorBase += GicRedistributorGranularity;\r
   }\r
 \r
   // The Redistributor has not been found for the current CPU\r