Google GitGoogle Git
Sign in
gbmc/pldm/320cf8bd968210ad0364c50b45f2e31bd44e01be
commit
320cf8bd968210ad0364c50b45f2e31bd44e01be
[log]
author
Vishnunithyasoundhar S <vishnunithyasoundhar.s@intel.com>
Tue Apr 28 10:50:22 2026 +0530
committer
ManojKiran Eda <manojkiran.eda@gmail.com>
Thu Apr 30 12:07:54 2026 +0000
tree
2aeabdab3a297e98160fe60a7f0c0c8bc3a3e310
parent
14457704e4fb3b4374cb61373c245a87d685f7a4 [diff]
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>
1 file changed
tree: 2aeabdab3a297e98160fe60a7f0c0c8bc3a3e310
  1. common/
  2. configurations/
  3. docs/
  4. fw-update/
  5. host-bmc/
  6. libpldmresponder/
  7. oem/
  8. platform-mc/
  9. pldmd/
  10. pldmtool/
  11. requester/
  12. softoff/
  13. subprojects/
  14. test/
  15. tools/
  16. utilities/
  17. .clang-format
  18. .clang-tidy
  19. .eslintignore
  20. .gitignore
  21. .linter-ignore
  22. LICENSE
  23. meson.build
  24. meson.options
  25. OWNERS
  26. README.md
README.md

PLDM - Platform Level Data Model

License

Overview

PLDM (Platform Level Data Model) is a key component of the OpenBMC project, providing a standardized data model and message formats for various platform management functionalities. It defines a method to manage, monitor, and control the firmware and hardware of a system.

The OpenBMC PLDM project aims to implement the specifications defined by the Distributed Management Task Force (DMTF), allowing for interoperable management interfaces across different hardware and firmware components.

Features

  • Standardized Messaging: Adheres to the DMTF's PLDM specifications, enabling consistent and interoperable communication between different components.
  • Modularity: Supports multiple PLDM types, including base, FRU,Firmware update, Platform Monitoring and Control, and BIOS Control and Configuration.
  • Extensibility: Easily extendable to support new PLDM types and custom OEM commands.
  • Integration: Seamlessly integrates with other OpenBMC components for comprehensive system management.

Getting Started

Prerequisites

To build and run PLDM, you need the following dependencies:

  • Meson
  • Ninja

Alternatively, source an OpenBMC ARM/x86 SDK.

Building

To build the PLDM project, follow these steps:

meson setup build && meson compile -C build

To run unit tests

The simplest way of running the tests is as described by the meson man page:

meson test -C build

Alternatively, tests can be run in the OpenBMC CI docker container using these steps.

To enable pldm verbosity

pldm daemon accepts a command line argument --verbose or --v or -v to enable the daemon to run in verbose mode. It can be done via adding this option to the environment file that pldm service consumes.

echo 'PLDMD_ARGS="--verbose"' > /etc/default/pldmd
systemctl restart pldmd

To disable pldm verbosity

rm /etc/default/pldmd
systemctl restart pldmd

Documentation

For complete documentation on the functionality and usage of this repository, please refer to the docs folder.