commit 9e682c5139007f1b61421bf2473f064394276d5b
author Patrick Williams <patrick@stwcx.xyz> Fri Apr 01 17:03:45 2022 -0500
committer Jayanth Othayoth <ojayanth@gmail.com> Fri Apr 08 13:58:16 2022 +0000
tree 90253c302924e2ba58a13968d4b686d1a7369040
parent ad298b6e05a37d229a05ce82746dd77bb8bdb64c

main: move busname claiming later

We should not claim the bus name until the daemon is fully initialized,
especially because we create objects on the dbus that we don't send a
signal for (the manager itself).

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I303fa9a4ab40c93996f9ad64afdc85147c034c86

dump_manager_main.cpp

diff --git a/dump_manager_main.cpp b/dump_manager_main.cpp
index 7ee876a..ce858e4 100644
--- a/dump_manager_main.cpp
+++ b/dump_manager_main.cpp
@@ -65,7 +65,6 @@
 
     // Add sdbusplus ObjectManager for the 'root' path of the DUMP manager.
     sdbusplus::server::manager::manager objManager(bus, DUMP_OBJPATH);
-    bus.request_name(DUMP_BUSNAME);
 
     try
     {
@@ -90,6 +89,9 @@
         phosphor::dump::elog::Watch eWatch(bus, mgr);
         bus.attach_event(eventP.get(), SD_EVENT_PRIORITY_NORMAL);
 
+        // Daemon is all set up so claim the busname now.
+        bus.request_name(DUMP_BUSNAME);
+
         auto rc = sd_event_loop(eventP.get());
         if (rc < 0)
         {