Browse Source

[4097a] Removed merging of global configured options into subnets

Francis Dupont 9 years ago
parent
commit
db7f698d7f
2 changed files with 6 additions and 2 deletions
  1. 2 0
      src/lib/dhcpsrv/cfg_option.h
  2. 4 2
      src/lib/dhcpsrv/parsers/dhcp_parsers.cc

+ 2 - 0
src/lib/dhcpsrv/cfg_option.h

@@ -264,6 +264,8 @@ public:
     /// as a parameter. If an item exists in the destination it is not
     /// copied.
     ///
+    /// @note: this method is not longer used so should become private.
+    ///
     /// @param [out] other Configuration object to merge to.
     void mergeTo(CfgOption& other) const;
 

+ 4 - 2
src/lib/dhcpsrv/parsers/dhcp_parsers.cc

@@ -1256,8 +1256,10 @@ SubnetConfigParser::createSubnet() {
         subnet_->setIface(iface);
     }
 
-    // Merge globally defined options to the subnet specific options.
-    CfgMgr::instance().getStagingCfg()->getCfgOption()->mergeTo(*options_);
+    // Here globally defined options were merged to the subnet specific
+    // options but it is no longer the case (they have a different
+    // and not consecutive priority).
+
     // Copy all options to the subnet configuration.
     options_->copyTo(*subnet_->getCfgOption());
     // Append suboptions to the top-level options.