commit 9ede1b74c0feaee6abaa228cd097133dd74edb3e
author William A. Kennington III <wak@google.com> Mon Nov 21 01:59:28 2022 -0800
committer William A. Kennington III <wak@google.com> Fri Dec 09 20:13:15 2022 +0000
tree 3413a920ac61bfe05d2135c1f8d730e9d025e028
parent a4c18d4e50b74f1c19ebdfaaf7073ab89d6524bb

treewide: Codify memory pinning

This enforces movability constraints.

Change-Id: I49dbd43a791c1d70eb7e97f30708231c053901d9
Signed-off-by: William A. Kennington III <wak@google.com>

src/dhcp_configuration.cpp

diff --git a/src/dhcp_configuration.cpp b/src/dhcp_configuration.cpp
index 154bbee..5907f4a 100644
--- a/src/dhcp_configuration.cpp
+++ b/src/dhcp_configuration.cpp
@@ -23,15 +23,16 @@
 using namespace sdbusplus::xyz::openbmc_project::Common::Error;
 
 Configuration::Configuration(sdbusplus::bus_t& bus,
-                             stdplus::const_zstring objPath, Manager& parent) :
+                             stdplus::const_zstring objPath,
+                             stdplus::PinnedRef<Manager> parent) :
     Iface(bus, objPath.c_str(), Iface::action::defer_emit),
-    bus(bus), manager(parent)
+    manager(parent)
 {
     config::Parser conf;
     std::filesystem::directory_entry newest_file;
     time_t newest_time = 0;
     for (const auto& dirent :
-         std::filesystem::directory_iterator(manager.getConfDir()))
+         std::filesystem::directory_iterator(manager.get().getConfDir()))
     {
         struct stat st = {};
         stat(dirent.path().native().c_str(), &st);
@@ -65,8 +66,8 @@
 
     auto name = ConfigIntf::sendHostNameEnabled(value);
 
-    manager.writeToConfigurationFile();
-    manager.reloadConfigs();
+    manager.get().writeToConfigurationFile();
+    manager.get().reloadConfigs();
 
     return name;
 }
@@ -79,8 +80,8 @@
     }
 
     auto name = ConfigIntf::hostNameEnabled(value);
-    manager.writeToConfigurationFile();
-    manager.reloadConfigs();
+    manager.get().writeToConfigurationFile();
+    manager.get().reloadConfigs();
 
     return name;
 }
@@ -93,8 +94,8 @@
     }
 
     auto ntp = ConfigIntf::ntpEnabled(value);
-    manager.writeToConfigurationFile();
-    manager.reloadConfigs();
+    manager.get().writeToConfigurationFile();
+    manager.get().reloadConfigs();
 
     return ntp;
 }
@@ -107,8 +108,8 @@
     }
 
     auto dns = ConfigIntf::dnsEnabled(value);
-    manager.writeToConfigurationFile();
-    manager.reloadConfigs();
+    manager.get().writeToConfigurationFile();
+    manager.get().reloadConfigs();
 
     return dns;
 }