Commit 47445080 authored by Francis Dupont's avatar Francis Dupont
Browse files

[3899] Moved checkType() first in addPool()

parent 2b6f9782
......@@ -262,6 +262,9 @@ const PoolPtr Subnet::getPool(Lease::Type type, const isc::asiolink::IOAddress&
void
Subnet::addPool(const PoolPtr& pool) {
// check if the type is valid (and throw if it isn't)
checkType(pool->getType());
// Check that the pool is in range with a subnet only if this is
// not a pool of IPv6 prefixes. The IPv6 prefixes delegated for
// the particular subnet don't need to match the prefix of the
......@@ -281,9 +284,6 @@ Subnet::addPool(const PoolPtr& pool) {
/// @todo: Check that pools do not overlap
// check if the type is valid (and throw if it isn't)
checkType(pool->getType());
// Add the pool to the appropriate pools collection
getPoolsWritable(pool->getType()).push_back(pool);
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment