)]}'
{
  "commit": "fd0d94ccbebd91e23219a6364077f4e6fe028535",
  "tree": "3f85401f3f155d3ca49b1ded07fd8476231f692b",
  "parents": [
    "502aa3feb72aa2a16667098cafd4e35eeade6618"
  ],
  "author": {
    "name": "cmchang",
    "email": "cmchang@google.com",
    "time": "Mon Aug 31 09:02:23 2026 +0000"
  },
  "committer": {
    "name": "mdb:gbmc-git-owners",
    "email": "superproject-auto-update@invalid",
    "time": "Tue Oct 06 21:53:35 2026 -0700"
  },
  "message": "phosphor-debug-collector: fix phosphor-dump-manager crash in multihost\ncper patch and rebase patches.\n\nBackground:\nOn multi-host platforms, injecting crashdump/CPER error events on\nsecondary nodes (node 1, node 2) caused phosphor-dump-manager to\ncrash with segmentation faults (SEGV) and std::out_of_range exceptions.\nThis was caused by two main issues:\n1. faultIdTable was defined as a global inline table shared across all\n   host managers in the process. When one host looked up a faultId\n   registered by another host, calling entries.at() on its own map threw\n   std::out_of_range.\n\nSolution:\n1. Move faultIdTable from a global variable into an instance member of\n   Manager, ensuring complete host isolation.\n2. Replace unchecked entries.at() lookups with entries.find() and clean\n   up stale entries if not found.\n\nChanges Made:\n- dump_manager_faultlog.hpp:\n  - Add faultIdTable instance member to Manager.\n- dump_manager_faultlog.cpp:\n  - Update createDump() to pass \u0026faultIdTable to faultlog::Entry.\n  - Safe lookup in faultIdTable via entries.find() instead of at().\n  - Fix faultLogFilePath construction using path operator/.\n- faultlog_dump_entry.hpp:\n  - Remove global faultIdTable and accept faultIdTableMap pointer in\n    Entry constructor.\n- faultlog_dump_entry.cpp:\n  - Erase from faultIdTableMap before parentMap-\u003eerase(id).\n\nTested:\n```\nOn server3 amd platform\nVerified multi-host error injection across node 1 and node 2 on target.\nConfirmed that crashdump and CPER fault log entries are\ncreated, replaced, and deleted without crashing or memory corruption.\n```\n\nFusion-Link:\nplatform5: https://fusion2.corp.google.com/24b076cd-6709-3973-9119-46a49784f9ec\nplatform15: https://fusion2.corp.google.com/fa4fd09b-f45c-31a7-b2b9-a85dc36eeb0f\nplatform18: https://fusion2.corp.google.com/20dcaf3d-8bd4-35e6-a94a-1f0598913f93\nplatform17: https://fusion2.corp.google.com/03c22596-6265-3169-bd40-b040256119ef\nplatform11+ext: https://fusion2.corp.google.com/49713010-0073-3960-839e-fdc8abc29d07\nplatform11: https://fusion2.corp.google.com/07a0f1c6-6bc0-3925-a0f4-1d2785b262e4\nplatform25: https://fusion2.corp.google.com/dc3636e4-3f72-31b7-ab64-4f33623c3b3d\n\nGoogle-Bug-Id:553850432\nChange-Id: If317aae9e394b82b402b108c3b2867e053354dc6\nSigned-off-by: cmchang \u003ccmchang@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "ac5870a01010ad24a187ec4653d1d1504878dd4e",
      "old_mode": 57344,
      "old_path": "meta-gbmc-staging",
      "new_id": "56987b708b6b964eab503a67afe3ca6c04f22afe",
      "new_mode": 57344,
      "new_path": "meta-gbmc-staging"
    }
  ]
}