commit 1f5ff9dd442b053445760637c06ff51835b92e8a
author Luka Strizic <lstrz@google.com> Tue Dec 02 19:36:53 2025 +0000
committer Luka Strižić <lstrz@google.com> Tue Dec 02 13:46:17 2025 -0800
tree 94ef8bb186f65ecfda4a02f15f23746a07d4bb5c
parent dd56cc187eed1aa0bd1eef7fbee7e35e31afbdb4

requester: Adds a missing try-catch

A try-catch in propertiesChangedCb was missing. Now it's added.

Tested: Successfully restarted pldmd on a machine with same specs as the one in the bug. https://paste.googleplex.com/5838230064857088
Google-Bug-Id: 464928106
Change-Id: Ic7f3d210dddaa9857841a6f53c9e26db7fae3e92
Signed-off-by: Luka Strizic <lstrz@google.com>

requester/mctp_endpoint_discovery.cpp

diff --git a/requester/mctp_endpoint_discovery.cpp b/requester/mctp_endpoint_discovery.cpp
index db764bf..0afe939 100644
--- a/requester/mctp_endpoint_discovery.cpp
+++ b/requester/mctp_endpoint_discovery.cpp
@@ -310,8 +310,18 @@
 
         if (key == MCTPConnectivityProp)
         {
-            service = pldm::utils::DBusHandler().getService(objPath.c_str(),
-                                                            MCTPInterface);
+            try
+            {
+                service = pldm::utils::DBusHandler().getService(objPath.c_str(),
+                                                                MCTPInterface);
+            }
+            catch (const sdbusplus::exception_t& e)
+            {
+                error(
+                    "Error getting service for path '{PATH}', error - {ERROR}",
+                    "PATH", objPath, "ERROR", e);
+                return;
+            }
             const MctpEndpointProps& epProps =
                 getMctpEndpointProps(service, objPath);