commit ec0b8dce50cf970f705a4dd1ef6a219b5a4abe04
author Brandon Wyman <bjwyman@gmail.com> Fri Oct 08 21:49:43 2021 +0000
committer Shawn McCarney <shawnmm@us.ibm.com> Thu Oct 28 17:22:41 2021 +0000
tree a56efa2e45707c0f9923c2e11464b2e4c65a2b9f
parent da369c792c06204a7e74ae56da3dcc741567f9ed

psu-ng: Clear the additionalData each loop

In the analyze() function of the psuManager, clear out the
additionalData each time we loop to the next power supply. If we do not
do this, and multiple power supplies encounter faults, call out data for
a different power supply may end up being added to the wrong error(s).

Tested:
    Simulated triple power supply fault while chassis power on.
    Verified correct call out details, extras noted earlier not there.
    Faults: psu0 comm fault, psu1 missing, psu3 UV.

Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I1d22dda450a08abc59b95c9de9c6750ac78a439f

phosphor-power-supply/psu_manager.cpp

diff --git a/phosphor-power-supply/psu_manager.cpp b/phosphor-power-supply/psu_manager.cpp
index 378c0c1..4cfa801 100644
--- a/phosphor-power-supply/psu_manager.cpp
+++ b/phosphor-power-supply/psu_manager.cpp
@@ -377,6 +377,7 @@
 
         for (auto& psu : psus)
         {
+            additionalData.clear();
             // TODO: Fault priorities #918
             if (!psu->isFaultLogged() && !psu->isPresent())
             {