Changeset 79621 in vbox for trunk/src/VBox/NetworkServices/Dhcpd

Timestamp:

Jul 9, 2019 1:14:53 AM (6 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

131923

Message:

Dhcpd,Main/DHCPServerImpl: LeasesFilename should be plural everywhere. Moved default calc in dhcpd into the config class. bugref:9288

Location:

trunk/src/VBox/NetworkServices/Dhcpd

Files:

• Config.cpp (modified) (2 diffs)
• Config.h (modified) (2 diffs)
• DHCPD.cpp (modified) (4 diffs)
• DHCPD.h (modified) (1 diff)

trunk/src/VBox/NetworkServices/Dhcpd/Config.cpp

@@ -29 +29 @@
 #include <iprt/string.h>
 #include <iprt/uuid.h>
+#include <iprt/cpp/path.h>
 
 #include <VBox/com/com.h>       /* For log initialization. */
@@ -735 +736 @@
         m_strTrunk = "";
 
-    m_strLeaseFilename = pElmServer->findAttributeValue("leaseFilename"); /* optional */
+    m_strLeasesFilename = pElmServer->findAttributeValue("leasesFilename"); /* optional */
+    if (m_strLeasesFilename.isEmpty())
+    {
+        int rc = m_strLeasesFilename.assignNoThrow(getHome());
+        if (RT_SUCCESS(rc))
+            rc = RTPathAppendCxx(m_strLeasesFilename, getBaseName());
+        if (RT_SUCCESS(rc))
+            rc = m_strLeasesFilename.appendNoThrow("-Dhcpd.leases");
+        if (RT_FAILURE(rc))
+            throw ConfigFileError("Unexpected error constructing default m_strLeasesFilename value: %Rrc", rc);
+    }
 
     i_getIPv4AddrAttribute(pElmServer, "IPAddress", &m_IPv4Address);

trunk/src/VBox/NetworkServices/Dhcpd/Config.h

@@ -46 +46 @@
     RTCString       m_strNetwork;       /**< The name of the internal network the DHCP server is connected to. */
     RTCString       m_strBaseName;      /**< m_strNetwork sanitized to be usable in a path component. */
-    RTCString       m_strLeaseFilename; /**< The lease filename if one given.  Dhcpd will pick a default if empty. */
+    RTCString       m_strLeasesFilename;/**< The lease DB filename. */
 
     RTCString       m_strTrunk;         /**< The trunk name of the internal network. */
@@ -93 +93 @@
     const RTCString    &getNetwork() const RT_NOEXCEPT          { return m_strNetwork; }
     const RTCString    &getBaseName() const RT_NOEXCEPT         { return m_strBaseName; }
-    const RTCString    &getLeaseFilename() const RT_NOEXCEPT    { return m_strLeaseFilename; }
+    const RTCString    &getLeasesFilename() const RT_NOEXCEPT   { return m_strLeasesFilename; }
 
     const RTCString    &getTrunk() const RT_NOEXCEPT            { return m_strTrunk; }

trunk/src/VBox/NetworkServices/Dhcpd/DHCPD.cpp

@@ -25 +25 @@
 
 #include <iprt/message.h>
-#include <iprt/cpp/path.h>
 
 
@@ -44 +43 @@
     Assert(pConfig);
     AssertReturn(!m_pConfig, VERR_INVALID_STATE);
-
-    /* leases filename */
-    int rc;
-    if (pConfig->getLeaseFilename().isEmpty())
-        rc = m_strLeasesFilename.assignNoThrow(pConfig->getLeaseFilename());
-    else
-    {
-        rc = m_strLeasesFilename.assignNoThrow(pConfig->getHome());
-        if (RT_SUCCESS(rc))
-            rc = RTPathAppendCxx(m_strLeasesFilename, pConfig->getBaseName());
-        if (RT_SUCCESS(rc))
-            rc = m_strLeasesFilename.appendNoThrow("-Dhcpd.leases");
-    }
+    m_pConfig = pConfig;
+
+    /* Load the lease database, ignoring most issues except being out of memory: */
+    int rc = m_db.init(pConfig);
     if (RT_SUCCESS(rc))
     {
-        /* Load the lease database, ignoring most issues except being out of memory: */
-        rc = m_db.init(pConfig);
-        if (RT_SUCCESS(rc))
-        {
-            rc = i_loadLeases();
-            if (rc != VERR_NO_MEMORY)
-            {
-                m_pConfig = pConfig;
-                rc = VINF_SUCCESS;
-            }
-            else
-            {
-                LogRel(("Ran out of memory loading leases from '%s'.  Try rename or delete the file.\n",
-                        m_strLeasesFilename.c_str()));
-                RTMsgError("Ran out of memory loading leases from '%s'.  Try rename or delete the file.\n",
-                           m_strLeasesFilename.c_str());
-            }
-        }
+        rc = i_loadLeases();
+        if (rc != VERR_NO_MEMORY)
+            return VINF_SUCCESS;
+
+        /** @todo macro for this: */
+        LogRel((   "Ran out of memory loading leases from '%s'.  Try rename or delete the file.\n",
+                   pConfig->getLeasesFilename().c_str()));
+        RTMsgError("Ran out of memory loading leases from '%s'.  Try rename or delete the file.\n",
+                   pConfig->getLeasesFilename().c_str());
     }
     return rc;
@@ -83 +64 @@
 
 /**
- * Load leases from m_strLeasesFilename.
+ * Load leases from pConfig->getLeasesFilename().
  */
 int DHCPD::i_loadLeases() RT_NOEXCEPT
 {
-    return m_db.loadLeases(m_strLeasesFilename);
-}
-
-
-/**
- * Save the current leases to m_strLeasesFilename, doing expiry first.
+    return m_db.loadLeases(m_pConfig->getLeasesFilename());
+}
+
+
+/**
+ * Save the current leases to pConfig->getLeasesFilename(), doing expiry first.
  *
  * This is called after m_db is updated during a client request, so the on disk
@@ -104 +85 @@
 {
     m_db.expire();
-    m_db.writeLeases(m_strLeasesFilename);
+    m_db.writeLeases(m_pConfig->getLeasesFilename());
 }
 

trunk/src/VBox/NetworkServices/Dhcpd/DHCPD.h

@@ -40 +40 @@
     /** The DHCP configuration. */
     const Config   *m_pConfig;
-    /** The lease database filename. */
-    RTCString       m_strLeasesFilename;
     /** The lease database. */
     Db              m_db;