tree 2aeabdab3a297e98160fe60a7f0c0c8bc3a3e310
parent 14457704e4fb3b4374cb61373c245a87d685f7a4
author Vishnunithyasoundhar S <vishnunithyasoundhar.s@intel.com> 1777353622 +0530
committer ManojKiran Eda <manojkiran.eda@gmail.com> 1777550874 +0000

platform-mc: fix pldm crash due to  use-after-free on terminus

initTerminus() and configEventReceiver() iterate over or hold
references into the termini map while suspending at co_await points.
A concurrent removeMctpTerminus() call can erase map entries during
suspension, invalidating iterators and dangling references.

Fix by:
- Snapshotting TIDs into a vector before iterating so the loop is
  not affected by map mutations.
- Taking a local shared_ptr copy of the terminus so the Terminus
  object stays alive even if the map entry is erased while suspended.
- Adding termini.contains(tid) guards after each co_await suspension
  point to skip further processing when a terminus is removed.

Change-Id: I3fa6dfbd0c0b077381bfa5429c531c856de79919
Signed-off-by: Vishnunithyasoundhar S <vishnunithyasoundhar.s@intel.com>
Co-developed-by: Arun P. Mohanan <arun.p.m@intel.com>