commit 9fa54b52f83c00fd713085e6849d3f261672d008
author Zev Weiss <zev@bewilderbeest.net> Wed Sep 02 21:20:33 2020 +0000
committer Jason Bills <jason.m.bills@linux.intel.com> Tue Oct 20 22:35:57 2020 +0000
tree eb8bffbed26d2bf0bddf288761452d82c13ee90f
parent bce86a63643e1d7cdf6d42e143738013ee47a8da

Fix possible infinite-loop in getCPUBusNums()

Due to a variable-name typo, if cpuBusNum1 had a non-zero value, the
loop adding each byte to cpu.cpuBusNums would never terminate.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: I1c2c9bedffac0dbd00c9d4e6de54a6331919ca80

src/peci_pcie.cpp

diff --git a/src/peci_pcie.cpp b/src/peci_pcie.cpp
index fa42ea6..a41cbcf 100644
--- a/src/peci_pcie.cpp
+++ b/src/peci_pcie.cpp
@@ -120,7 +120,7 @@
                     size_t busNum = cpuBusNum1 & 0xFF;
                     cpu.cpuBusNums.insert(busNum);
                     // Shift right by one byte
-                    cpuBusNum >>= 8;
+                    cpuBusNum1 >>= 8;
                 }
                 cpu.skipCpuBuses = true;
             }