From: Francis Dupont <fdupont@isc.org>
Date: Tue, 9 Jul 2019 14:55:56 +0000 (+0200)
Subject: [576-cb-cmds-empty-value-for-next-server-parameter-is-accepted-which-causes-misconfig... 
X-Git-Tag: Kea-1.6.0-beta2~75
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1c59d63879b922a13e3254139519fdccb902b6ae;p=thirdparty%2Fkea.git

[576-cb-cmds-empty-value-for-next-server-parameter-is-accepted-which-causes-misconfiguration] Moved the body to .cc
---

diff --git a/src/lib/dhcpsrv/network.cc b/src/lib/dhcpsrv/network.cc
index dea0f702c4..27562b2740 100644
--- a/src/lib/dhcpsrv/network.cc
+++ b/src/lib/dhcpsrv/network.cc
@@ -117,6 +117,24 @@ Network::hrModeFromString(const std::string& hr_mode_name) {
     }
 }
 
+Optional<IOAddress>
+Network::getGlobalProperty(Optional<IOAddress> property,
+                           const std::string& global_name) const {
+    if (!global_name.empty() && fetch_globals_fn_) {
+        ConstElementPtr globals = fetch_globals_fn_();
+        if (globals && (globals->getType() == Element::map)) {
+            ConstElementPtr global_param = globals->get(global_name);
+            if (global_param) {
+                std::string global_str = global_param->stringValue();
+                if (!global_str.empty()) {
+                    return (IOAddress(global_str));
+                }
+            }
+        }
+    }
+    return (property);
+}
+
 ElementPtr
 Network::toElement() const {
     ElementPtr map = Element::createMap();
diff --git a/src/lib/dhcpsrv/network.h b/src/lib/dhcpsrv/network.h
index d526f30938..65848eb133 100644
--- a/src/lib/dhcpsrv/network.h
+++ b/src/lib/dhcpsrv/network.h
@@ -589,21 +589,7 @@ protected:
     /// of @c property.
     util::Optional<asiolink::IOAddress>
     getGlobalProperty(util::Optional<asiolink::IOAddress> property,
-                      const std::string& global_name) const {
-        if (!global_name.empty() && fetch_globals_fn_) {
-            data::ConstElementPtr globals = fetch_globals_fn_();
-            if (globals && (globals->getType() == data::Element::map)) {
-                data::ConstElementPtr global_param = globals->get(global_name);
-                if (global_param) {
-                    std::string global_str = global_param->stringValue();
-                    if (!global_str.empty()) {
-                        return (asiolink::IOAddress(global_str));
-                    }
-                }
-            }
-        }
-        return (property);
-    }
+                      const std::string& global_name) const;
 
     /// @brief Returns a value associated with a network using inheritance.
     ///